[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:                putnopvut
====================================================================== 
Project:                    Asterisk
Issue ID:                   13142
Category:                   Applications/app_voicemail/IMAP
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     assigned
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-08-01 09:21 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.
====================================================================== 

---------------------------------------------------------------------- 
 (0090972) svnbot (reporter) - 2008-08-01 09:21
 http://bugs.digium.com/view.php?id=13142#c90972 
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 135067

U   trunk/apps/app_voicemail.c
U   trunk/configs/voicemail.conf.sample

------------------------------------------------------------------------
r135067 | mmichelson | 2008-08-01 09:21:37 -0500 (Fri, 01 Aug 2008) | 13
lines

IMAP storage functioned under the assumption that folders
such as "Work" and "Family" would be subfolders of the
INBOX. This is an invalid assumption to make, but it could
be desirable to set up folders in this manner, so a new
option for voicemail.conf, "imapparentfolder" has been
added to allow for this.

(closes issue http://bugs.digium.com/view.php?id=13142)
Reported by: jaroth
Patches:
      parentfolder.patch uploaded by jaroth (license 50)


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

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

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2008-08-01 09:21 svnbot         Checkin                                      
2008-08-01 09:21 svnbot         Note Added: 0090972                          
======================================================================