[asterisk-dev] [Code Review] 3745: func_uri: URIENCODE/URIDECODE - Remove warning messages when empty string is passed for the argument

[Jonathan Rose]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/3745/
-----------------------------------------------------------

(Updated July 14, 2014, 11:48 a.m.)


Review request for Asterisk Developers and Matt Jordan.


Changes
-------

Hit rmudgett's finding


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 (updated)
-----

  /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/20140714/5008be4a/attachment.html>