[Home]

Summary:ASTERISK-09868: Voicemail does not attach audio file to email when sending to multiple voicemail boxes with ODBC storage
Reporter:Leif Madsen (lmadsen)Labels:
Date Opened:2007-07-12 15:20:40Date Closed:2007-08-03 15:04:15
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Applications/app_voicemail
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) bug0010194
Description:When using something like Voicemail(100@default&101@default&102@default) the extra voicemail boxes do not get the audio file attached to the email because the system does not pull out the audio file from the database so sendmail can grab the file from the file system.

****** ADDITIONAL INFORMATION ******

Patch forthcoming from Vlad.
Comments:By: Владислав (vovochka) 2007-07-17 15:22:14

Pulls the file out of the database to the file system, sends mail, and desposes it. Needs testing.
By: Leif Madsen (lmadsen) 2007-07-18 11:37:08

Marking as ready for testing. Will test myself.
By: Leif Madsen (lmadsen) 2007-07-18 11:49:04

Sorry... did not work :(

   -- Executing [reject@sub-route:9] VoiceMail("SIP/ss1tampa.mixnetworks.com-088ab680", "107@telcel&103@telcel&201@telcel|s") in new stack
   -- <SIP/ss1tampa.mixnetworks.com-088ab680> Playing 'beep' (language 'en')
   -- Recording the message
   -- x=0, open writing:  /var/spool/asterisk/voicemail/telcel/107/tmp/nW1ALK format: wav, 0x88bcbf0
   -- User hung up
[Jul 18 13:02:13] NOTICE[16279]: app_voicemail.c:2623 copy_message: Copying message from 107@telcel to 103@telcel
[Jul 18 13:02:13] WARNING[16279]: app_voicemail.c:1282 delete_file: SQL Execute error!
[DELETE FROM voicemessages WHERE dir=? AND msgnum=?]

[Jul 18 13:02:13] WARNING[16279]: app_voicemail.c:1702 base_encode: Failed to open log file: /var/spool/asterisk/voicemail/telcel/103/INBOX/msg0005.wav: No such file or directory
[Jul 18 13:02:13] NOTICE[16279]: app_voicemail.c:2623 copy_message: Copying message from 107@telcel to 201@telcel
[Jul 18 13:02:13] WARNING[16279]: app_voicemail.c:1282 delete_file: SQL Execute error!
[DELETE FROM voicemessages WHERE dir=? AND msgnum=?]

[Jul 18 13:02:13] WARNING[16279]: app_voicemail.c:1702 base_encode: Failed to open log file: /var/spool/asterisk/voicemail/telcel/201/INBOX/msg0001.wav: No such file or directory
[Jul 18 13:02:13] WARNING[16279]: app_voicemail.c:1282 delete_file: SQL Execute error!
[DELETE FROM voicemessages WHERE dir=? AND msgnum=?]

 == Parsing '/var/spool/asterisk/voicemail/telcel/107/INBOX/msg0006.txt': Found
[Jul 18 13:02:13] WARNING[16279]: app_voicemail.c:1093 retrieve_file: SQL Get Data error!
[SELECT * FROM voicemessages WHERE dir=? AND msgnum=?]

 == Spawn extension (sub-route, reject, 9) exited non-zero on 'SIP/ss1tampa.mixnetworks.com-088ab680'
By: Leif Madsen (lmadsen) 2007-07-18 11:58:22

Oops this is the wrong bug. I need to apply 10193 first to make sure the audio actually gets copied to the mailbox. Will reply back after I test again.
By: Leif Madsen (lmadsen) 2007-07-18 12:25:22

This patch works along with 10193. Once both patches are applied, audio is now in the mailboxes and attached to the email.

Matt Nicholson was saying this might be a bit of a hack, so someone may wish to review it and see if there is a better solution before committing, but this does work fine.
By: Brett Bryant (bbryant) 2007-07-18 12:27:34

vovochka, you need to sign a license agreement by clicking "sign license" at the top of the page, and reupload your patch. We cannot look at your code unless you've signed a license with us first.
By: Leif Madsen (lmadsen) 2007-07-18 12:35:45

Hrmmm... vovochka actually works at Digium in the Custom Development department :)
By: Leif Madsen (lmadsen) 2007-07-18 12:47:37

Vlad works at Digium, but seems he needs to click the little license button or something. Either that or maybe someone at Digium can override this and setup his license (I can only assume he had to sign something when he was hired anyways to state that Digium gets any code written by him, but I'll leave that to someone else to decide what to do with this)
By: Russell Bryant (russell) 2007-07-18 13:04:24

he needs to just fill it out and it will get marked as accepted
By: Leif Madsen (lmadsen) 2007-08-02 12:06:17

OK, Vlad submitted his license agreement. Deleted the original file because apparently you can't submit the license after a patch has been uploaded, so will get Vlad to re-upload.

LJAM.
By: Leif Madsen (lmadsen) 2007-08-02 12:09:03

License has been signed... file will be reuploaded, so changing status.
By: Владислав (vovochka) 2007-08-03 10:40:30

Uploaded a patch again after signing agreement.
By: Digium Subversion (svnbot) 2007-08-03 14:56:40

Repository: asterisk
Revision: 78101

------------------------------------------------------------------------
r78101 | russell | 2007-08-03 14:56:38 -0500 (Fri, 03 Aug 2007) | 10 lines

(closes issue ASTERISK-9868)
Reported by: blitzrage
Patches:
     bug0010194 uploaded by vovochka
Tested by: blitzrage

Fix a problem when you call Voicemail() with multiple mailboxes specified and
ODBC_STORAGE is in use.  The audio part of the message was only given to the
first mailbox specified.

------------------------------------------------------------------------
By: Digium Subversion (svnbot) 2007-08-03 15:04:15

Repository: asterisk
Revision: 78102

------------------------------------------------------------------------
r78102 | russell | 2007-08-03 15:04:15 -0500 (Fri, 03 Aug 2007) | 18 lines

Merged revisions 78101 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r78101 | russell | 2007-08-03 15:14:06 -0500 (Fri, 03 Aug 2007) | 10 lines

(closes issue ASTERISK-9868)
Reported by: blitzrage
Patches:
     bug0010194 uploaded by vovochka
Tested by: blitzrage

Fix a problem when you call Voicemail() with multiple mailboxes specified and
ODBC_STORAGE is in use.  The audio part of the message was only given to the
first mailbox specified.

........

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