- add sht_is_null route function - add sht_inc route function - add sht_dec route function
#### Pre-Submission Checklist - [x] Commit message has the format required by CONTRIBUTING guide - [x] Commits are split per component (core, individual modules, libs, utils, ...) - [x] Each component has a single commit (if not, squash them into one commit) - [x] No commits to README files for modules (changes must be done to docbook files in `doc/` subfolder, the README file is autogenerated)
#### Type Of Change - [ ] Small bug fix (non-breaking change which fixes an issue) - [x] New feature (non-breaking change which adds new functionality) - [ ] Breaking change (fix or feature that would change existing functionality)
#### Checklist: - [ ] PR should be backported to stable branches - [x] Tested changes locally - [ ] Related to issue #XXXX (replace XXXX with an open issue number)
#### Description Implementing the identical functions from KEMI in the route functions.
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/4499
-- Commit Summary --
* htable: feature parity with kemi funcs
-- File Changes --
M src/modules/htable/doc/htable.xml (2) M src/modules/htable/doc/htable_admin.xml (105) M src/modules/htable/htable.c (95)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/4499.patch https://github.com/kamailio/kamailio/pull/4499.diff
@devopsec pushed 1 commit.
9839f945e34e652392d4100252afab6d0138f9a6 htable: feature parity with kemi funcs
devopsec left a comment (kamailio/kamailio#4499)
of course i forgot clang-format again... re-submitted with formatting
miconda left a comment (kamailio/kamailio#4499)
The Kemi functions were added to correspond to operations with variables.
``` $shv(x=>y) == $null $shvinc(x=>y) $shvdec(x=>y) ```
I am not really against adding also functions, but returning 0 to kamailio.cfg, as I could see quickly in the docs from commit, stops the execution of config.
@devopsec pushed 1 commit.
408dfe10be3fce7a67198eebebb1f7a050e148ef auth: use AUTH_FLAG_NOINVNC flag
@devopsec pushed 1 commit.
4f3901424a0e202babd7eff54f166359a38e1880 auth: use AUTH_FLAG_NOINVNC flag
devopsec left a comment (kamailio/kamailio#4499)
@miconda I sepcified in the docs the return was "< 0" but I understand that was not as clear at a glance. I updated that portion of the doc to be more explicit and l also moved the return value for parameter parsing errors to 3 instead (reviewing this I saw it conflicted with the meaning of -1 return value).
miconda left a comment (kamailio/kamailio#4499)
Merging it, thanks!
@miconda pushed 1 commit.
19222113948851fc206bcb445c756d4ffcea5c64 Merge branch 'master' into htable_cmd_updates
miconda left a comment (kamailio/kamailio#4499)
It was merged manually to fix some conflicts.
Closed #4499.