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:50 | Date Closed: | 2011-06-07 14:08:27 |
Priority: | Minor | Regression? | No |
Status: | Closed/Complete | Components: | 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. |