<div dir="auto">Maybe<div dir="auto">q_</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Jun 28, 2021, 2:15 PM Henning Westerholt <<a href="mailto:hw@skalatan.de">hw@skalatan.de</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">





<div lang="DE" link="blue" vlink="purple" style="word-wrap:break-word">
<div class="m_2165621772853416078WordSection1">
<p class="MsoNormal"><span lang="EN-GB">Think the same, as we have already three modules implementing message queues:<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-GB"><u></u> <u></u></span></p>
<p class="MsoNormal"><span lang="EN-GB">- mqtt<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-GB">- rabbitmq<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-GB">- kazoo<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-GB"><u></u> <u></u></span></p>
<p class="MsoNormal"><span lang="EN-GB">I was thinking of “mq_” prefix, but this obviously do not fit well to the existing names.<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-GB"><u></u> <u></u></span></p>
<p class="MsoNormal"><span lang="EN-GB">Cheers,<u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-GB"><u></u> <u></u></span></p>
<p class="MsoNormal"><span lang="EN-GB">Henning<u></u><u></u></span></p>
<div>
<p class="MsoNormal"><span lang="EN-GB">-- <u></u>
<u></u></span></p>
<p class="MsoNormal"><span lang="EN-GB">Henning Westerholt –
</span><span><a href="https://skalatan.de/blog/" target="_blank" rel="noreferrer"><span lang="EN-GB" style="color:#0563c1">https://skalatan.de/blog/</span></a></span><span lang="EN-GB"><u></u><u></u></span></p>
<p class="MsoNormal"><span lang="EN-GB">Kamailio services –
</span><span><a href="https://gilawa.com/" target="_blank" rel="noreferrer"><span lang="EN-GB" style="color:#0563c1">https://gilawa.com</span></a></span><span>
<span lang="EN-GB"><u></u><u></u></span></span></p>
</div>
<p class="MsoNormal"><span lang="EN-GB"><u></u> <u></u></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-dev <<a href="mailto:sr-dev-bounces@lists.kamailio.org" target="_blank" rel="noreferrer">sr-dev-bounces@lists.kamailio.org</a>>
<b>On Behalf Of </b>Olle E. Johansson<br>
<b>Sent:</b> Monday, June 28, 2021 1:10 PM<br>
<b>To:</b> Kamailio (SER) - Development Mailing List <<a href="mailto:sr-dev@lists.kamailio.org" target="_blank" rel="noreferrer">sr-dev@lists.kamailio.org</a>><br>
<b>Subject:</b> Re: [sr-dev] [kamailio/kamailio] nats: new nats message consumer module (#2790)<u></u><u></u></p>
</div>
</div>
<p class="MsoNormal" style="margin-left:35.4pt"><u></u> <u></u></p>
<blockquote style="margin-top:5.0pt;margin-bottom:5.0pt">
<p class="MsoNormal" style="margin-left:35.4pt">
I find the name with potential to create confusion, being closed to <code><span style="font-size:10.0pt">NAT</span></code> and we all know how popular that is and what it refers to in VoIP, but at the end I am fine with it given that it is the name of the external
 project. Maybe in the future if we end up with many distributed messaging platform modules, we can add a common prefix to all of them like we added for db module in the past.<u></u><u></u></p>
</blockquote>
<div>
<p class="MsoNormal" style="margin-left:35.4pt">“msg_”<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal" style="margin-left:35.4pt">“mbr_”<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal" style="margin-left:35.4pt">“mbus_”<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal" style="margin-left:35.4pt"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal" style="margin-left:35.4pt">Adding a prefix is a good thing.<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal" style="margin-left:35.4pt"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal" style="margin-left:35.4pt">/O<u></u><u></u></p>
<div>
<p class="MsoNormal" style="margin-left:35.4pt"><br>
<br>
<u></u><u></u></p>
<blockquote style="margin-top:5.0pt;margin-bottom:5.0pt">
<div>
<p class="MsoNormal" style="margin-left:35.4pt">On 28 Jun 2021, at 09:42, Daniel-Constantin Mierla <<a href="mailto:notifications@github.com" target="_blank" rel="noreferrer">notifications@github.com</a>> wrote:<u></u><u></u></p>
</div>
<p class="MsoNormal" style="margin-left:35.4pt"><u></u> <u></u></p>
<div>
<div>
<p class="MsoNormal" style="margin-left:35.4pt"><u></u> <u></u></p>
</div>
<p class="MsoNormal" style="margin-left:35.4pt">
Thanks for the new module!<u></u><u></u></p>
<p class="MsoNormal" style="margin-left:35.4pt">
I find the name with potential to create confusion, being closed to <code><span style="font-size:10.0pt">NAT</span></code> and we all know how popular that is and what it refers to in VoIP, but at the end I am fine with it given that it is the name of the external
 project. Maybe in the future if we end up with many distributed messaging platform modules, we can add a common prefix to all of them like we added for db module in the past.<u></u><u></u></p>
<p class="MsoNormal" style="margin-left:35.4pt">
Regarding the code, I noticed that shm/pkg_malloc() results are not checked for NULL, which will result in crashes on low/out of memory cases.
<a href="https://github.com/eschmidbauer" target="_blank" rel="noreferrer">@eschmidbauer</a> has already git commit access, so I am fine to merge it and then push follow up commits for memory allocation checks.<u></u><u></u></p>
<p class="MsoNormal" style="margin-left:35.4pt">
<span style="font-size:12.0pt;color:#666666">—<br>
You are receiving this because you are subscribed to this thread.<br>
Reply to this email directly, <a href="https://github.com/kamailio/kamailio/pull/2790#issuecomment-869451310" target="_blank" rel="noreferrer">
view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/ABO7UZKHS7YN5JFPHCLSAQ3TVARUZANCNFSM47MJ3QRQ" target="_blank" rel="noreferrer">
unsubscribe</a>.</span><span style="font-size:12.0pt;color:#666666"><img border="0" width="1" height="1" style="width:.0104in;height:.0104in" id="m_2165621772853416078_x0000_i1025" src="https://github.com/notifications/beacon/ABO7UZOVESUHPNRN3KO5S63TVARUZA5CNFSM47MJ3QR2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOGPJMMLQ.gif"></span><span style="font-size:12.0pt;color:#666666"><u></u><u></u></span></p>
<p class="MsoNormal" style="margin-left:35.4pt">_______________________________________________<br>
Kamailio (SER) - Development Mailing List<br>
<a href="mailto:sr-dev@lists.kamailio.org" target="_blank" rel="noreferrer">sr-dev@lists.kamailio.org</a><br>
<a href="https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev" target="_blank" rel="noreferrer">https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev</a><u></u><u></u></p>
</div>
</blockquote>
</div>
<p class="MsoNormal" style="margin-left:35.4pt"><u></u> <u></u></p>
</div>
</div>
</div>

_______________________________________________<br>
Kamailio (SER) - Development Mailing List<br>
<a href="mailto:sr-dev@lists.kamailio.org" target="_blank" rel="noreferrer">sr-dev@lists.kamailio.org</a><br>
<a href="https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev" rel="noreferrer noreferrer" target="_blank">https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev</a><br>
</blockquote></div>