Skip to content

Conversation

@sfgeorge
Copy link

@sfgeorge sfgeorge commented Jul 19, 2019

Document that #execute_agi_command returns nil upon hangup.

@sfgeorge sfgeorge force-pushed the DTPORTAL-16140-detect-hangup-in-execute_agi_command branch 2 times, most recently from 250a4e6 to 24328b9 Compare July 20, 2019 00:39
@sfgeorge sfgeorge changed the title DTPORTAL-16140: Detect hangup in execute_agi_command DTPORTAL-16140: Document that #execute_agi_command returns nil upon hangup Jul 20, 2019
@sfgeorge
Copy link
Author

Since this PR is whittled down to no longer change functionality, and merely to document & enhance specs, I'll close this PR and open it on the upstream adhearsion/adhearsion repo instead.

@sfgeorge sfgeorge force-pushed the DTPORTAL-16140-detect-hangup-in-execute_agi_command branch from 24328b9 to 9c10a5e Compare July 21, 2019 01:26
@sfgeorge sfgeorge closed this Jul 21, 2019
@sfgeorge sfgeorge reopened this Jul 21, 2019
@sfgeorge
Copy link
Author

lol, whoops. This only makes sense here at the moment. The upstream adhearsion doesn't have the protection that our fork has to ensure that hungup calls correctly send a nil event in order to cause #execute_agi_command to end. (When I tried this new spec on the upstream adhearsion without the fix, the spec merely hanged indefinitely)

@sfgeorge
Copy link
Author

Closing (again) in favor of #21. 🚪🐈

@sfgeorge sfgeorge closed this Jul 21, 2019
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.

1 participant