[Home]

Summary:ASTERISK-14021: don't warn, when pipe is used in app_system
Reporter:pj (pj)Labels:
Date Opened:2009-04-27 13:05:22Date Closed:2009-04-27 14:36:25
Priority:TrivialRegression?No
Status:Closed/CompleteComponents:Core/PBX
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:
Description:Using pipe is completelly legal when executing system commands,
eg. this dialplan command shouldn't generate any warnings

       System(echo -e abc | mail -s test foo@bar);





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

[Apr 27 19:56:08] WARNING[19826]: pbx.c:1343 pbx_exec: The application delimiter is now the comma, not the pipe.  Did you forget to convert your dialplan?  (System(echo -e abc | mail -s test foo@bar))
Comments:By: pj (pj) 2009-04-27 13:07:43

maybe better will be to put this issue into core/pbx category

By: Leif Madsen (lmadsen) 2009-04-27 13:08:57

Issue has been assigned, but Tilghman is sick today, and this is low priority. Thanks!

By: Digium Subversion (svnbot) 2009-04-27 14:34:50

Repository: asterisk
Revision: 190726

U   trunk/main/pbx.c

------------------------------------------------------------------------
r190726 | tilghman | 2009-04-27 14:34:49 -0500 (Mon, 27 Apr 2009) | 4 lines

Don't warn on pipe in the System call.
(closes issue ASTERISK-14021)
Reported by: pj

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

http://svn.digium.com/view/asterisk?view=rev&revision=190726

By: Digium Subversion (svnbot) 2009-04-27 14:36:05

Repository: asterisk
Revision: 190727

_U  branches/1.6.0/
U   branches/1.6.0/main/pbx.c

------------------------------------------------------------------------
r190727 | tilghman | 2009-04-27 14:36:05 -0500 (Mon, 27 Apr 2009) | 11 lines

Merged revisions 190726 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
 r190726 | tilghman | 2009-04-27 14:34:48 -0500 (Mon, 27 Apr 2009) | 4 lines
 
 Don't warn on pipe in the System call.
 (closes issue ASTERISK-14021)
  Reported by: pj
........

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

http://svn.digium.com/view/asterisk?view=rev&revision=190727

By: Digium Subversion (svnbot) 2009-04-27 14:36:16

Repository: asterisk
Revision: 190728

_U  branches/1.6.1/
U   branches/1.6.1/main/pbx.c

------------------------------------------------------------------------
r190728 | tilghman | 2009-04-27 14:36:15 -0500 (Mon, 27 Apr 2009) | 11 lines

Merged revisions 190726 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
 r190726 | tilghman | 2009-04-27 14:34:48 -0500 (Mon, 27 Apr 2009) | 4 lines
 
 Don't warn on pipe in the System call.
 (closes issue ASTERISK-14021)
  Reported by: pj
........

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

http://svn.digium.com/view/asterisk?view=rev&revision=190728

By: Digium Subversion (svnbot) 2009-04-27 14:36:24

Repository: asterisk
Revision: 190729

_U  branches/1.6.2/
U   branches/1.6.2/main/pbx.c

------------------------------------------------------------------------
r190729 | tilghman | 2009-04-27 14:36:24 -0500 (Mon, 27 Apr 2009) | 11 lines

Merged revisions 190726 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
 r190726 | tilghman | 2009-04-27 14:34:48 -0500 (Mon, 27 Apr 2009) | 4 lines
 
 Don't warn on pipe in the System call.
 (closes issue ASTERISK-14021)
  Reported by: pj
........

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

http://svn.digium.com/view/asterisk?view=rev&revision=190729

By: tsm (tsm) 2012-03-21 10:16:14.274-0500

Issue seems to still persist in 1.8.7.0.

These lines below cause the issue.
exten => s,n,ExecIf($[ "${DB_EXISTS(AMPUSER/${EXTTOCALL}/email)}"="1" & [${DIALSTATUS} == CANCEL | ${DIALSTATUS} == BUSY | ${DIALSTATUS} == NOANSWER] & "${OUTBOUND_GROUP}" = "" & "${DIAL_TRUNK}" = "" & "${RingGroupMethod}" = "none"]?System(echo "" | mail -s "MissedCall (${EXTTOCALL}) - ${CALLERID(name)} <${CALLERID(num)}>" ${DB(AMPUSER/${EXTTOCALL}/email)}))
exten => s,n,ExecIf($["${DB_EXISTS(AMPGROUP/${NODEST}/email)}"="1" & ${DIALSTATUS} == CANCEL & "${OUTBOUND_GROUP}" = "" & "${DIAL_TRUNK}" = "" & "${RingGroupMethod}" != "none"]?System(echo "" | mail -s "MissedCallGroup (${NODEST}) - ${CALLERID(name)} <${CALLERID(num)}>" ${DB(AMPGROUP/${NODEST}/email)}))

By: uros gaber (urosh) 2012-11-20 05:26:51.134-0600

This is also present in 10.8.0.