[Asterisk-code-review] chan_sip.c: Enable pickup on channel after having received 183 Progress (asterisk[16])

[Mark Petersen]

Mark Petersen has uploaded this change for review. ( https://gerrit.asterisk.org/c/asterisk/+/17782 )


Change subject: chan_sip.c: Enable pickup on channel after having received 183 Progress
......................................................................

chan_sip.c: Enable pickup on channel after having received 183 Progress

fix issue with pickup on device that uses "183 Progress" and not "180 Ringing"

ASTERISK-29832

Change-Id: I4c7d223870f8ce9a7354e0f73d4e4cb2e8b58841
---
M channels/chan_sip.c
A doc/CHANGES-staging/chan_sip_pickup_on_183_progress.txt
2 files changed, 10 insertions(+), 0 deletions(-)



  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/82/17782/1

diff --git a/channels/chan_sip.c b/channels/chan_sip.c
index e19e19d..097f2ee 100644
--- a/channels/chan_sip.c
+++ b/channels/chan_sip.c
@@ -24217,6 +24217,10 @@
 					&update_connected);
 			}
 			sip_handle_cc(p, req, AST_CC_CCNR);
+			if (ast_channel_state(p->owner) != AST_STATE_UP) {
+				ast_setstate(p->owner, AST_STATE_RINGING);
+			}
+
 		}
 		if (find_sdp(req)) {
 			if (p->invitestate != INV_CANCELLED) {
diff --git a/doc/CHANGES-staging/chan_sip_pickup_on_183_progress.txt b/doc/CHANGES-staging/chan_sip_pickup_on_183_progress.txt
new file mode 100644
index 0000000..6ecbae4
--- /dev/null
+++ b/doc/CHANGES-staging/chan_sip_pickup_on_183_progress.txt
@@ -0,0 +1,6 @@
+Subject: chan_sip
+Subject: Channels
+
+Enable pickup on channel after having received 183 Progress
+
+fix issue with pickup on device that uses "183 Progress" and not "180 Ringing"

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

Gerrit-Project: asterisk
Gerrit-Branch: 16
Gerrit-Change-Id: I4c7d223870f8ce9a7354e0f73d4e4cb2e8b58841
Gerrit-Change-Number: 17782
Gerrit-PatchSet: 1
Gerrit-Owner: Mark Petersen <bugs.digium.com at zombie.dk>
Gerrit-MessageType: newchange
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20220104/11399fdf/attachment.html>