Skip to content

channels/pjsip: Add tests for SIP MESSAGE redirect functionality #117

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.

Sign up for GitHub

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

Jump to bottom
Open
maximilianfridrich wants to merge 1 commit into
base: master
Choose a base branch
from
Open

channels/pjsip: Add tests for SIP MESSAGE redirect functionality #117

maximilianfridrich wants to merge 1 commit into from

Conversation

@maximilianfridrich
Copy link
Contributor

@maximilianfridrich maximilianfridrich commented Nov 7, 2025 •
edited
Loading

This commit adds comprehensive test coverage for SIP MESSAGE redirect handling with 305 Use Proxy responses in Asterisk's PJSIP stack.

Test coverage includes:

  • Basic MESSAGE redirect with follow_redirect_methods=message configured
  • Verification that redirects are NOT followed when follow_redirect_methods is unset
  • Multiple Contact header handling with q-value priority
  • Contact failover when higher-priority contacts reject (4xx responses)
  • Redirect loop detection and prevention

github-actions[bot]
github-actions bot previously requested changes Nov 7, 2025
View reviewed changes
Copy link

@github-actions github-actions bot left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Pull Request Checklist Complete

@github-actions
Copy link

github-actions bot commented Nov 7, 2025

Workflow PRCheck failed
master-gates: FAILED TEST: channels/pjsip/message/message_redirect
master-gates: FAILED TEST: channels/pjsip/message/message_redirect_multiple

@maximilianfridrich
Copy link
Contributor Author

maximilianfridrich commented Nov 7, 2025 •
edited
Loading

The two new message/message_redirect* tests that failed are exactly the ones that I would expect to fail without the new Asterisk improvement. The tests pass in the corresponding Asterisk PR with the redirect improvement which uses this testsuite PR: asterisk/asterisk#1580 (link to corresponding job).

@maximilianfridrich
Copy link
Contributor Author

maximilianfridrich commented Nov 7, 2025 •
edited by gtjoseph
Loading

cherry-pick-to: 20
cherry-pick-to: 22
cherry-pick-to: 23
asterisk-test-pr: 1580

@gtjoseph
Copy link
Member

gtjoseph commented Nov 7, 2025

I probably forgot to announce this but you can now add asterisk-test-pr: <pr> to testsuite PRs just like you added testsuite-test-pr to your asterisk PR. I'll update your comment.

@github-actions github-actions bot dismissed their stale review November 7, 2025 13:52

Pull Request Checklist Complete

@github-actions
Copy link

github-actions bot commented Nov 7, 2025

Workflow PRCheck completed successfully

@maximilianfridrich
channels/pjsip: Add tests for SIP MESSAGE redirect functionality
a5437ba 
This commit adds comprehensive test coverage for SIP MESSAGE redirect
handling with 305 Use Proxy responses in Asterisk's PJSIP stack.

Test coverage includes:
- Basic MESSAGE redirect with follow_redirect_methods=message configured
- Verification that redirects are NOT followed when
  follow_redirect_methods is unset
- Multiple Contact header handling with q-value priority
- Contact failover when higher-priority contacts reject (4xx responses)
- Redirect loop detection and prevention
@maximilianfridrich
Copy link
Contributor Author

maximilianfridrich commented Dec 12, 2025

Updated in accordance with the Asterisk PR - the redirect feature is now configured using the new pjsip endpoint option follow_redirect_methods.

github-actions[bot]
github-actions bot requested changes Dec 12, 2025
View reviewed changes
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Attention! This pull request may contain issues that could prevent it from being accepted. Please review the checklist below and take the recommended action. If you believe any of these are not applicable, just add a comment and let us know.

  • The PR description does not match the commit message body. This can cause confusion for reviewers and future maintainers. GitHub doesn't automatically update the PR description when you update the commit message so if you've updated the commit with a force-push, please update the PR description to match the new commit message body.

Documentation:

@github-actions
Copy link

github-actions bot commented Dec 12, 2025

Workflow PRCheck completed successfully

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

Reviewers

@github-actions github-actions[bot] github-actions[bot] requested changes

@jcolp jcolp jcolp approved these changes

Assignees

No one assigned

Labels

pr-submit-tests-passed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@maximilianfridrich @gtjoseph @jcolp