Jump to content

Porting important fixes from mod_kazoo to mod_erlang_event


Recommended Posts

Hi there, 

It seems mod_kazoo is primarily originated from mod_erlang_event. I understand performance issues & limitations, but it appears mod_erlang_event has not been updated in decade!  I'm trying mod_erlang_event & seems register_event_handler call fails and after doing some basic comparisons here are few things I found:

#1- In mod_kazoo we are using write() API instead FreeSwitch switch_socket_send(). It did solve the issues but curious why ? 

 mod_kazoo: 
   write(listener->sockdes, msgbuf, index)
 mod_elang_event:     
   switch_socket_send(sock, msgbuf, (switch_size_t *) &index);

#2 ei_init() is also added in kz_erl_init  & teardown kz_erl_shutdown()  for new Erlang releases.


Please share if there any other known issues that can be shared and we can port in mod_erlang_event module

Thanks a lot

Edited by btel (see edit history)
Link to comment
Share on other sites

  • 2600Hz Employees

For clarity: we leveraged mod_erlang_event at the start of the project. We wrote, from scratch, mod_kazoo to better handle our use cases.

The main one was mod_erlang_event expected only one Erlang node to connect to the FreeSWITCH node; it was a happy bug in handling connections that allowed us to have multiple ecallmgrs connect to mod_erlang_event.

But there are many performance and features in mod_kazoo that KAZOO utilizes that mod_erlang_event didn't provide.

mod_erlang_event is also written (and probably not maintained anymore) by Andrew Thompson (aka Vagabond). I'm not surprised the code isn't working.

In short, mod_erlang_event is not compatible with KAZOO installations - any use or issues with it can be taken up with Signalwire but not sure they're keen on maintaining it either.

Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

×
×
  • Create New...