fixes the use case when spaces should not be excluded 'BS ParkingTestCase 1' ``` xlog("L_INFO", "$ci|log exploding test\n"); $var(test) = "Notify=false;Unregister=false;Presence-ID=1001@kamailio.io;Account-Name=BS ParkingTestCase 1"; xavp_params_explode($var(test), "mytest"); xlog("L_INFO", "$ci|log exploded test\n"); ```
@miconda i believe this should be included in 5.1 and backported to 5.0
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/1303
-- Commit Summary --
* core : honor parse_param separator
-- File Changes --
M src/core/parser/parse_param.c (2)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/1303.patch https://github.com/kamailio/kamailio/pull/1303.diff
@lazedo - I think this breaks the rules in the sip grammar. The parameter value must be enclosed in quotes (single or double) if you have spaces inside it. Also, comma is separator for headers with multiple bodies, so again it cannot appear in parameter value which is not quoted.
Is any limitation you don't enclose the params values within quotes?
@miconda thanks for pointing that out. checking again, maybe jumped in here too fast.
@miconda enclosing the params with quotes fixes it. sorry for jumping too fast, but when i saw the `separator` and then the exceptions..., didn't think it could break sip grammar. thanks for the explanation.
Closed #1303.
-
Daniel-Constantin Mierla -
lazedo