[SR-Users] potential PRACK related topos bug

Daniel-Constantin Mierla miconda at gmail.com
Tue Apr 20 12:20:41 CEST 2021


On 20.04.21 11:23, Juha Heinanen wrote:
> Daniel-Constantin Mierla writes:
>
>> The 404 is not sent from the C code of the topos module, can you
>> identify what are the cases in your config when a 404 is replied? PRACK
>> should have To-tag, so it should via requests within dialog branch.
> Yes, 404 is sent from the script:
>
>    if (!loose_route_mode("1")) {
>         if (is_method("ACK")) {
>             if (t_check_trans()) {
>                 t_relay();
>             } else {
>                 xdbg("Discarding unmatched $rm <$ru> by <$fu>\n");
>             };
>         } else {
>             xnotice("Discarding in-dialog $rm <$ru> without Route header\n");
>             send_reply("404", "Not found");
>         };
>         exit;
>     };
>
> This has worked OK also with 183/PRACK when topos is not in use.  If I
> add PRACK to the if test:
>
>         if (is_method("ACK") || is_method("PRACK")) {
>
> then "Discarding unmatched PRACK ..." is written to log.

Can you try with latest master branch? I pushed a commit to topos module.

Cheers,
Daniel

-- 
Daniel-Constantin Mierla -- www.asipto.com
www.twitter.com/miconda -- www.linkedin.com/in/miconda
Kamailio Advanced Training - Online
May 17-20, 2021 (Europe Timezone) - June 7-10, 2021 (America Timezone)
  * https://www.asipto.com/sw/kamailio-advanced-training-online/




More information about the sr-users mailing list