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 a park timeout #49

Open goodboy opened 6 years ago

goodboy commented 6 years ago

This is to address concerns that @k4ml had in #47.

I've added the use of the park_timeout channel variable to our CI dialplan as well as a test to verify that it works as intended and rejects unhanded sessions after 3 seconds. There is an additional test case which verifies that if a session which is parked (with a timeout) is moved out of the CHANNEL_PARK state the timeout is not carried into the downstream app.

goodboy commented 6 years ago

Just as a note for anyone who's following this and #47 - I've spoken to core FS and they'll consider a PR for a park_activity_timeout which would be a timeout that's reset each time the park app loop is re-entered after each app invocation. This would serve as a type of idle timeout where if no app are activated in time the channel will be destroyed.

I just need to find the time to write up the patch.