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

--===============0743432567==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable

I think it could be related to:
https://github.com/kamailio/kamailio/issues/2479

Commit:
https://github.com/kamailio/kamailio/commit/2e7710e292a649c6c57e46d5994d3b2cf=
3815865

Thanks!
Joel.

On Thu, Oct 21, 2021 at 4:22 PM Joel Serrano <joel(a)textplus.com> wrote:

> 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.
>
>
>



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

PGRpdiBkaXI9Imx0ciI+SSB0aGluayBpdCBjb3VsZCBiZSByZWxhdGVkIHRvOsKgPGEgaHJlZj0i
aHR0cHM6Ly9naXRodWIuY29tL2thbWFpbGlvL2thbWFpbGlvL2lzc3Vlcy8yNDc5Ij5odHRwczov
L2dpdGh1Yi5jb20va2FtYWlsaW8va2FtYWlsaW8vaXNzdWVzLzI0Nzk8L2E+wqA8ZGl2Pjxicj48
L2Rpdj48ZGl2PkNvbW1pdDrCoDxhIGhyZWY9Imh0dHBzOi8vZ2l0aHViLmNvbS9rYW1haWxpby9r
YW1haWxpby9jb21taXQvMmU3NzEwZTI5MmE2NDljNmM1N2U0NmQ1OTk0ZDNiMmNmMzgxNTg2NSI+
aHR0cHM6Ly9naXRodWIuY29tL2thbWFpbGlvL2thbWFpbGlvL2NvbW1pdC8yZTc3MTBlMjkyYTY0
OWM2YzU3ZTQ2ZDU5OTRkM2IyY2YzODE1ODY1PC9hPjwvZGl2PjxkaXY+PGJyPjwvZGl2PjxkaXY+
VGhhbmtzITwvZGl2PjxkaXY+Sm9lbC48L2Rpdj48ZGl2PjwvZGl2PjwvZGl2Pjxicj48ZGl2IGNs
YXNzPSJnbWFpbF9xdW90ZSI+PGRpdiBkaXI9Imx0ciIgY2xhc3M9ImdtYWlsX2F0dHIiPk9uIFRo
dSwgT2N0IDIxLCAyMDIxIGF0IDQ6MjIgUE0gSm9lbCBTZXJyYW5vICZsdDs8YSBocmVmPSJtYWls
dG86am9lbEB0ZXh0cGx1cy5jb20iPmpvZWxAdGV4dHBsdXMuY29tPC9hPiZndDsgd3JvdGU6PGJy
PjwvZGl2PjxibG9ja3F1b3RlIGNsYXNzPSJnbWFpbF9xdW90ZSIgc3R5bGU9Im1hcmdpbjowcHgg
MHB4IDBweCAwLjhleDtib3JkZXItbGVmdDoxcHggc29saWQgcmdiKDIwNCwyMDQsMjA0KTtwYWRk
aW5nLWxlZnQ6MWV4Ij48ZGl2IGRpcj0ibHRyIj5IZWxsbyzCoDxkaXY+PGJyPjwvZGl2PjxkaXY+
SSYjMzk7bSBpbiB0aGUgcHJvY2VzcyBvZiB1cGRhdGluZyBzb21lIHNlcnZlcnMgdGhhdCBhcmUg
cnVubmluZyBLYW0gdjUuMS40IHRvIHY1LjUuMi48L2Rpdj48ZGl2Pjxicj48L2Rpdj48ZGl2Pjxk
aXY+QWZ0ZXIgdGhlIHVwZ3JhZGUgSSYjMzk7dmUgbm90aWNlZCB0aGF0IEkmIzM5O3ZlIGxvc3Qg
c29tZSBjdXN0b20gbWV0cmljcyB0aGF0IHdlIGhhdmUgZGVmaW5lZCB2aWEgY29uZmlnLXNjcmlw
dC7CoDwvZGl2PjxkaXY+PGJyPjwvZGl2PjxkaXY+U25pcHBldCBvZiB0aGUgcmVsZXZhbnQgY29u
ZmlnOjwvZGl2PjxkaXY+PGJyPjwvZGl2PjwvZGl2PjxkaXY+PGZvbnQgZmFjZT0ibW9ub3NwYWNl
Ij5ldmVudF9yb3V0ZVtkaWFsb2c6c3RhcnRdIHs8YnI+wqAgwqAgIyBHYXRoZXIgbWV0cmljcyBy
ZWdhcmRpbmcgc291cmNlIElQPGJyPsKgIMKgIHhsb2coJnF1b3Q7TF9XQVJOJnF1b3Q7LCAmcXVv
dDtbRExHXSAtIENoZWNraW5nIGNvdW50cnkgZm9yICgkc2kgfCAkVF9yZXEoJHNpKSkgLSBTVEFS
VFxuJnF1b3Q7KTs8YnI+wqAgwqAgaWYgKGdlb2lwMl9tYXRjaCgmcXVvdDskVF9yZXEoJHNpKSZx
dW90OywgJnF1b3Q7c3JjJnF1b3Q7KSkgezxicj7CoCDCoCDCoCDCoCAjIENvdW50cnkgZm91bmQ8
YnI+wqAgwqAgwqAgwqAgLi4uPGJyPsKgIMKgIH08YnI+wqAgwqAgeGxvZygmcXVvdDtMX1dBUk4m
cXVvdDssICZxdW90O1tETEddIC0gQ2hlY2tpbmcgY291bnRyeSBmb3IgKCRzaSB8ICRUX3JlcSgk
c2kpKSAtIEVORFxuJnF1b3Q7KTs8YnI+fTwvZm9udD48YnI+PC9kaXY+PGRpdj48YnI+PC9kaXY+
PGRpdj5OT1RFOiB4bG9nIGxpbmVzIGFkZGVkIGZvciBkZWJ1Z2dpbmcsIHRoZXkgbm9ybWFsbHkg
YXJlIG5vdCB0aGVyZS48L2Rpdj48ZGl2Pjxicj48L2Rpdj48ZGl2Pjxicj48L2Rpdj48ZGl2Pklu
IHZlcnNpb24gNS4xLjQgaW4gdGhlIGxvZ3MgSSBzZWU6wqA8L2Rpdj48ZGl2Pjxicj48L2Rpdj48
ZGl2Pjxmb250IGZhY2U9Im1vbm9zcGFjZSI+T2N0IDIxIDE1OjU1OjQ3IDEzY24zIHNiY1syMjYw
Nl06IFdBUk5JTkc6ICZsdDtzY3JpcHQmZ3Q7OiBbRExHXSAtIENoZWNraW5nIGNvdW50cnkgZm9y
IChBLkIuQy5EIHwgMTg2LjE4NS4xODYuNykgLSBTVEFSVDxicj5PY3QgMjEgMTU6NTU6NDcgMTNj
bjMgc2JjWzIyNjA2XTogV0FSTklORzogJmx0O3NjcmlwdCZndDs6IFtETEddIC0gQ2hlY2tpbmcg
Y291bnRyeSBmb3IgKEEuQi5DLkQgfCAxODYuMTg1LjE4Ni43KSAtIEVORDxicj48L2ZvbnQ+PC9k
aXY+PGRpdj48YnI+PC9kaXY+PGRpdj5UaGUgc2FtZSBjb25maWcsIG9uIHY1LjUuMiBpcyByZXR1
cm5pbmc6PC9kaXY+PGRpdj48YnI+PC9kaXY+PGRpdj48Zm9udCBmYWNlPSJtb25vc3BhY2UiPk9j
dCAyMSAxNTo1NzowOCBzYmMwNTA0IHNiY1syMjYwNl06IFdBUk5JTkc6ICZsdDtzY3JpcHQmZ3Q7
OiBbRExHXSAtIENoZWNraW5nIGNvdW50cnkgZm9yIChBLkIuQy5EwqB8ICZsdDtudWxsJmd0Oykg
LSBTVEFSVDxicj5PY3QgMjEgMTU6NTc6MDggc2JjMDUwNMKgc2JjWzIyNjA2XTogV0FSTklORzrC
oCZsdDtzY3JpcHQmZ3Q7OiBbRExHXSAtIENoZWNraW5nIGNvdW50cnkgZm9yIChBLkIuQy5EwqB8
ICZsdDtudWxsJmd0OykgLSBFTkQ8YnI+PC9mb250PjwvZGl2PjxkaXY+PGJyPjwvZGl2PjxkaXY+
QWZ0ZXIgZW5hYmxpbmcgZGVidWcgbG9ncyBJIHNlZTo8L2Rpdj48ZGl2Pjxicj48L2Rpdj48ZGl2
Pjxmb250IGZhY2U9Im1vbm9zcGFjZSI+V0FSTklORzogJmx0O3NjcmlwdCZndDs6IFtETEddIC0g
Q2hlY2tpbmcgY291bnRyeSBmb3IgKDE5OC4xLjU1LjEyNCB8ICZsdDtudWxsJmd0OykgLSBTVEFS
VDxicj48Yj48Zm9udCBjb2xvcj0iI2ZmMDAwMCI+REVCVUc6IHRteCBbdF92YXIuYzozMDldOiBw
dl9nZXRfdF92YXJfcmVxKCk6IHVzZWQgaW4gdW5zdXBwb3J0ZWQgcm91dGUgYmxvY2sgLSB0eXBl
IDY0PGJyPjwvZm9udD48L2I+REVCVUc6IGdlb2lwMiBbZ2VvaXAyX3B2LmM6NDg4XTogZ2VvaXAy
X3VwZGF0ZV9wdigpOiBhdHRlbXB0IHRvIG1hdGNoOiAmbHQ7bnVsbCZndDs8YnI+REVCVUc6IGdl
b2lwMiBbZ2VvaXAyX3B2LmM6NDkxXTogZ2VvaXAyX3VwZGF0ZV9wdigpOiBubyBtYXRjaCBmb3I6
ICZsdDtudWxsJmd0OzwvZm9udD48YnI+PC9kaXY+PGRpdj48YnI+PC9kaXY+PGRpdj48YnI+PC9k
aXY+PGRpdj5EaWQgc29tZXRoaW5nIGNoYW5nZSBpbiB0aGUgdG14IG1vZHVsZSB0aGF0IGNvdWxk
IGFmZmVjdCAkVF9yZXEocHYpIGJlaW5nIGFjY2Vzc2libGUgaW4gdGhlIGRpYWxvZzpzdGFydCBl
dmVudF9yb3V0ZT88L2Rpdj48ZGl2Pjxicj48L2Rpdj48ZGl2PkFueSBhbHRlcm5hdGl2ZXMgdG8g
Z2V0IHRoZSBzb3VyY2UgSVAgb2YgdGhlIHJlcXVlc3QgKG5vdCB0aGUgcmVwbHkpIGluIHRoZSBk
aWFsb2c6c3RhcnQgZXZlbnRfcm91dGU/PC9kaXY+PGRpdj48YnI+PC9kaXY+PGRpdj5MZXQgbWUg
a25vdyBpZiB5b3UgcHJlZmVyIHRoYXQgSSBvcGVuIGEgdGlja2V0IGZvciB0aGlzIGluIEdILjwv
ZGl2PjxkaXY+PGJyPjwvZGl2PjxkaXY+VGhhbmtzLMKgPC9kaXY+PGRpdj5Kb2VsLjwvZGl2Pjxk
aXY+PGJyPjwvZGl2PjxkaXY+PGJyPjwvZGl2PjwvZGl2Pgo8L2Jsb2NrcXVvdGU+PC9kaXY+Cg==

--===============0743432567==--