Re: [sr-dev] [kamailio/kamailio] userblacklist: added missing kemi functions (#2190)

Pull request merged manually (to adapt the commit logs). This changed code in different areas , so I decided to not backport it to stable branches right now. After some testing it could be backported to 5.3 stable branch, though. -- You are receiving this because you are subscribed to this

Re: [sr-dev] [kamailio/kamailio] userblacklist: added missing kemi functions (#2190)

Closed #2190. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/2190#event-2944776375___ Kamailio (SER) - Development Mailing List

Re: [sr-dev] [kamailio/kamailio] userblacklist: added missing kemi functions (#2190)

No action necessary from your side. :-) I will also have a look later today and merge it when I did not notice anything else. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

Re: [sr-dev] [kamailio/kamailio] userblacklist: added missing kemi functions (#2190)

Hi All, how could i proceed to get the pull request merged ? Kind regards. Federico Santulli > Il giorno 8 gen 2020, alle ore 00:34, Federico Santulli > ha scritto: > > Reopened #2190 . > > — > You are receiving this because you are

Re: [sr-dev] [kamailio/kamailio] userblacklist: added missing kemi functions (#2190)

Reopened #2190. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/2190#event-2929406765___ Kamailio (SER) - Development Mailing List

Re: [sr-dev] [kamailio/kamailio] userblacklist: added missing kemi functions (#2190)

Closed #2190. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/kamailio/kamailio/pull/2190#event-2929405030___ Kamailio (SER) - Development Mailing List

Re: [sr-dev] [kamailio/kamailio] userblacklist: added missing kemi functions (#2190)

@fsantulli pushed 1 commit. fbc8598ffc4e3c9706c25ac9d5e608c86a2005be userblacklist: added missing pkg_free before returning -- You are receiving this because you are subscribed to this thread. View it on GitHub:

Re: [sr-dev] [kamailio/kamailio] userblacklist: added missing kemi functions (#2190)

@fsantulli pushed 1 commit. 5f14c488d9ac416864562f888c40e4f005307c3f userblacklist: fixed rpc_child_init behaviour as changed add_source -- You are receiving this because you are subscribed to this thread. View it on GitHub:

Re: [sr-dev] [kamailio/kamailio] userblacklist: added missing kemi functions (#2190)

fsantulli commented on this pull request. > + LM_ERR("no table name\n"); + return -1; + } + /* try to add the table */ + if (add_source(table) != 0) { + LM_ERR("could not add table"); + return -1; + } + + gnode

Re: [sr-dev] [kamailio/kamailio] userblacklist: added missing kemi functions (#2190)

@fsantulli pushed 1 commit. 08a17afe1c18ab0f54287b7fb9d22bedad94983d userblacklist: moved pkg_free to kemi functions instead of native ones. -- You are receiving this because you are subscribed to this thread. View it on GitHub:

Re: [sr-dev] [kamailio/kamailio] userblacklist: added missing kemi functions (#2190)

miconda commented on this pull request. > + LM_ERR("no table name\n"); + return -1; + } + /* try to add the table */ + if (add_source(table) != 0) { + LM_ERR("could not add table"); + return -1; + } + + gnode =

Re: [sr-dev] [kamailio/kamailio] userblacklist: added missing kemi functions (#2190)

fsantulli commented on this pull request. > + LM_ERR("no table name\n"); + return -1; + } + /* try to add the table */ + if (add_source(table) != 0) { + LM_ERR("could not add table"); + return -1; + } + + gnode

Re: [sr-dev] [kamailio/kamailio] userblacklist: added missing kemi functions (#2190)

@fsantulli pushed 1 commit. d5294563c3be9bcc7e21579c1fdf59af2e3a8cc0 Added pkg_free at returning of check_whitelist and check_blacklist -- You are receiving this because you are subscribed to this thread. View it on GitHub:

Re: [sr-dev] [kamailio/kamailio] userblacklist: added missing kemi functions (#2190)

miconda commented on this pull request. > + LM_ERR("no table name\n"); + return -1; + } + /* try to add the table */ + if (add_source(table) != 0) { + LM_ERR("could not add table"); + return -1; + } + + gnode =

Re: [sr-dev] [kamailio/kamailio] userblacklist: added missing kemi functions (#2190)

Thanks for the contribution! The commit message should be formatted according to the contributing guide: * https://github.com/kamailio/kamailio/blob/master/.github/CONTRIBUTING.md#commit-message-rules You actually did the pull request title and comments like the commit message should have

[sr-dev] [kamailio/kamailio] userblacklist: added missing kemi functions (#2190)

- fixed dynamic sources loading - fixed match_mode in check_whitelist function !-- Kamailio Pull Request Template -- !-- IMPORTANT: - for detailed contributing guidelines, read: https://github.com/kamailio/kamailio/blob/master/.github/CONTRIBUTING.md - pull requests must be done to