[Home]

Summary:ASTERISK-11855: app_voicemail doesn't correctly handle multiple names assigned to a mailbox
Reporter:Jared Smith (jsmith)Labels:
Date Opened:2008-04-15 08:57:50Date Closed:2011-06-07 14:08:27
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Applications/app_voicemail
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:
Description:I've setup a testing context in voicemail.conf to show a colleague how you can assign multiple names to the same mailbox (for use by the Directory() application).

[testing]
123 => 1111,Jared Smith
123 => d,David Duffett
123 => d,Mark Spencer

This works great, except that the VoiceMailMain() application now no longer is able to access mailbox 123 with PIN number 1111.  Apparently there's a regression here.

****** STEPS TO REPRODUCE ******

In voicemail.conf:

[testing]
123 => 1111,Jared Smith
123 => d,John Public
123 => d,Mark Spencer

In extensions.conf:

exten => 123,1,Directory(testing)
exten => 124,1,Directory(testing,,f)
exten => 125,1,VoiceMailMain(@testing)
Comments:By: Tilghman Lesher (tilghman) 2008-04-15 10:06:50

I have to wonder if the old way was arguably a bug, that you could define two mailboxes with the same number and expect it to work.  I'd say that you were previously exploiting a bug to get a feature that you would have liked to have had, and this isn't really a regression, because of that.

By: Tilghman Lesher (tilghman) 2008-04-15 10:12:57

In fact, if you reverse the order in voicemail.conf, you could probably still make it work.

By: Jared Smith (jsmith) 2008-04-15 12:59:12

No, I wouldn't say that I was exploiting something, it was functionality that was intentionally added to give the ability to have multiple names for a given mailbox... hence the requirement that you put 'd' in the password field, which stands for "duplicate".  This functionality has been in Asterisk since at least 0.50, if not earlier.

Yes, placing the duplicates *before* the mailbox with the actual PIN solves the problem, but that either needs to be well documented or fixed in the implementation.

By: Jared Smith (jsmith) 2008-04-15 13:27:58

After talking this over with Qwell in #asterisk-bugs, apparently I'm full of bologna and have no idea what I'm talking about.  The code has never been there to correctly deal with multiple aliases for a mailbox.  Please close the bug.

By: Jason Parker (jparker) 2008-04-15 14:25:07

Closing, but I'm going to look into adding an alias => option to voicemail.conf, so this can be done in a uniform way.