#507 b2b: CANCEL w/o Reason hdr

1.8.x
closed-fixed
modules (454)
7
2012-05-15
2012-04-10
Ovidiu Sas
No

Rev 8875 changed the way transaction are cancelled.
Before, the b2b logic was able to push a Reason header down to tm to be appended to the CANCEL request.
The new way of sending CANCEL, doesn't allow setting extra headers in CANCEL requests:
http://opensips.svn.sourceforge.net/viewvc/opensips/trunk/modules/b2b_entities/dlg.c?r1=8875&r2=8874&pathrev=8875
line 1687

The tm is able to automatically add Reason header for cancelled transaction when a branch is answered, but for b2b. the transaction is not properly detected as being in "200" state.
When a call is forked in b2b mode and the answered, CANCEL requests are sent out to the other branches, but the Reason header is not set.
The reason for this is because in tm:build_cancel() the Trans->uas.status is still set to "180".

We have two issues here:
1. the ability to push extra headers in CANCEL via tm api was lost;
2. the tm uas transaction status is not properly set into a 200 state when a 200 ok is received on a branch.

Discussion

  • Bogdan-Andrei Iancu

    • assigned_to: nobody --> bogdan_iancu
    • status: open --> open-accepted
     
  • Bogdan-Andrei Iancu

    • priority: 5 --> 7
     
  • Bogdan-Andrei Iancu

    OK, issue found, the fix is under way.

     
  • Bogdan-Andrei Iancu

    The fix is on SVN trunk and 1.8.

    Thanks and regards,
    Bogdan

     
  • Bogdan-Andrei Iancu

    • status: open-accepted --> closed-fixed
     

Log in to post a comment.

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:





No, thanks