Module: kamailio
Branch: master
Commit: de529a682afa37482fdc7992e5224c89298c07d3
URL: 
https://github.com/kamailio/kamailio/commit/de529a682afa37482fdc7992e5224c89298c07d3

Author: Marco Capetta <mcape...@sipwise.com>
Committer: Marco Capetta <mcape...@sipwise.com>
Date: 2019-03-28T14:03:54+01:00

dialplan: fixed error log in rule_translate

- Print an error line only if the string fails a match with
all the subst_exp avp values.

---

Modified: src/modules/dialplan/dp_repl.c

---

Diff:  
https://github.com/kamailio/kamailio/commit/de529a682afa37482fdc7992e5224c89298c07d3.diff
Patch: 
https://github.com/kamailio/kamailio/commit/de529a682afa37482fdc7992e5224c89298c07d3.patch

---

diff --git a/src/modules/dialplan/dp_repl.c b/src/modules/dialplan/dp_repl.c
index 2cd568ca5f..06534a9dab 100644
--- a/src/modules/dialplan/dp_repl.c
+++ b/src/modules/dialplan/dp_repl.c
@@ -432,7 +432,7 @@ int rule_translate(sip_msg_t *msg, str *instr, dpl_node_t 
*rule,
                /*search for the pattern from the compiled subst_exp*/
                if (pcre_exec(subst_comp, NULL, instr->s, instr->len,
                                        0, 0, ovector, 3 * (MAX_REPLACE_WITH + 
1)) <= 0) {
-                       LM_ERR("the string %.*s matched "
+                       LM_DBG("the string %.*s matched "
                                        "the match_exp %.*s but not the 
subst_exp %.*s!\n",
                                        instr->len, instr->s,
                                        rule->match_exp.len, rule->match_exp.s,
@@ -721,6 +721,13 @@ int dp_translate_helper(sip_msg_t *msg, str *input, str 
*output, dpl_id_p idp,
                        pkg_free(re_list);
                        re_list = rt;
                } while(re_list);
+               if(rez<0) {
+                       LM_ERR("the string %.*s matched "
+                               "the match_exp %.*s but not the subst_exp 
%.*s!\n",
+                               input->len, input->s,
+                               rulep->match_exp.len, rulep->match_exp.s,
+                               rulep->subst_exp.len, rulep->subst_exp.s);
+               }
        }
        else {
                if(rule_translate(msg, input, rulep, rulep->subst_comp, 
output)!=0){


_______________________________________________
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev

Reply via email to