Asterisk
  1. Asterisk
  2. ASTERISK-4679

[patch] MWI compliance with RFC3842 - "Message-Account" in NOTIFY header

    Details

    • Type: Bug Bug
    • Status: Closed
    • Severity: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Target Release Version/s: None
    • Labels:
      None
    • Mantis ID:
      4801
    • Regression:
      No

      Description

      currently the MWI NOTIFY messages do not contain a "Message-Account" header (in opposition to RFC3842).
      this is patch necessary to make the voicemail retrieve button on snom phones work.
      there is a new config file option to set the dialplan extension which should be used for the NOTIFY messages (defaults to 'asterisk' for not breaking MWI buttons of other phones).

                • ADDITIONAL INFORMATION ******

      this patch is an offspring of ASTERISK-3564

        Activity

        Hide
        Frank Sautter added a comment -

        mark: snom phones are using the number presented in the mwi-notify as the extension dailled when pressing the 'Retrieve Messages' button. therefore it should be configurable and different from the mailbox number! the one thing that might be dicussed is the default value used in the mwi-notify (currently i'm using 'asterisk' for not breaking current behaviour - maybe this should be the mailbox number?)

        Show
        Frank Sautter added a comment - mark: snom phones are using the number presented in the mwi-notify as the extension dailled when pressing the 'Retrieve Messages' button. therefore it should be configurable and different from the mailbox number! the one thing that might be dicussed is the default value used in the mwi-notify (currently i'm using 'asterisk' for not breaking current behaviour - maybe this should be the mailbox number?)
        Hide
        Frank Sautter added a comment -

        updated to revision 801

        Show
        Frank Sautter added a comment - updated to revision 801
        Hide
        Olle Johansson added a comment -

        We never use the mailbox ID as an URI, we use the peer name for subscribes. In this case, the phones use this ID as an URI to call back to check the voicemail, which is an extension, either a generic one or one that by pattern includes a voicemail ID (8500 or maybe 8500xxxx, like 85001234) - it depends on the local dialplan. I think this new config option makes sense as we do not expose mailbox IDs.

        Show
        Olle Johansson added a comment - We never use the mailbox ID as an URI, we use the peer name for subscribes. In this case, the phones use this ID as an URI to call back to check the voicemail, which is an extension, either a generic one or one that by pattern includes a voicemail ID (8500 or maybe 8500xxxx, like 85001234) - it depends on the local dialplan. I think this new config option makes sense as we do not expose mailbox IDs.
        Hide
        Kevin P. Fleming (Inactive) added a comment -

        Committed to CVS HEAD, thanks!

        Show
        Kevin P. Fleming (Inactive) added a comment - Committed to CVS HEAD, thanks!
        Hide
        Digium Subversion added a comment -

        Repository: asterisk
        Revision: 6365

        U trunk/channels/chan_sip.c
        U trunk/configs/sip.conf.sample

        ------------------------------------------------------------------------
        r6365 | kpfleming | 2008-01-15 15:44:53 -0600 (Tue, 15 Jan 2008) | 2 lines

        support configurable Message-Account value in NOTIFY messages (issue ASTERISK-4679)

        ------------------------------------------------------------------------

        http://svn.digium.com/view/asterisk?view=rev&revision=6365

        Show
        Digium Subversion added a comment - Repository: asterisk Revision: 6365 U trunk/channels/chan_sip.c U trunk/configs/sip.conf.sample ------------------------------------------------------------------------ r6365 | kpfleming | 2008-01-15 15:44:53 -0600 (Tue, 15 Jan 2008) | 2 lines support configurable Message-Account value in NOTIFY messages (issue ASTERISK-4679 ) ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=6365

          People

          • Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development