[Asterisk-code-review] chan pjsip: Register for "BEFORE MEDIA" responses (asterisk[master])

George Joseph asteriskteam at digium.com
Thu Jun 7 09:32:27 CDT 2018


Hello Jenkins2, 

I'd like you to reexamine a change. Please visit

    https://gerrit.asterisk.org/9132

to look at the new patch set (#2).

Change subject: chan_pjsip:  Register for "BEFORE_MEDIA" responses
......................................................................

chan_pjsip:  Register for "BEFORE_MEDIA" responses

chan_pjsip wasn't registering for "BEFORE_MEDIA" responses which meant
it was not updating HANGUPCAUSE for 4XX responses.  If the remote end
sent a "180 Ringing", then a "486 Busy", the hangup cause was left at
"180 Normal Clearing".

* Removed chan_pjsip_incoming_response from the original session
  supplement (which was handling only "AFTER MEDIA") and added it to a
  new session supplement which accepts both "BEFORE_MEDIA" and
  "AFTER_MEDIA".

* Also cleaned up some cleanup code in load module.

ASTERISK-27902

Change-Id: If9b860541887aca8ac2c9f2ed51ceb0550fb007a
---
M channels/chan_pjsip.c
1 file changed, 15 insertions(+), 5 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/32/9132/2
-- 
To view, visit https://gerrit.asterisk.org/9132
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: If9b860541887aca8ac2c9f2ed51ceb0550fb007a
Gerrit-Change-Number: 9132
Gerrit-PatchSet: 2
Gerrit-Owner: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Jenkins2
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20180607/908fe910/attachment-0001.html>


More information about the asterisk-code-review mailing list