[Asterisk-code-review] res/res pjsip sip message: added sip message event notifier (asterisk[master])

[Joshua Colp]

Joshua Colp has posted comments on this change. ( https://gerrit.asterisk.org/10715 )

Change subject: res/res_pjsip_sip_message: added sip message event notifier
......................................................................


Patch Set 3:

(1 comment)

Others can chime in on their thoughts as well.

https://gerrit.asterisk.org/#/c/10715/3//COMMIT_MSG
Commit Message:

https://gerrit.asterisk.org/#/c/10715/3//COMMIT_MSG@9
PS3, Line 9: This module is sending event messages for giving channel's SIP
           : messages. This module will be helpful for checking more detail SIP
           : messages. Because of the ISDN cause codes are not 1:1 compatible with
           : SIP response code, it was hard to know what was the real problem. Not
           : only for response code, this would be useful for examining the received
           : SIP messages easily.
Personally I don't like that this just sends the SIP messages to ARI. It really wasn't made for that and it's not really fa debugging mechanism, and it has a performance penalty.

If the real reason is the cause codes, then we already have protocol specific reason support[1] and this could likely be extended to be sent out over ARI as well.

[1] https://wiki.asterisk.org/wiki/pages/viewpage.action?pageId=20185363



-- 
To view, visit https://gerrit.asterisk.org/10715
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I1dedcc678265ebb74d35e7a2f58bc4b5084bcfe1
Gerrit-Change-Number: 10715
Gerrit-PatchSet: 3
Gerrit-Owner: sungtae kim <pchero21 at gmail.com>
Gerrit-Reviewer: Jenkins2 (1000185)
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: sungtae kim <pchero21 at gmail.com>
Gerrit-Comment-Date: Wed, 28 Nov 2018 12:36:21 +0000
Gerrit-HasComments: Yes
Gerrit-HasLabels: No
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20181128/7fadd534/attachment.html>