Summary: | ASTERISK-24250: [patch] Voicemail with multi-recipients To: header fix | ||||
Reporter: | abelbeck (abelbeck) | Labels: | |||
Date Opened: | 2014-08-20 07:51:56 | Date Closed: | 2014-11-08 18:17:29.000-0600 | ||
Priority: | Major | Regression? | No | ||
Status: | Closed/Complete | Components: | Applications/app_voicemail | ||
Versions: | 13.0.0-beta1 | Frequency of Occurrence | Constant | ||
Related Issues: |
| ||||
Environment: | AstLinux | Attachments: | ( 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. |