[MERGED] osmo-sip-connector[master]: sip: Treat SIP 183 as progress too

Harald Welte gerrit-no-reply at lists.osmocom.org
Mon Feb 13 22:09:12 UTC 2017


Harald Welte has submitted this change and it was merged.

Change subject: sip: Treat SIP 183 as progress too
......................................................................


sip: Treat SIP 183 as progress too

It doesn't fix early media yet but brings us one step
closer to it:

   The 183 (Session Progress) response is used to convey information
   about the progress of the call that is not otherwise classified.  The
   Reason-Phrase, header fields, or message body MAY be used to convey
   more details about the call progress.

Change-Id: Ibf264f251e41c06a7b4839acc0d0853e6400291c
---
M src/sip.c
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Harald Welte: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/src/sip.c b/src/sip.c
index 139c620..afff393 100644
--- a/src/sip.c
+++ b/src/sip.c
@@ -156,7 +156,7 @@
 		if (leg->state == SIP_CC_INITIAL)
 			leg->state = SIP_CC_DLG_CNFD;
 
-		if (status == 180)
+		if (status == 180 || status == 183)
 			call_progress(leg, sip);
 		else if (status == 200)
 			call_connect(leg, sip);

-- 
To view, visit https://gerrit.osmocom.org/1794
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ibf264f251e41c06a7b4839acc0d0853e6400291c
Gerrit-PatchSet: 1
Gerrit-Project: osmo-sip-connector
Gerrit-Branch: master
Gerrit-Owner: Holger Freyther <holger at freyther.de>
Gerrit-Reviewer: Harald Welte <laforge at gnumonks.org>
Gerrit-Reviewer: Jenkins Builder


More information about the gerrit-log mailing list