[Asterisk-code-review] chan_sip.c Session timers get removed on UPDATE (asterisk[16])

[Mark Petersen]

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


Change subject: chan_sip.c Session timers get removed on UPDATE
......................................................................

chan_sip.c Session timers get removed on UPDATE

fix that if asterisk recive an SIP refer with Session-Timers UAC that asteriks maintain Session-Timers when sending UPDATE

ASTERISK-29843

Change-Id: I8e9a21c13bf757fa34d778f49ba3cf859b29ae5c
---
M channels/chan_sip.c
A doc/CHANGES-staging/chan_sip_session-timer_on_update.txt
2 files changed, 7 insertions(+), 1 deletion(-)



  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/87/17787/1

diff --git a/channels/chan_sip.c b/channels/chan_sip.c
index 07cef0d..95ecd5b 100644
--- a/channels/chan_sip.c
+++ b/channels/chan_sip.c
@@ -12505,7 +12505,7 @@
 	   in-dialog messages.
 	*/
 	if (p->stimer && p->stimer->st_active == TRUE && p->stimer->st_active_peer_ua == TRUE
-	    && sipmethod == SIP_INVITE) {
+	    && (sipmethod == SIP_INVITE || sipmethod == SIP_UPDATE)) {
 		char se_hdr[256];
 		snprintf(se_hdr, sizeof(se_hdr), "%d;refresher=%s", p->stimer->st_interval,
 			p->stimer->st_ref == SESSION_TIMER_REFRESHER_US ? "uac" : "uas");
diff --git a/doc/CHANGES-staging/chan_sip_session-timer_on_update.txt b/doc/CHANGES-staging/chan_sip_session-timer_on_update.txt
new file mode 100644
index 0000000..0782e9e
--- /dev/null
+++ b/doc/CHANGES-staging/chan_sip_session-timer_on_update.txt
@@ -0,0 +1,6 @@
+Subject: chan_sip
+
+Session timers get removed on UPDATE
+fix that if asterisk recive an SIP refer with Session-Timers UAC that asteriks maintain Session-Timers when sending UPDATE
+
+

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

Gerrit-Project: asterisk
Gerrit-Branch: 16
Gerrit-Change-Id: I8e9a21c13bf757fa34d778f49ba3cf859b29ae5c
Gerrit-Change-Number: 17787
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/20220107/cda20400/attachment.html>