-
Notifications
You must be signed in to change notification settings - Fork 770
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
Consider session terminated on 408 and 481 response #3457
Conversation
This failed pjsua test. |
If you have the test case for this one, please let us know. |
Yes we do have the test case:https://www.etsi.org/deliver/etsi_ts/102000_102099/10202702/04.01.01_60/ts_10202702v040101p.pdf The pjsip based application is the UAC
The error is that first the UAC (pjsip based application) sends a BYE and receives an OK. Then the UAS (test software) sends a BYE on which UAC reply OK. According to the test, it should send 481 Call Leg/Transaction Does Not Exist. |
More details is that the second BYE sent by the test software have |
I think you may have mistaken this with the other PR #3456 |
yeah sorry about that. They are similar but not the same.
Compared to the other where a new BYE is sent from the test software, a 481 is sent and after that a BYE. According to the test it should be terminated. The actual test is this: SIP_CC_OE_CR_V_009 |
The difference is therefore that #3456 Terminates correctly, While this should get terminated with due to a 481. Also, the second BYE is again sent with |
The failed test is because the patch also affects other method as well (such as UPDATE during early state). Since the issue is only for BYE, I think it's better to add check for the method. Please check the modified patch: |
Client must consider session terminated as soon as BYE is passed to the transaction client. On reseption of 408 or 481, the UAC MUST consider the session and the dialog terminated.
069d7d3
to
c0fa171
Compare
Applied the modified patch |
Btw, would you be able to immediately confirm whether any of our proposed patches work? Or do you need to wait some time, for example you have to resubmit to them and wait for the test results back? |
We can retest them, but it may take some time for us to get the time to go through the test. The original patches do work. |
Client must consider session terminated as soon as BYE is passed to the transaction client. On reseption of 408 or 481, the UAC MUST consider the session and the dialog terminated.