[asterisk-dev] [Code Review] Support text messages outside of a call

[Russell Bryant]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/1042/
-----------------------------------------------------------

Review request for Asterisk Developers.


Summary
-------

This branch contains a proposal for adding protocol independent support for processing text messages into and out of the dialplan, outside of a call.  The file doc/asterisk-messaging.txt contains more details on the proposal.  The introduction of the document is quoted here:

"    Asterisk has some limited support today for messaging.  The support that
exists primarily includes passing text messages in the context of a call.  The
SIP and IAX2 protocols have support for this, but that's it.

    There are a couple of other messaging protocols that are supported: Skype
and XMPP (Jabber).  The support of these is very minimal and not very integrated
into the architecture of Asterisk since these messages are not in the context of
a phone call.  They provide a combination of dialplan and manager interface
interfaces that are specific to each protocol.  There just is no current
architectural concept of dealing with text messages.

    The purpose of this proposal is to introduce text messaging into the
architecture of Asterisk.  For messaging support to exist in the true spirit of
Asterisk architecture, the design needs to achieve the following two goals:

    a) Protocol Independence
    b) Scriptable message routing

    The rest of this document goes through some details about how these goals
will be achieved in a way that is both architecturally compatible with Asterisk
as well as practical to implement."

----------

In addition to the documented proposal, I have made some good progress on implementation.  Here is what is done so far:

 - core modifications to allow sending incoming messages through the dialplan
 - core modifications to allow outbound messages from the dialplan
 - modifications to res_jabber to allow inbound and outbound messages in the new architecture
 - The beginning of changes to chan_sip, with support for inbound and outbound MESSAGE outside of a call


Diffs
-----

  /trunk/res/res_jabber.c 297183 
  /trunk/main/message.c PRE-CREATION 
  /trunk/main/asterisk.c 297183 
  /trunk/main/channel.c 297183 
  /trunk/include/asterisk/message.h PRE-CREATION 
  /trunk/include/asterisk/_private.h 297183 
  /trunk/include/asterisk/channel.h 297183 
  /trunk/include/asterisk/jabber.h 297183 
  /trunk/doc/asterisk-messaging.txt PRE-CREATION 
  /trunk/configs/jabber.conf.sample 297183 
  /trunk/channels/chan_sip.c 297183 
  /trunk/testsuite-tests/message_unauth/test-config.yaml PRE-CREATION 
  /trunk/testsuite-tests/message_unauth/sipp/message_recv.xml PRE-CREATION 
  /trunk/testsuite-tests/message_unauth/sipp/message.xml PRE-CREATION 
  /trunk/testsuite-tests/message_from_call/sipp/message_recv.xml PRE-CREATION 
  /trunk/testsuite-tests/message_from_call/test-config.yaml PRE-CREATION 
  /trunk/testsuite-tests/message_unauth/configs/ast1/extensions.conf PRE-CREATION 
  /trunk/testsuite-tests/message_unauth/run-test PRE-CREATION 
  /trunk/testsuite-tests/message_from_call/configs/ast1/extensions.conf PRE-CREATION 
  /trunk/testsuite-tests/message_from_call/run-test PRE-CREATION 
  /trunk/testsuite-tests/message_from_call/sipp/call.xml PRE-CREATION 

Diff: https://reviewboard.asterisk.org/r/1042/diff


Testing
-------

The branch contains two automated tests:
 - Use SIPp to send a MESSAGE to Asterisk and then receive a MESSAGE with no calls
 - Use SIPp to send a call to Asterisk that shows how an out-of-call MESSAGE can be sent from the dialplan while processing a normal call

I have also done some manual testing of XMPP messages in and out of Asterisk using this code.


Thanks,

Russell

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.digium.com/pipermail/asterisk-dev/attachments/20101202/1a322b69/attachment.htm