[Home]

Summary:ASTERISK-10225: Followme attempts to open stream to no existing file
Reporter:Paul Belanger (pabelanger)Labels:
Date Opened:2007-09-03 16:17:53Date Closed:2007-09-04 15:43:14
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Applications/app_followme
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) app_followme.81445.diff
( 1) followme-10634.diff
Description:Using the following syntax:

FollowMe(516@x|n);

Causes the following error to be thrown (See Additional Information)

Looking into the source for app_followme.c, I believe the problem is around line 966:
---
/* The following call looks like we're going to playback the file, but we're actually   */
/* just checking to see if we *can* play it.                                       */

I'll attempt to write a patch in the following days.  I will also update the application to use the /var/spool/asterisk/tmp folder rather than /var/spool/asterisk to start tempory files.


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

   -- Executing [516@ael-extensions:1] FollowMe("SIP/209.217.98.194-086e6000", "516@xxx|n") in new stack
[Sep  3 17:26:53] WARNING[96669]: file.c:563 ast_openstream_full: File /var/spool/asterisk/followme.1188854803.0 does not exist in any format
[Sep  3 17:26:53] WARNING[96669]: file.c:813 ast_streamfile: Unable to open /var/spool/asterisk/followme.1188854803.0 (format (ulaw|gsm)): No such file or directory
Comments:By: James Golovich (jamesgolovich) 2007-09-03 16:31:49

Can you include your followme.conf?  I wasn't able to reproduce this with my sample setup
By: Paul Belanger (pabelanger) 2007-09-03 16:41:02

followme.conf
---
[general]
;
featuredigittimeout=>5000
; The number of ms to wait for a digit input for the callee on whether to take the call or
; not before we consider them "done" entering digits.
;
takecall=>1
; The global default keypress for the callee to take taking the current call. This can be
; a single digit or multiple digits. Default is "1".
;
declinecall=>2
; The global default keypress for the callee to decline taking the current call. This can
; be a single digit or multiple digits. Default is "2".
;
call-from-prompt=>followme/call-from
; The global default for the 'Incoming call from' message.
;
norecording-prompt=>followme/no-recording
; The global default for the 'You have an incoming call' message when the caller elects
; not to leave their name or the option isn't set for them to do so.
;
options-prompt=>followme/options
; The global default for the 'Press 1 to accept this call or press 2 to decline it' message.
;
pls-hold-prompt=>followme/pls-hold-while-try
; The global default for 'Please hold while we try and connect your call' message.
;
status-prompt=>followme/status
; The global default for 'The party you're calling isn't at their desk' message.
;
sorry-prompt=>followme/sorry
; The global default for 'I'm sorry, but we were unable to locate your party' message.
;
;
[232@xxx]
musicclass=>default
context=>ael-numbers
number=>232,25

[516@xxx]
musicclass=>default
context=>ael-numbers
number=>516,25
By: Jason Parker (jparker) 2007-09-04 11:30:21

I believe this is what the code is really trying to do.  Can you test this patch?
By: Paul Belanger (pabelanger) 2007-09-04 12:42:40

I'm having troubles testing on my asterisk-trunk box right now, problems with pbx_ael.c.  Going to quickly apply patch to asterisk-1.4 box.

PB
By: Paul Belanger (pabelanger) 2007-09-04 13:43:01

qwell - Code works great on tags/1.4.11 and branches/1.4.  Like I said I problems with /trunk, but they are my issues.  I don't see any reason why this would not work.

I also modified the source to use /spool/asterisk/tmp now.

PB
By: Digium Subversion (svnbot) 2007-09-04 15:40:41

Repository: asterisk
Revision: 81456

------------------------------------------------------------------------
r81456 | qwell | 2007-09-04 15:40:41 -0500 (Tue, 04 Sep 2007) | 13 lines

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

(closes issue ASTERISK-10225)
........
r81455 | qwell | 2007-09-04 15:54:51 -0500 (Tue, 04 Sep 2007) | 4 lines

Rather than attempt to play a file, we can just check whether it exists.

Issue 10634, patch by me, testing by pabelanger, sanity checked by bweschke

........

------------------------------------------------------------------------
By: Jason Parker (jparker) 2007-09-04 15:43:14

Note, I opted to not add the change in directory.