[SR-Users] AVPs missing in onreply_route

[Sebastian Damm]

Hi Daniel,

thanks for looking into it again.

On Mon, Aug 1, 2011 at 2:51 PM, Daniel-Constantin Mierla
<miconda at gmail.com> wrote:
> thanks for further investigation. Indeed, the avps were not backed
up/restored when sending self-generated request.
>
> I just committed a patch to tm module in master branch. If you can give it
a try and report if it works ok, then I will backport to 3.1.

OK, I took a default config, added this uac part in RELAY route:

      xlog("BEFORE UAC: avp(s:testavp) = $avp(s:testavp)\n");
       avp_print();

       $uac_req(method)="SPAM";
       $uac_req(ruri)="sip:store at 127.0.0.1:5160";
       $uac_req(furi)="sip:kamailio at sipgate.net";
       $uac_req(hdrs)="Content-Type: text/csv\r\n";
       $uac_req(body)="foo bar baz";
       uac_req_send();

       xlog("AFTER UAC: avp(s:testavp) = $avp(s:testavp)\n");
       avp_print();

And with an unpatched 3.1.4 looks like this:

Aug  2 09:09:10 flake /usr/sbin/kamailio[20378]: ERROR: <script>: BEFORE
UAC: avp(s:testavp) = ThisIsJustATest
Aug  2 09:09:10 flake /usr/sbin/kamailio[20378]: INFO: avpops
[avpops_impl.c:1328]: p=0x7f7c0c0e95e8, flags=0x0113
Aug  2 09:09:10 flake /usr/sbin/kamailio[20378]: INFO: avpops
[avpops_impl.c:1332]: #011#011#011name=<testavp>
Aug  2 09:09:10 flake /usr/sbin/kamailio[20378]: INFO: avpops
[avpops_impl.c:1340]: #011#011#011val_str=<ThisIsJustATest / 15>
Aug  2 09:09:10 flake /usr/sbin/kamailio[20378]: ERROR: <script>: AFTER UAC:
avp(s:testavp) = <null>
Aug  2 09:09:10 flake /usr/sbin/kamailio[20379]: ERROR: <script>: DEFAULT
REPLY ROUTE: avp(s:testavp) = <null>
Aug  2 09:09:10 flake /usr/sbin/kamailio[20380]: ERROR: <script>: REPLY
ROUTE REPLY_ONE: avp(s:testavp) = <null>
Aug  2 09:09:13 flake /usr/sbin/kamailio[20379]: ERROR: <script>: FAILURE
ROUTE FAIL_ONE: avp(s:testavp) = <null>


Then I installed the nightly build (Ubuntu Lucid), and tried that again:

Aug  2 09:26:52 flake /usr/sbin/kamailio[22676]: ERROR: <script>: BEFORE
UAC: avp(s:testavp) = ThisIsJustATest
Aug  2 09:26:52 flake /usr/sbin/kamailio[22676]: INFO: avpops
[avpops_impl.c:1328]: p=0x7f04261307f8, flags=0x0113
Aug  2 09:26:52 flake /usr/sbin/kamailio[22676]: INFO: avpops
[avpops_impl.c:1332]: #011#011#011name=<testavp>
Aug  2 09:26:52 flake /usr/sbin/kamailio[22676]: INFO: avpops
[avpops_impl.c:1340]: #011#011#011val_str=<ThisIsJustATest / 15>
Aug  2 09:26:52 flake /usr/sbin/kamailio[22676]: ERROR: <script>: AFTER UAC:
avp(s:testavp) = ThisIsJustATest
Aug  2 09:26:52 flake /usr/sbin/kamailio[22676]: INFO: avpops
[avpops_impl.c:1328]: p=0x7f04261307f8, flags=0x0113
Aug  2 09:26:52 flake /usr/sbin/kamailio[22676]: INFO: avpops
[avpops_impl.c:1332]: #011#011#011name=<testavp>
Aug  2 09:26:52 flake /usr/sbin/kamailio[22676]: INFO: avpops
[avpops_impl.c:1340]: #011#011#011val_str=<ThisIsJustATest / 15>
Aug  2 09:26:52 flake /usr/sbin/kamailio[22675]: ERROR: <script>: DEFAULT
REPLY ROUTE: avp(s:testavp) = <null>
Aug  2 09:26:52 flake /usr/sbin/kamailio[22674]: ERROR: <script>: REPLY
ROUTE REPLY_ONE: avp(s:testavp) = ThisIsJustATest
Aug  2 09:26:52 flake /usr/sbin/kamailio[22674]: INFO: avpops
[avpops_impl.c:1328]: p=0x7f04261307f8, flags=0x0113
Aug  2 09:26:52 flake /usr/sbin/kamailio[22674]: INFO: avpops
[avpops_impl.c:1332]: #011#011#011name=<testavp>
Aug  2 09:26:52 flake /usr/sbin/kamailio[22674]: INFO: avpops
[avpops_impl.c:1340]: #011#011#011val_str=<ThisIsJustATest / 15>
Aug  2 09:26:54 flake /usr/sbin/kamailio[22676]: ERROR: <script>: FAILURE
ROUTE FAIL_ONE: avp(s:testavp) = ThisIsJustATest
Aug  2 09:26:54 flake /usr/sbin/kamailio[22676]: INFO: avpops
[avpops_impl.c:1328]: p=0x7f04261307f8, flags=0x0113
Aug  2 09:26:54 flake /usr/sbin/kamailio[22676]: INFO: avpops
[avpops_impl.c:1332]: #011#011#011name=<testavp>
Aug  2 09:26:54 flake /usr/sbin/kamailio[22676]: INFO: avpops
[avpops_impl.c:1340]: #011#011#011val_str=<ThisIsJustATest / 15>


Looks exactly as expected. Thanks! I think you can go ahead and
backport it to 3.1.

Best Regards,
Sebastian

--
Dipl.-Inf.
Sebastian Damm - VoIP-Engineer - damm at sipgate.de

sipgate GmbH - Gladbacher Str. 74 - 40219 Düsseldorf
HRB Düsseldorf 39841 - Geschäftsführer: Thilo Salmon, Tim Mois
Steuernummer: 106/5724/7147, Umsatzsteuer-ID: DE219349391

www.sipgate.de - www.sipgate.at - www.sipgate.co.uk
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.sip-router.org/pipermail/sr-users/attachments/20110802/a1e181d4/attachment-0001.htm>