chan_sip: make progressinband default to no
After the "progressinband" value setting of "never" was updated to never send a 183 this separated its use from the "no" value. Since "never" was the default, but most users probably expect "no" this patch updates the default for the "progressinband" setting to "no." ASTERISK-24835 #close Reported by: Andrew Nagy Review: https://reviewboard.asterisk.org/r/4606/ ........ Merged revisions 434654 from http://svn.asterisk.org/svn/asterisk/branches/13 git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@434655 65c4cc65-6c06-0410-ace0-fbb531ad65f3
This commit is contained in:
parent
8bae18ab93
commit
66f3fd0028
3
CHANGES
3
CHANGES
|
@ -131,6 +131,9 @@ chan_pjsip
|
||||||
more information. Defaults to 'no' as setting it to 'yes' can result in
|
more information. Defaults to 'no' as setting it to 'yes' can result in
|
||||||
many unnecessary messages being sent to the caller.
|
many unnecessary messages being sent to the caller.
|
||||||
|
|
||||||
|
* The configuration setting 'progressinband' now defaults to 'no', which
|
||||||
|
matches the actual behavior of previous versions.
|
||||||
|
|
||||||
res_pjsip
|
res_pjsip
|
||||||
------------------
|
------------------
|
||||||
* A new CLI command has been added: "pjsip show settings", which shows
|
* A new CLI command has been added: "pjsip show settings", which shows
|
||||||
|
|
|
@ -29943,8 +29943,8 @@ static int handle_common_options(struct ast_flags *flags, struct ast_flags *mask
|
||||||
ast_clear_flag(&flags[0], SIP_PROG_INBAND);
|
ast_clear_flag(&flags[0], SIP_PROG_INBAND);
|
||||||
if (ast_true(v->value))
|
if (ast_true(v->value))
|
||||||
ast_set_flag(&flags[0], SIP_PROG_INBAND_YES);
|
ast_set_flag(&flags[0], SIP_PROG_INBAND_YES);
|
||||||
else if (strcasecmp(v->value, "never"))
|
else if (!strcasecmp(v->value, "never"))
|
||||||
ast_set_flag(&flags[0], SIP_PROG_INBAND_NO);
|
ast_set_flag(&flags[0], SIP_PROG_INBAND_NEVER);
|
||||||
} else if (!strcasecmp(v->name, "promiscredir")) {
|
} else if (!strcasecmp(v->name, "promiscredir")) {
|
||||||
ast_set_flag(&mask[0], SIP_PROMISCREDIR);
|
ast_set_flag(&mask[0], SIP_PROMISCREDIR);
|
||||||
ast_set2_flag(&flags[0], ast_true(v->value), SIP_PROMISCREDIR);
|
ast_set2_flag(&flags[0], ast_true(v->value), SIP_PROMISCREDIR);
|
||||||
|
|
|
@ -291,8 +291,8 @@
|
||||||
|
|
||||||
/* Sending PROGRESS in-band settings */
|
/* Sending PROGRESS in-band settings */
|
||||||
#define SIP_PROG_INBAND (3 << 25) /*!< DP: three settings, uses two bits */
|
#define SIP_PROG_INBAND (3 << 25) /*!< DP: three settings, uses two bits */
|
||||||
#define SIP_PROG_INBAND_NEVER (0 << 25)
|
#define SIP_PROG_INBAND_NO (0 << 25)
|
||||||
#define SIP_PROG_INBAND_NO (1 << 25)
|
#define SIP_PROG_INBAND_NEVER (1 << 25)
|
||||||
#define SIP_PROG_INBAND_YES (2 << 25)
|
#define SIP_PROG_INBAND_YES (2 << 25)
|
||||||
|
|
||||||
#define SIP_USEPATH (1 << 27) /*!< GDP: Trust and use incoming Path headers? */
|
#define SIP_USEPATH (1 << 27) /*!< GDP: Trust and use incoming Path headers? */
|
||||||
|
|
|
@ -382,10 +382,10 @@ srvlookup=yes ; Enable DNS SRV lookups on outbound calls
|
||||||
; In order for "noanswer" applications to work, you need to run
|
; In order for "noanswer" applications to work, you need to run
|
||||||
; the progress() application in the priority before the app.
|
; the progress() application in the priority before the app.
|
||||||
|
|
||||||
;progressinband=never ; If we should generate in-band ringing always
|
;progressinband=no ; If we should generate in-band ringing. Always
|
||||||
; use 'never' to never use in-band signalling, even in cases
|
; use 'never' to never use in-band signalling, even in cases
|
||||||
; where some buggy devices might not render it
|
; where some buggy devices might not render it
|
||||||
; Valid values: yes, no, never Default: never
|
; Valid values: yes, no, never Default: no
|
||||||
;useragent=Asterisk PBX ; Allows you to change the user agent string
|
;useragent=Asterisk PBX ; Allows you to change the user agent string
|
||||||
; The default user agent string also contains the Asterisk
|
; The default user agent string also contains the Asterisk
|
||||||
; version. If you don't want to expose this, change the
|
; version. If you don't want to expose this, change the
|
||||||
|
|
Loading…
Reference in New Issue