[Home]

Summary:ASTERISK-12477: vmail.cgi does not look in users.conf -- misses users defined through AsteriskGUI
Reporter:George Pajari (netvoice)Labels:
Date Opened:2008-07-29 03:25:50Date Closed:2008-12-10 09:47:05.000-0600
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Applications/app_voicemail
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) 20080814__bug13187.diff.txt
( 1) 20080911__bug13187.diff.txt
Description:vmail.cgi only looks in voice.conf and so does not see the voicemail accounts defined through the AsteriskGUI which are stored in users.conf
Comments:By: Leif Madsen (lmadsen) 2008-08-11 09:35:33

Thanks for taking this Sean! Assigned per discussion in #asterisk-bugs.
By: Tilghman Lesher (tilghman) 2008-08-11 09:57:21

Apparently I got to the discussion a bit late.
By: Leif Madsen (lmadsen) 2008-08-13 15:16:05

Well there wasn't so much a discussion as a, "hey, go ahead and assign this to me" :)
By: msales (msales) 2008-08-29 14:36:49

I installed this patch.  Afterwards I couldn't login with users from users.conf or voicemail.conf and had to revert.
By: Jay Eames (alchamist) 2008-09-11 01:52:25

Patch not working with users.conf but still works with voicemail.conf - using asterisk-gui v2 and asterisk 1.4.21.2
By: Tilghman Lesher (tilghman) 2008-09-11 13:04:52

Aha, found the problem.  Try this new patch.
By: msales (msales) 2008-09-11 13:32:25

I'm having issues downloading the patch:

patching file contrib/scripts/vmail.cgi
Hunk #1 FAILED at 193.
1 out of 1 hunk FAILED -- saving rejects to file contrib/scripts/vmail.cgi.rej
By: Sean Bright (seanbright) 2008-09-11 14:20:23

If you applied Corydon's previous patch, you have to revert your vmail.cgi back to the version that came from the source distribution before applying the new patch.
By: Jay Eames (alchamist) 2008-09-12 02:24:44

Just applied the patch to the original vmail.cgi, and it didn't work. Did a little digging and I think it's related to the whitespace in the vars. If I trim $var and $value in check_login_users then it seems to work ok.
By: Digium Subversion (svnbot) 2008-09-19 19:40:33

Repository: asterisk
Revision: 143736

U   branches/1.4/contrib/scripts/vmail.cgi

------------------------------------------------------------------------
r143736 | seanbright | 2008-09-19 19:40:33 -0500 (Fri, 19 Sep 2008) | 9 lines

Make vmail.cgi work with mailboxes defined in users.conf, too.

(closes issue ASTERISK-12477)
Reported by: netvoice
Patches:
     20080911__bug13187.diff.txt uploaded by Corydon76 (license 14)
     (Slightly modified to take alchamist's comments on mantis into account)
Tested by: msales, alchamist, seanbright

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

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