From joel@textplus.com Fri Oct 22 01:23:28 2021
From: Joel Serrano <joel@textplus.com>
To: sr-users@lists.kamailio.org
Subject: [SR-Users] T_req(pv) not accessible in dialog event route in v5.5.2
Date: Thu, 21 Oct 2021 16:22:54 -0700
Message-ID:
 <CAMtXxQmOPk-kjoy8FnWDLHZ3ZphwgrptcJLoQ2WGMjBS0gSMag@mail.gmail.com>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============1340900117=="

--===============1340900117==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit

Hello,

I'm in the process of updating some servers that are running Kam v5.1.4 to
v5.5.2.

After the upgrade I've noticed that I've lost some custom metrics that we
have defined via config-script.

Snippet of the relevant config:

event_route[dialog:start] {
    # Gather metrics regarding source IP
    xlog("L_WARN", "[DLG] - Checking country for ($si | $T_req($si)) -
START\n");
    if (geoip2_match("$T_req($si)", "src")) {
        # Country found
        ...
    }
    xlog("L_WARN", "[DLG] - Checking country for ($si | $T_req($si)) -
END\n");
}

NOTE: xlog lines added for debugging, they normally are not there.


In version 5.1.4 in the logs I see:

Oct 21 15:55:47 13cn3 sbc[22606]: WARNING: <script>: [DLG] - Checking
country for (A.B.C.D | 186.185.186.7) - START
Oct 21 15:55:47 13cn3 sbc[22606]: WARNING: <script>: [DLG] - Checking
country for (A.B.C.D | 186.185.186.7) - END

The same config, on v5.5.2 is returning:

Oct 21 15:57:08 sbc0504 sbc[22606]: WARNING: <script>: [DLG] - Checking
country for (A.B.C.D | <null>) - START
Oct 21 15:57:08 sbc0504 sbc[22606]: WARNING: <script>: [DLG] - Checking
country for (A.B.C.D | <null>) - END

After enabling debug logs I see:

WARNING: <script>: [DLG] - Checking country for (198.1.55.124 | <null>) -
START

*DEBUG: tmx [t_var.c:309]: pv_get_t_var_req(): used in unsupported route
block - type 64*DEBUG: geoip2 [geoip2_pv.c:488]: geoip2_update_pv():
attempt to match: <null>
DEBUG: geoip2 [geoip2_pv.c:491]: geoip2_update_pv(): no match for: <null>


Did something change in the tmx module that could affect $T_req(pv) being
accessible in the dialog:start event_route?

Any alternatives to get the source IP of the request (not the reply) in the
dialog:start event_route?

Let me know if you prefer that I open a ticket for this in GH.

Thanks,
Joel.



--===============1340900117==
Content-Type: text/html
Content-Transfer-Encoding: base64
Content-Disposition: attachment; filename="attachment.html"
MIME-Version: 1.0

PGRpdiBkaXI9Imx0ciI+SGVsbG8swqA8ZGl2Pjxicj48L2Rpdj48ZGl2PkkmIzM5O20gaW4gdGhl
IHByb2Nlc3Mgb2YgdXBkYXRpbmcgc29tZSBzZXJ2ZXJzIHRoYXQgYXJlIHJ1bm5pbmcgS2FtIHY1
LjEuNCB0byB2NS41LjIuPC9kaXY+PGRpdj48YnI+PC9kaXY+PGRpdj48ZGl2PkFmdGVyIHRoZSB1
cGdyYWRlIEkmIzM5O3ZlIG5vdGljZWQgdGhhdCBJJiMzOTt2ZSBsb3N0IHNvbWUgY3VzdG9tIG1l
dHJpY3MgdGhhdCB3ZSBoYXZlIGRlZmluZWQgdmlhIGNvbmZpZy1zY3JpcHQuwqA8L2Rpdj48ZGl2
Pjxicj48L2Rpdj48ZGl2PlNuaXBwZXQgb2YgdGhlIHJlbGV2YW50IGNvbmZpZzo8L2Rpdj48ZGl2
Pjxicj48L2Rpdj48L2Rpdj48ZGl2Pjxmb250IGZhY2U9Im1vbm9zcGFjZSI+ZXZlbnRfcm91dGVb
ZGlhbG9nOnN0YXJ0XSB7PGJyPsKgIMKgICMgR2F0aGVyIG1ldHJpY3MgcmVnYXJkaW5nIHNvdXJj
ZSBJUDxicj7CoCDCoCB4bG9nKCZxdW90O0xfV0FSTiZxdW90OywgJnF1b3Q7W0RMR10gLSBDaGVj
a2luZyBjb3VudHJ5IGZvciAoJHNpIHwgJFRfcmVxKCRzaSkpIC0gU1RBUlRcbiZxdW90Oyk7PGJy
PsKgIMKgIGlmIChnZW9pcDJfbWF0Y2goJnF1b3Q7JFRfcmVxKCRzaSkmcXVvdDssICZxdW90O3Ny
YyZxdW90OykpIHs8YnI+wqAgwqAgwqAgwqAgIyBDb3VudHJ5IGZvdW5kPGJyPsKgIMKgIMKgIMKg
IC4uLjxicj7CoCDCoCB9PGJyPsKgIMKgIHhsb2coJnF1b3Q7TF9XQVJOJnF1b3Q7LCAmcXVvdDtb
RExHXSAtIENoZWNraW5nIGNvdW50cnkgZm9yICgkc2kgfCAkVF9yZXEoJHNpKSkgLSBFTkRcbiZx
dW90Oyk7PGJyPn08L2ZvbnQ+PGJyPjwvZGl2PjxkaXY+PGJyPjwvZGl2PjxkaXY+Tk9URTogeGxv
ZyBsaW5lcyBhZGRlZCBmb3IgZGVidWdnaW5nLCB0aGV5IG5vcm1hbGx5IGFyZSBub3QgdGhlcmUu
PC9kaXY+PGRpdj48YnI+PC9kaXY+PGRpdj48YnI+PC9kaXY+PGRpdj5JbiB2ZXJzaW9uIDUuMS40
IGluIHRoZSBsb2dzIEkgc2VlOsKgPC9kaXY+PGRpdj48YnI+PC9kaXY+PGRpdj48Zm9udCBmYWNl
PSJtb25vc3BhY2UiPk9jdCAyMSAxNTo1NTo0NyAxM2NuMyBzYmNbMjI2MDZdOiBXQVJOSU5HOiAm
bHQ7c2NyaXB0Jmd0OzogW0RMR10gLSBDaGVja2luZyBjb3VudHJ5IGZvciAoQS5CLkMuRCB8IDE4
Ni4xODUuMTg2LjcpIC0gU1RBUlQ8YnI+T2N0IDIxIDE1OjU1OjQ3IDEzY24zIHNiY1syMjYwNl06
IFdBUk5JTkc6ICZsdDtzY3JpcHQmZ3Q7OiBbRExHXSAtIENoZWNraW5nIGNvdW50cnkgZm9yIChB
LkIuQy5EIHwgMTg2LjE4NS4xODYuNykgLSBFTkQ8YnI+PC9mb250PjwvZGl2PjxkaXY+PGJyPjwv
ZGl2PjxkaXY+VGhlIHNhbWUgY29uZmlnLCBvbiB2NS41LjIgaXMgcmV0dXJuaW5nOjwvZGl2Pjxk
aXY+PGJyPjwvZGl2PjxkaXY+PGZvbnQgZmFjZT0ibW9ub3NwYWNlIj5PY3QgMjEgMTU6NTc6MDgg
c2JjMDUwNCBzYmNbMjI2MDZdOiBXQVJOSU5HOiAmbHQ7c2NyaXB0Jmd0OzogW0RMR10gLSBDaGVj
a2luZyBjb3VudHJ5IGZvciAoQS5CLkMuRMKgfCAmbHQ7bnVsbCZndDspIC0gU1RBUlQ8YnI+T2N0
IDIxIDE1OjU3OjA4IHNiYzA1MDTCoHNiY1syMjYwNl06IFdBUk5JTkc6wqAmbHQ7c2NyaXB0Jmd0
OzogW0RMR10gLSBDaGVja2luZyBjb3VudHJ5IGZvciAoQS5CLkMuRMKgfCAmbHQ7bnVsbCZndDsp
IC0gRU5EPGJyPjwvZm9udD48L2Rpdj48ZGl2Pjxicj48L2Rpdj48ZGl2PkFmdGVyIGVuYWJsaW5n
IGRlYnVnIGxvZ3MgSSBzZWU6PC9kaXY+PGRpdj48YnI+PC9kaXY+PGRpdj48Zm9udCBmYWNlPSJt
b25vc3BhY2UiPldBUk5JTkc6ICZsdDtzY3JpcHQmZ3Q7OiBbRExHXSAtIENoZWNraW5nIGNvdW50
cnkgZm9yICgxOTguMS41NS4xMjQgfCAmbHQ7bnVsbCZndDspIC0gU1RBUlQ8YnI+PGI+PGZvbnQg
Y29sb3I9IiNmZjAwMDAiPkRFQlVHOiB0bXggW3RfdmFyLmM6MzA5XTogcHZfZ2V0X3RfdmFyX3Jl
cSgpOiB1c2VkIGluIHVuc3VwcG9ydGVkIHJvdXRlIGJsb2NrIC0gdHlwZSA2NDxicj48L2ZvbnQ+
PC9iPkRFQlVHOiBnZW9pcDIgW2dlb2lwMl9wdi5jOjQ4OF06IGdlb2lwMl91cGRhdGVfcHYoKTog
YXR0ZW1wdCB0byBtYXRjaDogJmx0O251bGwmZ3Q7PGJyPkRFQlVHOiBnZW9pcDIgW2dlb2lwMl9w
di5jOjQ5MV06IGdlb2lwMl91cGRhdGVfcHYoKTogbm8gbWF0Y2ggZm9yOiAmbHQ7bnVsbCZndDs8
L2ZvbnQ+PGJyPjwvZGl2PjxkaXY+PGJyPjwvZGl2PjxkaXY+PGJyPjwvZGl2PjxkaXY+RGlkIHNv
bWV0aGluZyBjaGFuZ2UgaW4gdGhlIHRteCBtb2R1bGUgdGhhdCBjb3VsZCBhZmZlY3QgJFRfcmVx
KHB2KSBiZWluZyBhY2Nlc3NpYmxlIGluIHRoZSBkaWFsb2c6c3RhcnQgZXZlbnRfcm91dGU/PC9k
aXY+PGRpdj48YnI+PC9kaXY+PGRpdj5BbnkgYWx0ZXJuYXRpdmVzIHRvIGdldCB0aGUgc291cmNl
IElQIG9mIHRoZSByZXF1ZXN0IChub3QgdGhlIHJlcGx5KSBpbiB0aGUgZGlhbG9nOnN0YXJ0IGV2
ZW50X3JvdXRlPzwvZGl2PjxkaXY+PGJyPjwvZGl2PjxkaXY+TGV0IG1lIGtub3cgaWYgeW91IHBy
ZWZlciB0aGF0IEkgb3BlbiBhIHRpY2tldCBmb3IgdGhpcyBpbiBHSC48L2Rpdj48ZGl2Pjxicj48
L2Rpdj48ZGl2PlRoYW5rcyzCoDwvZGl2PjxkaXY+Sm9lbC48L2Rpdj48ZGl2Pjxicj48L2Rpdj48
ZGl2Pjxicj48L2Rpdj48L2Rpdj4K

--===============1340900117==--