Skip to content
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

Asterisk12 async agi events #143

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

ilgiz-badamshin
Copy link

No description provided.

@coveralls
Copy link

coveralls commented Sep 28, 2017

Coverage Status

Coverage decreased (-6.2%) to 91.956% when pulling d662829 on ilgiz-badamshin:asterisk12 into f8b5999 on marcelog:master.

@marcelog
Copy link
Owner

Hello,

Thanks for the pull. Could you please provide a description for the changes and also unit tests?

Cheers,

@ilgiz-badamshin
Copy link
Author

Asterisk 12+ trigger AsyncAGIStartEvent instead AsyncEvent with sub event "Start". And it trigger AsyncAGIExecEvent where we should handle lastCommmandId. So extracted two interfaces to detect this events.
Also added "ASYNCAGI BREAK" call in AsyncClientImpl to return control to dialplan:
https://wiki.asterisk.org/wiki/display/AST/AGICommand_asyncagi+break

@coveralls
Copy link

coveralls commented Nov 24, 2017

Coverage Status

Coverage decreased (-6.6%) to 91.485% when pulling efb5748 on ilgiz-badamshin:asterisk12 into f8b5999 on marcelog:master.

dkgroot added a commit to chan-sccp/PAMI that referenced this pull request Apr 24, 2019
…lgiz-badamshin

Asterisk12 async agi events
Asterisk 12+ trigger AsyncAGIStartEvent instead AsyncEvent with sub event "Start".
It will trigger an AsyncAGIExecEvent, where can handle lastCommmandId.
Extracted two interfaces to detect these seperate events.
Also added "ASYNCAGI BREAK" call in AsyncClientImpl to return control to dialplan:
See: https://wiki.asterisk.org/wiki/display/AST/AGICommand_asyncagi+break

PR by @ilgiz-badamshin (marcelog#143)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants