this was borrowed from the `kazoo` and `nsq` module json transformations i think it makes the most sense in this module as I'd like to use the transformation without loading the `kazoo` or `nsq` modules. It can be very useful for parsing json data from pseudo-variables like $rb. if this can be merged, I will deprecate the transformation in `nsq` and maintain this code.
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/1404
-- Commit Summary --
* json: add transformation
-- File Changes --
M src/modules/json/doc/json_admin.xml (26) M src/modules/json/json_funcs.c (283) M src/modules/json/json_funcs.h (27) M src/modules/json/json_mod.c (40) A src/modules/json/json_trans.c (451) A src/modules/json/json_trans.h (51)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/1404.patch https://github.com/kamailio/kamailio/pull/1404.diff
Is the name of the transformation the same across these modules? If yes, that creates a conflict if all modules are going to be loaded -- I understand you are going to remove from nsq, but there is also in kazoo.
transformation is not the same -- it is entirely changed to not conflict with `kazoo` or `nsq` example transformations: - json module: `$(rb{json.parse,Event-Package}` - nsq module: `$(rb{nsq.json,Event-Package}` - kazoo module: `$(rb{kz.json,Event-Package}`
Merged #1404.
-
Daniel-Constantin Mierla -
Emmanuel Schmidbauer