-
Notifications
You must be signed in to change notification settings - Fork 34
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Handle Asterisk 'channel not found' errors correctly #169
Conversation
…not found' output
…xecution of AGI commands
…hannels Ref should not be sent until after recording starts
@vindir This should be done with the exception of output components, which I will fix shortly. |
Changes Unknown when pulling 2c72638 on feature/channel-not-found into * on develop*. |
You are a a can of awesome sauce. I'll get it merged and tested once it's ready. |
…ry to execute on dead channels
@vindir Please test with your project and let me know if this solves the problem :) Once you report back I'll get this merged to develop. |
…ture separate from AGI
def execute(ami_client) | ||
ami_client.send_action 'AGI', 'Channel' => @channel, 'Command' => agi_command, 'CommandID' => id | ||
rescue RubyAMI::Error => e |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Need to get rid of the duplication between this and Call#send_ami_action before merging.
WIP. This needs extending to all touches to Asterisk for both AMI and AGI, including handling differently in components vs basic commands.