I find the name with potential to create confusion, being closed to NAT 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.

“msg_”
“mbr_”
“mbus_”

Adding a prefix is a good thing.

/O

On 28 Jun 2021, at 09:42, Daniel-Constantin Mierla <notifications@github.com> wrote:


Thanks for the new module!

I find the name with potential to create confusion, being closed to NAT 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.

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. @eschmidbauer has already git commit access, so I am fine to merge it and then push follow up commits for memory allocation checks.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

_______________________________________________
Kamailio (SER) - Development Mailing List
sr-dev@lists.kamailio.org
https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-dev