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

app_dial: Add dial time for progress/ringing. #589

Merged

Conversation

InterLinked1
Copy link
Contributor

Add a timeout option to control the amount of time to wait if no early media is received before giving up. This allows aborting early if the destination
is not being responsive.

Resolves: #588

UserNote: The timeout argument to Dial now allows
specifying the maximum amount of time to dial if
early media is not received.

@InterLinked1
Copy link
Contributor Author

cherry-pick-to: 18
cherry-pick-to: 20
cherry-pick-to: 21

@chrsmj
Copy link

chrsmj commented Feb 20, 2024

The use of pipe as argument separator is discouraged, per conversation at AstriCon. (It was used before the switch to comma many ages ago.) Maybe try caret ? @jcolp

Copy link
Member

@jcolp jcolp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should have test coverage added as well.

apps/app_dial.c Outdated Show resolved Hide resolved
Add a timeout option to control the amount of time
to wait if no early media is received before giving
up. This allows aborting early if the destination
is not being responsive.

Resolves: asterisk#588

UserNote: The timeout argument to Dial now allows
specifying the maximum amount of time to dial if
early media is not received.
@InterLinked1
Copy link
Contributor Author

Tests added here: asterisk/testsuite#44

@InterLinked1 InterLinked1 requested a review from jcolp March 3, 2024 19:50
@gtjoseph gtjoseph added the cherry-pick-test Trigger dry run of cherry-picks label Mar 5, 2024
@github-actions github-actions bot added cherry-pick-testing-in-progress Cherry-Pick tests in progress cherry-pick-checks-passed Cherry-Pick checks passed cherry-pick-gates-failed Cherry-Pick gates failed and removed cherry-pick-test Trigger dry run of cherry-picks cherry-pick-testing-in-progress Cherry-Pick tests in progress labels Mar 5, 2024
@gtjoseph gtjoseph added the cherry-pick-test Trigger dry run of cherry-picks label Mar 6, 2024
@github-actions github-actions bot added cherry-pick-testing-in-progress Cherry-Pick tests in progress cherry-pick-checks-passed Cherry-Pick checks passed cherry-pick-gates-failed Cherry-Pick gates failed and removed cherry-pick-test Trigger dry run of cherry-picks cherry-pick-checks-passed Cherry-Pick checks passed cherry-pick-gates-failed Cherry-Pick gates failed cherry-pick-testing-in-progress Cherry-Pick tests in progress labels Mar 6, 2024
Copy link

github-actions bot commented Mar 6, 2024

Successfully merged to branch master and cherry-picked to ["18","20","21"]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[new-feature]: app_dial: Allow Dial to be aborted if early media is not received
4 participants