----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviewboard.asterisk.org/r/4314/#review14081 -----------------------------------------------------------
I'm not sure I agree with this change. This can happen, like you said, due to MWI being indicated before an eventual registration. However, this can also happen due to a misconfiguration (either in Asterisk or the endpoint the MWI is being sent to) and not end up fixing itself without intervening. In the case of a misconfiguration, a warning goes a lot further towards pointing out the issue than a verbose message. On the one hand, I can understand the annoyance in seeing a warning when things actually are fine, but I also don't want to withhold a message when something is genuinely wrong. I wonder if there is some sort of compromise that can be had here. - Mark Michelson On Jan. 2, 2015, 10:41 p.m., George Joseph wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviewboard.asterisk.org/r/4314/ > ----------------------------------------------------------- > > (Updated Jan. 2, 2015, 10:41 p.m.) > > > Review request for Asterisk Developers. > > > Repository: Asterisk > > > Description > ------- > > When res_pjsip loads and an endpoint auto-subscribes a mailbox for mwi, if a > contact hasn't registered yet, res_pjsip_mwi spits out a warning. This is a > perfectly normal situation though and doesn't require something as serious as > a warning. It's also self correcting. The device will start getting mwi as > soon as it registers. > > This patch changes the warning to a verb/3. > > > Diffs > ----- > > branches/13/res/res_pjsip_mwi.c 430163 > > Diff: https://reviewboard.asterisk.org/r/4314/diff/ > > > Testing > ------- > > > Thanks, > > George Joseph > >
-- _____________________________________________________________________ -- 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
