Skip to content

tests/channels/pjsip/message: Add out-of-dialog Contact stripping test#124

Open
AlexisHadj wants to merge 1 commit intoasterisk:masterfrom
AlexisHadj:testRFC3428
Open

tests/channels/pjsip/message: Add out-of-dialog Contact stripping test#124
AlexisHadj wants to merge 1 commit intoasterisk:masterfrom
AlexisHadj:testRFC3428

Conversation

@AlexisHadj
Copy link

This new test verifies that Asterisk correctly strips the Contact header from outbound MESSAGE requests, as required by RFC 3428.

1-Added SIPp scenarios to send and receive MESSAGEs with inverse
validation on the Contact header.
2-Added Asterisk configuration (pjsip.conf, extensions.conf) to
support the test.
3-Registered the new test in the pjsip message test suite.

Note: This test is associated with the change in Pull Request #1729

This new test verifies that Asterisk correctly strips the Contact
header from outbound MESSAGE requests, as required by RFC 3428.

1-Added SIPp scenarios to send and receive MESSAGEs with inverse
  validation on the Contact header.
2-Added Asterisk configuration (pjsip.conf, extensions.conf) to
  support the test.
3-Registered the new test in the pjsip message test suite.
@AlexisHadj
Copy link
Author

AlexisHadj commented Feb 12, 2026

Note: This test is associated with the change in Pull Request #1729 therefore should be merged after Pull Request #1729 . That is the reason master-gates check fails with ".....Regexp matched but should not....""

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 title does not match the commit title. This can cause confusion for reviewers and future maintainers. GitHub doesn't automatically update the PR title when you update the commit message so if you've updated the commit with a force-push, please update the PR title to match the new commit message body.
  • 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.
  • The are no cherry-pick-to headers in any comment in this PR. If the PR applies to more than just the branch it was submitted against, please add a comment with one or more cherry-pick-to: <branch> headers or a comment with cherry-pick-to: none to indicate that this PR shouldn't be cherry-picked to any other branch. See the Code Contribution documentation for more information.

Documentation:

@github-actions
Copy link

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

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.

1 participant