search

friends-of-freeswitch / switchio

asyncio powered FreeSWITCH cluster control
http://switchio.rtfd.io
Mozilla Public License 2.0
191 stars 28 forks source link

Add sendmsg support #52

Closed goodboy closed 6 years ago

goodboy commented 6 years ago

Since moving to FreeSWITCH 1.6 I've had to disable a couple of the unit tests because of that I thought was a problem with certain events not being emitted by FS (in this case RECORD_STOP). After further investigation as part of #49 and #50 I am instead convinced that the problem is actually in using uuid_broadcast.

I believe this command is not working reliably (at least the same way it used to) and that parked sessions are not executing new dialplan apps requested using this method.

uuid_broadcast was only used initially due to switchy's dependence on the SWIG connection client (which we've since discarded) and not having an async way to invoke dialplan apps (i.e. there was no async way to support sendmsg). Now that we control the lower inbound protocol it makes to add this API support and stop using uuid_broadcast which is just a wrapper around what sendmsg would be doing anyway.

@moises-silva I'd appreciate your feedback on this as well!

goodboy commented 6 years ago

Heh I was totally right. Everything works perfectly with sendmsg...