[prev in list] [next in list] [prev in thread] [next in thread] 

List:       sr-users
Subject:    Re: [SR-Users] bug in tm module?
From:       Henning Westerholt <hw () gilawa ! com>
Date:       2022-03-23 20:21:26
Message-ID: PAXPR05MB8558B125061A99B7E6A3D352BF189 () PAXPR05MB8558 ! eurprd05 ! prod ! outlook ! com
[Download RAW message or body]

Hello,

If you think there is a bug in the tm module, you could create an issue in our \
tracker here: https://github.com/kamailio/kamailio/issues/new?assignees=&labels=&template=bug_report.md


Please also add some description of the wrong behaviour that you are seeing in the \
server to it.

Thanks,

Henning

From: sr-users <sr-users-bounces@lists.kamailio.org> On Behalf Of Daniel Paape
Sent: Wednesday, March 23, 2022 9:16 PM
To: sr-users@lists.kamailio.org
Subject: [SR-Users] bug in tm module?

Hello
 I'm running K5.5.3 and have checked the 'issues' link in github and saw nothing \
regarding what I am seeing.  I think that there is a bug in the tm module with \
respect to the "tm:local-response" event route.  In the "t_reply.c" file, there is a \
static variable called '_tm_local_response_set_lookup'.  This variable is initialized \
at load time to zero.  It is checked in the '_reply_light()' routine and will \
initiate a local callback from the config script if "armed".  The problem as I see it \
is that if the callback is readied, the variable is set to one.  But it is never \
reset. So the practical outworking is as follows.  A register is received and the \
request_route arms the callback.  The register requires an authorization (local \
database sqlite).  After the 401 is sent back, the callback via the module event \
route is called as expected.  All good except that the module event route script \
fragment is never executed again after the first call. I'm no expert on the tm module \
but I would think that somewhere in there, the variable should be reset to zero so \
that subsequent transactions can initiate the event route again.

thanks for any help,
Dan


