Add sendmsg support #52
Assignees
Labels
Comments
|
Heh I was totally right. Everything works perfectly with |
goodboy
pushed a commit
that referenced
this issue
Dec 3, 2017
Deprecate `Session.broadcast()` as the FS core behaviour is unreliable. Add `execute()` support and use throughout the session API for all FS core app commands. Resolves #52
Merged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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 usinguuid_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_broadcastwas only used initially due toswitchy'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 supportsendmsg). Now that we control the lower inbound protocol it makes to add this API support and stop usinguuid_broadcastwhich is just a wrapper around whatsendmsgwould be doing anyway.@moises-silva I'd appreciate your feedback on this as well!
The text was updated successfully, but these errors were encountered: