---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4592/#review15093
---
I think this change is widening the scope of the inband_progress option beyond
what it was intended to do. The intended purpose of inband_progress is to
determine whether AST_CONTROL_RINGING should be handled by sending a 180
(out-of-band ringing) or 183 with early media (in-band ringing) to the caller.
With this change, setting inband_progress to no also prevents any early media
from being sent, which is likely not what people intend when they set the
option.
However, the ability to prevent early media by transforming a progress
indication into a 180 Ringing makes sense as something someone may want to do.
I just think this behavior should be relegated to a separate option.
- Mark Michelson
On April 6, 2015, 6:32 p.m., Kevin Harwell wrote:
---
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/4592/
---
(Updated April 6, 2015, 6:32 p.m.)
Review request for Asterisk Developers.
Bugs: ASTERISK-24835
https://issues.asterisk.org/jira/browse/ASTERISK-24835
Repository: Asterisk
Description
---
The issue started out as early media not playing in chan_sip. However, it was
then reported that once the 'progressinband' option was set to 'yes' it would
work, but that the default behavior was different than that of chan_pjsip.
chan_pjsip's 'inband_progress' option defaults to 'no' as well, but it still
sends a 183. It turns out chan_pjsip was always sending a 183 despite
whatever the 'inband_progress' option was set to. Fixed it so chan_pjsip only
sends a 183 if 'inband_progress' is set to 'yes'.
Diffs
-
branches/13/channels/chan_pjsip.c 434021
Diff: https://reviewboard.asterisk.org/r/4592/diff/
Testing
---
Duplicated the issue with chan_pjsip always sending the 183. After the patch
it will only send it when 'inband_progress' is set to 'yes'
Thanks,
Kevin Harwell
--
_
-- Bandwidth and Colocation Provided by http://www.api-digital.com --
asterisk-dev mailing list
To UNSUBSCRIBE or update options visit:
http://lists.digium.com/mailman/listinfo/asterisk-dev