[Attachment #3 (text/html)]

<html xmlns:v="urn:schemas-microsoft-com:vml" \
xmlns:o="urn:schemas-microsoft-com:office:office" \
xmlns:w="urn:schemas-microsoft-com:office:word" \
xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" \
xmlns="http://www.w3.org/TR/REC-html40"> <head>
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">
<meta name="Generator" content="Microsoft Word 15 (filtered medium)">
<style><!--
/* Font Definitions */
@font-face
	{font-family:"Cambria Math";
	panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
	{font-family:Calibri;
	panose-1:2 15 5 2 2 2 4 3 2 4;}
@font-face
	{font-family:"Helvetica Neue";}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
	{margin:0cm;
	font-size:11.0pt;
	font-family:"Calibri",sans-serif;}
a:link, span.MsoHyperlink
	{mso-style-priority:99;
	color:#0563C1;
	text-decoration:underline;}
span.E-MailFormatvorlage20
	{mso-style-type:personal-reply;
	font-family:"Calibri",sans-serif;
	color:windowtext;}
.MsoChpDefault
	{mso-style-type:export-only;
	font-size:10.0pt;}
@page WordSection1
	{size:612.0pt 792.0pt;
	margin:70.85pt 70.85pt 2.0cm 70.85pt;}
div.WordSection1
	{page:WordSection1;}
--></style><!--[if gte mso 9]><xml>
<o:shapedefaults v:ext="edit" spidmax="1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext="edit">
<o:idmap v:ext="edit" data="1" />
</o:shapelayout></xml><![endif]-->
</head>
<body lang="DE" link="#0563C1" vlink="#954F72" style="word-wrap:break-word">
<div class="WordSection1">
<p class="MsoNormal"><span lang="EN-GB" \
style="mso-fareast-language:EN-US">Hello,<o:p></o:p></span></p> <p \
class="MsoNormal"><span lang="EN-GB" \
style="mso-fareast-language:EN-US"><o:p>&nbsp;</o:p></span></p> <p \
class="MsoNormal"><span lang="EN-GB" style="mso-fareast-language:EN-US">If you think \
there is a bug in the tm module, you could create an issue in our tracker here: <a \
href="https://github.com/kamailio/kamailio/issues/new?assignees=&amp;labels=&amp;template=bug_report.md">
 https://github.com/kamailio/kamailio/issues/new?assignees=&amp;labels=&amp;template=bug_report.md</a><o:p></o:p></span></p>
 <p class="MsoNormal"><span lang="EN-GB" \
style="mso-fareast-language:EN-US"><o:p>&nbsp;</o:p></span></p> <p \
class="MsoNormal"><span lang="EN-GB" style="mso-fareast-language:EN-US">Please also \
add some description of the wrong behaviour that you are seeing in the server to \
it.<o:p></o:p></span></p> <p class="MsoNormal"><span lang="EN-GB" \
style="mso-fareast-language:EN-US"><o:p>&nbsp;</o:p></span></p> <p \
class="MsoNormal"><span lang="EN-GB" \
style="mso-fareast-language:EN-US">Thanks,<o:p></o:p></span></p> <p \
class="MsoNormal"><span lang="EN-GB" \
style="mso-fareast-language:EN-US"><o:p>&nbsp;</o:p></span></p> <p \
class="MsoNormal"><span lang="EN-GB" \
style="mso-fareast-language:EN-US">Henning<o:p></o:p></span></p> <p \
class="MsoNormal"><span lang="EN-GB" \
style="mso-fareast-language:EN-US"><o:p>&nbsp;</o:p></span></p> <div>
<div style="border:none;border-top:solid #E1E1E1 1.0pt;padding:3.0pt 0cm 0cm 0cm">
<p class="MsoNormal" style="margin-left:35.4pt"><b>From:</b> sr-users \
&lt;sr-users-bounces@lists.kamailio.org&gt; <b>On Behalf Of </b>Daniel Paape<br>
<b>Sent:</b> Wednesday, March 23, 2022 9:16 PM<br>
<b>To:</b> sr-users@lists.kamailio.org<br>
<b>Subject:</b> [SR-Users] bug in tm module?<o:p></o:p></p>
</div>
</div>
<p class="MsoNormal" style="margin-left:35.4pt"><o:p>&nbsp;</o:p></p>
<div>
<div>
<div>
<div>
<div>
<div>
<div>
<div>
<div>
<div>
<div>
<div>
<div>
<p class="MsoNormal" style="margin-left:35.4pt"><span \
style="font-size:10.0pt;font-family:&quot;Helvetica Neue&quot;;color:black">Hello \
<o:p></o:p></span></p> <div>
<p class="MsoNormal" style="margin-left:35.4pt"><span \
style="font-size:10.0pt;font-family:&quot;Helvetica Neue&quot;;color:black">&nbsp;I'm \
running K5.5.3 and have checked the 'issues' link in github and saw nothing regarding \
what I am seeing.&nbsp; I think that there is a bug  in the tm module with respect to \
the &quot;tm:local-response&quot; event route.&nbsp; In the &quot;t_reply.c&quot; \
file, there is a static variable called '_tm_local_response_set_lookup'.&nbsp; This \
variable is initialized at load time to zero.&nbsp; It is checked in the \
'_reply_light()' routine  and will initiate a local callback from the config script \
if &quot;armed&quot;.&nbsp; The problem as I see it is that if the callback is \
readied, the variable is set to one.&nbsp; But it is never \
reset.<o:p></o:p></span></p> </div>
<div>
<p class="MsoNormal" style="margin-left:35.4pt"><span \
style="font-size:10.0pt;font-family:&quot;Helvetica Neue&quot;;color:black">So the \
practical outworking is as follows.&nbsp; A register is received and the \
request_route arms the callback.&nbsp; The register requires an authorization  (local \
database sqlite).&nbsp; After the 401 is sent back, the callback via the module event \
route is called as expected.&nbsp; All good except that the module event route script \
fragment is never executed again after the first call.<o:p></o:p></span></p> </div>
<div>
<p class="MsoNormal" style="margin-left:35.4pt"><span \
style="font-size:10.0pt;font-family:&quot;Helvetica Neue&quot;;color:black">I'm no \
expert on the tm module but I would think that somewhere in there, the variable \
should be reset to zero so that subsequent transactions  can initiate the event route \
again.<o:p></o:p></span></p> </div>
<div>
<p class="MsoNormal" style="margin-left:35.4pt"><span \
style="font-size:10.0pt;font-family:&quot;Helvetica \
Neue&quot;;color:black"><o:p>&nbsp;</o:p></span></p> </div>
<div>
<p class="MsoNormal" style="margin-left:35.4pt"><span \
style="font-size:10.0pt;font-family:&quot;Helvetica Neue&quot;;color:black">thanks \
for any help,<o:p></o:p></span></p> </div>
<div>
<p class="MsoNormal" style="margin-left:35.4pt"><span \
style="font-size:10.0pt;font-family:&quot;Helvetica \
Neue&quot;;color:black">Dan<o:p></o:p></span></p> </div>
</div>
</div>
</div>
</div>
</div>
</div>
</div>
</div>
</div>
</div>
</div>
</div>
<p class="MsoNormal" style="margin-left:35.4pt"><span \
style="font-size:12.0pt;color:black"><o:p>&nbsp;</o:p></span></p> </div>
</div>
</body>
</html>



__________________________________________________________
Kamailio - Users Mailing List - Non Commercial Discussions
  * sr-users@lists.kamailio.org
Important: keep the mailing list in the recipients, do not reply only to the sender!
Edit mailing list options or unsubscribe:
  * https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-users

--===============1261054962==--


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic