[Home]

Summary:ASTERISK-24250: [patch] Voicemail with multi-recipients To: header fix
Reporter:abelbeck (abelbeck)Labels:
Date Opened:2014-08-20 07:51:56Date Closed:2014-11-08 18:17:29.000-0600
Priority:MajorRegression?No
Status:Closed/CompleteComponents:Applications/app_voicemail
Versions:13.0.0-beta1 Frequency of
Occurrence
Constant
Related
Issues:
is caused byASTERISK-24045 [patch]Voicemail to email at multiple email addresses
Environment:AstLinuxAttachments:( 0) voicemail-420577-to-comma-fix.diff
Description:Per revision 420577 adding multiple recipients to voicemail emails, enclosed is a fix to add the proper number of comma's in multiple To: recipients.

Originally 420577 looked ahead for a '|' which is off-by-one, but instead use the emailsbuf from strsep() will return NULL when no more comma's are needed.

BTW, I applied the 420577 revision patch to both Asterisk 1.8 and 11, it applies without fuzz and works properly.
Comments:By: abelbeck (abelbeck) 2014-08-20 07:52:36.905-0500

Fix to add the proper number of comma's in multiple To: recipients

By: abelbeck (abelbeck) 2014-08-20 07:55:56.778-0500

Reference ASTERISK-24045

By: Matt Jordan (mjordan) 2014-09-04 16:16:08.260-0500

I'm going to assign this to Jason to at least take a look at the patch, since he did the original work on ASTERISK-24045.

By: Matt Jordan (mjordan) 2014-09-08 14:00:15.688-0500

{quote}
<Qwell> mjordan: ASTERISK-24250 - I can't comment or anything.  Looks correct.
{quote}

Which is a +1 in my book.

By: Zane Conkle (zconkle) 2014-10-27 21:28:33.887-0500

FYI: Had to manually merge this in to 13.0.0 to get multiple voicemail working.