Summary: | ASTERISK-02714: app_record is broken saying that filename argument is not present | ||
Reporter: | cmslaght (cmslaght) | Labels: | |
Date Opened: | 2004-10-31 00:23:35 | Date Closed: | 2004-11-02 08:01:53.000-0600 |
Priority: | Minor | Regression? | No |
Status: | Closed/Complete | Components: | Core/General |
Versions: | Frequency of Occurrence | ||
Related Issues: | |||
Environment: | Attachments: | ( 0) record.diff.txt | |
Description: | In at line 89 a filename is required. This app was working up until a week or so ago. Error causes the caller to get hung up on. | ||
Comments: | By: Clod Patry (junky) 2004-10-31 01:55:10.000-0600 It's normal that line 89 requires a filename, you need to pass a filename to the app_record to record your file. If you don't specify a filename, it's normal that app_record won't record. What do you expect exactly? Record to /dev/null by default? :P Please provide your dialplan. And next time you report a bug, please provide your dial plan if you've a problem with an app. Thanks. By: cmslaght (cmslaght) 2004-10-31 07:37:33.000-0600 Dialplan: ... exten => 1,2,Playback(pleaserecordyourname) exten => 1,3,Record(testchris.gsm|5|15) exten => 1,4,Dial(Zap/g2/${CHRIS-CELL},20) ... Console Log: -- Playing 'pleaserecordyourname' (language 'en') -- Executing Record("Zap/25-1", "testchris.gsm|5|15") in new stack Oct 31 08:48:18 WARNING[327700]: app_record.c:90 record_exec: Record requires an argument (filename) When it executes record, it immediately hangs up on the caller. edited on: 10-31-04 07:39 By: brc007 (brc007) 2004-10-31 12:56:04.000-0600 I have duplicated this bug on CVS HEAD. By: Clod Patry (junky) 2004-10-31 18:15:34.000-0600 i've some problem with my monitor, i've to buy another one. I'll spend more time when i'll get my new monitor. But maybe it's a problem with with the format, cause before, it was in this format: Record(filename:format[|silence][|maxduration][|option]) the difference i see: its the :format vs .format, and the silence was optional and it seems to be mandatory now. Probably just some errors when writing it. Thanks bkw for the patch, i'll take more time when i'll my monitor. By: Brian West (bkw918) 2004-10-31 18:18:14.000-0600 No YOU BROKE it. You're patch to allow the stop key to be a config option BROKE IT. haha By: Mark Spencer (markster) 2004-10-31 18:27:22.000-0600 Ouch. Sorry about that. |