> On June 19, 2014, 9:20 a.m., Matt Jordan wrote: > > Update: > > > > * The sample config with the new option > > * The UPGRADE.txt file documenting the new option > >
This patch should also be written against 1.8, as that is where this issue was first introduced. - Matt ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/3633/#review12193 ----------------------------------------------------------- On June 19, 2014, 9:20 a.m., Pavel Troller wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/3633/ > ----------------------------------------------------------- > > (Updated June 19, 2014, 9:20 a.m.) > > > Review request for Asterisk Developers. > > > Bugs: ASTERISK-23897 > https://issues.asterisk.org/jira/browse/ASTERISK-23897 > > > Repository: Asterisk > > > Description > ------- > > The change introduced in revision 413765 limits opening the media channel and > sending the PROGRESS control frame only to cases, when the Progress > Indication with value "Inband Info now available" is present in the SETUP ACK > message. However, there are exchanges (like Ericsson/Aastra MD110 BC13 or > Avaya Definity (revision unknown)), which don't send this PI in SETUP ACK, > but they send regular dial tone. Now the dial tone (and any subsequent audio > information until some progress indication is received from the PBX) cannot > be heard. My proposed solution is to introduce a new config option. I've > called it "alwayssendprogress" and it's a binary no/yes option. In the "no" > status, it keeps the new behaviour in place. In the "yes" status, it restores > the old one. > A better naming is possible, any proposals are welcome. > The supplied patch was tested on current V11 SVN and it seems to be working > as expected. > > > Diffs > ----- > > /branches/11/channels/sig_pri.c 416250 > /branches/11/channels/sig_pri.h 416250 > /branches/11/channels/chan_dahdi.c 416250 > > Diff: https://reviewboard.asterisk.org/r/3633/diff/ > > > Testing > ------- > > Regularly using the patch on three systems with no visible problems. > > > Thanks, > > Pavel Troller > >
-- _____________________________________________________________________ -- 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
