Summary:	ASTERISK-15643: [patch] Voicemail attachments are sent even with attach=no
Reporter:	rbosch (rbosch)	Labels:
Date Opened:	2010-02-16 16:40:59.000-0600	Date Closed:	2010-03-12 13:50:59.000-0600
Priority:	Minor	Regression?	No
Status:	Closed/Complete	Components:	Applications/app_voicemail
Versions:		Frequency of Occurrence
Related Issues:
Environment:		Attachments:	( 0) 20100311__issue16849.diff.txt
Description:	In voicemail.conf, user configuration is as follows: 903 => 1234,User1,user@test.net,,attach=no|saycid=no|envelope=no|delete=no voicemail notification email with an attachment is still sent.
Comments:	By: Leif Madsen (lmadsen) 2010-02-17 14:35:10.000-0600 What happens if you change the pipes (|) to commas (,)? By: rbosch (rbosch) 2010-02-17 14:43:56.000-0600 Still sends the email with attachment, exact same behavior. By: Leif Madsen (lmadsen) 2010-02-23 09:10:48.000-0600 I set the other issue as related as I had seen another issue yesterday which I was also going to relate, as it looked like there may be an issue with the parsing of options for mailboxes. However, the reporter ended up stating that the issue was a configuration error on his end. I'm still keeping these issues related incase there really is a parsing issue with mailboxes. By: Tilghman Lesher (tilghman) 2010-03-04 22:50:06.000-0600 What voicemail storage backend are you using?  Flat file, ODBC, or IMAP? By: Tilghman Lesher (tilghman) 2010-03-10 18:00:26.000-0600 No response from reporter. By: rbosch (rbosch) 2010-03-11 10:14:13.000-0600 Sorry for the delay in answering this question.  The backend I'm using is the default associated with installation of 1.6.2.2 which is the flatfile directory structure. By: Tilghman Lesher (tilghman) 2010-03-11 14:20:18.000-0600 Patch uploaded which should fix it.  Please test and confirm. By: rbosch (rbosch) 2010-03-11 17:14:59.000-0600 Confirming that the patch, applied to 1.6.2.4, fixes the issue.  Voicemail attachment is not included when option says no, and does send attachment when voicemail.conf says attach=yes. By: Digium Subversion (svnbot) 2010-03-12 13:43:25.000-0600 Repository: asterisk Revision: 251989 U   trunk/apps/app_voicemail.c ------------------------------------------------------------------------ r251989 | tilghman | 2010-03-12 13:43:24 -0600 (Fri, 12 Mar 2010) | 8 lines Don't override a user option with the global option. (closes issue ASTERISK-15643) Reported by: ip-rob Patches:       20100311__issue16849.diff.txt uploaded by tilghman (license 14) Tested by: ip-rob ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=251989 By: Digium Subversion (svnbot) 2010-03-12 13:49:07.000-0600 Repository: asterisk Revision: 251992 _U  branches/1.6.0/ U   branches/1.6.0/apps/app_voicemail.c ------------------------------------------------------------------------ r251992 | tilghman | 2010-03-12 13:49:06 -0600 (Fri, 12 Mar 2010) | 15 lines Merged revisions 251989 via svnmerge from https://origsvn.digium.com/svn/asterisk/trunk ........  r251989 | tilghman | 2010-03-12 13:43:23 -0600 (Fri, 12 Mar 2010) | 8 lines    Don't override a user option with the global option.    (closes issue ASTERISK-15643)   Reported by: ip-rob   Patches:         20100311__issue16849.diff.txt uploaded by tilghman (license 14)   Tested by: ip-rob ........ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=251992 By: Digium Subversion (svnbot) 2010-03-12 13:50:32.000-0600 Repository: asterisk Revision: 251993 _U  branches/1.6.1/ U   branches/1.6.1/apps/app_voicemail.c ------------------------------------------------------------------------ r251993 | tilghman | 2010-03-12 13:50:32 -0600 (Fri, 12 Mar 2010) | 15 lines Merged revisions 251989 via svnmerge from https://origsvn.digium.com/svn/asterisk/trunk ........  r251989 | tilghman | 2010-03-12 13:43:23 -0600 (Fri, 12 Mar 2010) | 8 lines    Don't override a user option with the global option.    (closes issue ASTERISK-15643)   Reported by: ip-rob   Patches:         20100311__issue16849.diff.txt uploaded by tilghman (license 14)   Tested by: ip-rob ........ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=251993 By: Digium Subversion (svnbot) 2010-03-12 13:50:58.000-0600 Repository: asterisk Revision: 251994 _U  branches/1.6.2/ U   branches/1.6.2/apps/app_voicemail.c ------------------------------------------------------------------------ r251994 | tilghman | 2010-03-12 13:50:58 -0600 (Fri, 12 Mar 2010) | 15 lines Merged revisions 251989 via svnmerge from https://origsvn.digium.com/svn/asterisk/trunk ........  r251989 | tilghman | 2010-03-12 13:43:23 -0600 (Fri, 12 Mar 2010) | 8 lines    Don't override a user option with the global option.    (closes issue ASTERISK-15643)   Reported by: ip-rob   Patches:         20100311__issue16849.diff.txt uploaded by tilghman (license 14)   Tested by: ip-rob ........ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=251994