7 May
2015
7 May
'15
5:17 p.m.
Hello,
This pull request adds 2 features.
--Prioritize the trusted records (highest to lowest priority)
--Match from-uri AND request-uri, if either is blank only one is matched.
Please review the pull request and let me know if any other changes are needed.
Thank you.
Emmanuel
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/152
-- Commit Summary --
* added RURI-pattern matching and priority
-- File Changes --
M modules/permissions/hash.c (65)
M modules/permissions/hash.h (8)
M modules/permissions/permissions.c (4)
M modules/permissions/permissions.h (2)
M modules/permissions/trusted.c (103)
M utils/kamctl/db_sqlite/permissions-create.sql (6)
M utils/kamctl/mysql/permissions-create.sql (6)
M utils/kamctl/oracle/permissions-create.sql (6)
M utils/kamctl/postgres/permissions-create.sql (6)
M utils/kamctl/xhttp_pi/permissions-table (2)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/152.patch
https://github.com/kamailio/kamailio/pull/152.diff
---
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/152
9 Jun
9 Jun
4:24 p.m.
New subject: [kamailio] added RURI-pattern matching and priority (#152)
Hello,
Any chance this can be reviewed before the feature request window closes?
Thanks,
Emmanuel
---
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/152#issuecomment-110356090
4:38 p.m.
New subject: [kamailio] added RURI-pattern matching and priority (#152)
Master branch will be frozen again after at least 6 months, plenty of time to get new
features in v4.4.
The patch got big because you changed the order of existing columns, not adding new column
as last one, but inserting in between, so it was not trivial to analyze quickly when I
tried a while ago as I found a bit of spare time.
Most likely I can allocate some time next week, as v4.3 will be out this week.
---
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/152#issuecomment-110359699
5 p.m.
New subject: [kamailio] added RURI-pattern matching and priority (#152)
Thank you for the response.
Yes, I did re-order the fields to make it more comprehensive in the long run.
I've been running this patch on my test machine for several weeks without any issues.
I hope you will consider reviewing and testing soon as we want to put this in production.
Thanks,
Emmanuel
---
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/152#issuecomment-110364617
11 Jun
11 Jun
4:06 p.m.
New subject: [kamailio] added RURI-pattern matching and priority (#152)
Some comments:
* loading ordered on priority is using an SQL extension, making the module to fail when
used with db_text, db_mongodb or other no-sql backends. This has to be reworked to get rid
of the ordering extension specific to sql or made a configuration file option (module
parameter)
* is it a reason to restrict the matching only to r-uri parts of user@domain? May be
useful to match also on transport or other parameters. I would live the matching against
the full value of the r-uri inside the code. In the config one can set the $ru to whatever
it likes to match
* the code around the printing of sort_order is looking weird, as you add sort_order.len
(which is set before to 0) to the buffer -- it is harmless but doesn't look good when
someone is checking the code
---
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/152#issuecomment-111125689
16 Jun
16 Jun
4:31 p.m.
New subject: [kamailio] added RURI-pattern matching and priority (#152)
Add the changes to the db schema -- you have to update the xml file in lib/srdb1/schema/.
---
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/152#issuecomment-112429436
4:32 p.m.
New subject: [kamailio] added RURI-pattern matching and priority (#152)
And extend the documentation of the module with the appropriate description and new
parameters -- changes have to be done to xml files in modules/permissions/doc/
---
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/152#issuecomment-112430014
5:08 p.m.
New subject: [kamailio] added RURI-pattern matching and priority (#152)
Hello,
made changes as you requested.
Please see latest commit.
Thanks,
Emmanuel
---
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/152#issuecomment-112444793
5:42 p.m.
New subject: [kamailio] added RURI-pattern matching and priority (#152)
The commit message is not relevant for the all changes done by the patch, but reflects the
last operation you did as per comments in this discussion. It is not suitable for the log
of source code changes history.
I will apply the patch manually this time if all ok with the review. But for the future be
sure you follow the guidelines for commit messages from:
* http://www.kamailio.org/wiki/devel/git-commit-guidelines#commit_message
Note also that you can have many commits in the same pull request. No need to rework a new
patch always, you can add another patch to complete the changes done with previous
patches.
---
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/152#issuecomment-112453713
5:47 p.m.
New subject: [kamailio] added RURI-pattern matching and priority (#152)
Hello,
You can download the full git diff (it applies clean to master branch) here
http://paste.debian.net/download/231826
Thank you.
Emmanuel
---
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/152#issuecomment-112456577
17 Jun
17 Jun
5:27 p.m.
New subject: [kamailio] added RURI-pattern matching and priority (#152)
Closed #152.
---
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/152#event-333230668
5:27 p.m.
New subject: [kamailio] added RURI-pattern matching and priority (#152)
I applied the patch manually, split in few commits, per components.
The diff for each commit is available also on github, at:
*
https://github.com/eschmidbauer/kamailio/commit/72259b0a9b1f0f9d96e12e4bfa1…
Add .diff to the commit path.
I fixed the upgrade of table version for trusted.
You have to make a new pull request to update the docs regarding the priority, to specify
what kind of value represents higher priority.
---
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/152#issuecomment-112823651
3446
days inactive
3487
days old
11 comments
2 participants
participants (2)
-
Daniel-Constantin Mierla -
Emmanuel Schmidbauer