[asterisk-dev] [Code Review] 3745: func_uri: URIENCODE - Remove warning messages when empty string is passed for the argument
Jonathan Rose
reviewboard at asterisk.org
Fri Jul 11 11:05:52 CDT 2014
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/3745/
-----------------------------------------------------------
Review request for Asterisk Developers and Matt Jordan.
Bugs: ASTERISK-23911
https://issues.asterisk.org/jira/browse/ASTERISK-23911
Repository: Asterisk
Description
-------
WARNING[4943]: func_uri.c:79 uriencode: Syntax: URIENCODE(<data>) - missing argument!
There is nothing especially wrong with trying to encode an empty string, so there is no need to warn for it.
Plus avoiding the warning requires making some dialplans more elaborate. This:
same => n,Stasis(myapp,${URIENCODE(${ARGS}))
becomes this:
same => n,GotoIf($[${LEN(${ARGS})} = 0]skip:)
same => n,Set(ARGS=${URIENCODE(${ARGS})})
same => n(skip),NoOp()
same => n,Stasis(MyApp,${ARGS})
for instance.
Diffs
-----
/branches/1.8/funcs/func_uri.c 418366
Diff: https://reviewboard.asterisk.org/r/3745/diff/
Testing
-------
Checked to make sure the warning doesn't get generated anymore. It doesn't.
Thanks,
Jonathan Rose
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20140711/439e35fc/attachment.html>
More information about the asterisk-dev
mailing list