[asterisk-bugs] [Asterisk 0013142]: [patch] Save to alternate folder throws errors due to incorrect IMAP mailbox specification in Cyrus IMAP

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=13142 
====================================================================== 
Reported By:                jaroth
Assigned To:                
====================================================================== 
Project:                    Asterisk
Issue ID:                   13142
Category:                   Applications/app_voicemail/IMAP
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     new
Asterisk Version:           SVN 
SVN Branch (only for SVN checkouts, not tarball releases):  trunk 
SVN Revision (number only!): 122710 
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             2008-07-23 13:50 CDT
Last Modified:              2008-07-31 11:44 CDT
====================================================================== 
Summary:                    [patch] Save to alternate folder throws errors due
to incorrect IMAP mailbox specification in Cyrus IMAP
Description: 
imap_mailbox_name() can create an incorrect mailbox name, specifically when
specifying a folder name (other than INBOX) in Cyrus IMAP.  Currently the
folder name would be specified as INBOX/Family, for instance.  Cyrus (and
possibly other IMAP implementations does not like the INBOX prefix). 
Instead of making a Cyrus-specific patch (the concensus is NOT to create
IMAP implementation-specific code), this patch uses an additional
(optional) parameter from the voicemail.conf file called imapparentfolder. 
The default value would be INBOX.  If this parameter is not set, INBOX will
not be prepended to the mailbox name.
====================================================================== 

---------------------------------------------------------------------- 
 (0090917) putnopvut (administrator) - 2008-07-31 11:44
 http://bugs.digium.com/view.php?id=13142#c90917 
---------------------------------------------------------------------- 
I think the reason that I coded the folder name as a subfolder of INBOX was
because at the time, I had no idea how to set up other mboxes on my system
to act as "folders." This definitely isn't a Cyrus-specific issue, and I
don't know of any IMAP servers that require folders to be subfolders of
INBOX. 

The parent folder idea still seems good though since it will allow for
users who do wish to set up their voicemail folders as subfolders of INBOX
to do so. The patch looks good, so I will commit it as soon as I finish
merging Asterisk 1.4's imap_consistency branch up to trunk today. 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2008-07-31 11:44 putnopvut      Note Added: 0090917                          
======================================================================