sailfin
  1. sailfin
  2. SAILFIN-1805

BYE is rejected with 403 during Re-INVITE

    Details

    • Issuezilla Id:
      1,805

      Description

      ___ Servlet _____________________ Phone ____

       
      -----INVITE--------------->
      <----200 OK----------------
      -----ACK------------------>
       
      -----Re-INVITE------------>
      <----200 OK----------------
      <----200 OK----------------
      <----BYE-------------------
      -----403 Forbidden-------->
      ...

      The problem: The BYE is rejected by the container (even though it shouldn't be)
      and never arrives at my servlet, and because of this, even though the phone is
      back on the hook, my servlet thinks that the session is still open and sends an
      ACK, and a later Re-INVITE causes the phone to start ringing, etc.

      running Sun GlassFish Communications Server 1.5 (9.1.1) (build b60g-fcs)
      on Linux 2.6.28-11-server #42-Ubuntu SMP Fri Apr 17 02:48:10 UTC 2009 i686 GNU/Linux

      Binod wrote:
      > I cant see the servlet sending ACK to the phone. In that case, the phone
      > (callee) shouldnt send the BYE, right?
      >
      > "However, the callee's UA MUST NOT send a BYE on a confirmed dialog
      > until it has received an ACK for its 2xx response or until the server
      > transaction times out."
      >
      > RFC 3261 says the above.

      haukex wrote:
      > I believe that that paragraph describes initial, dialog-creating INVITEs - in
      > our case the issue is Re-INVITEs in an existing dialog. Section 14 says about
      > Re-INVITEs: "[...] a UA MAY initiate a regular transaction while an
      > [Re-]INVITE transaction is in progress. A UA MAY also initiate an [Re-]INVITE
      > transaction while a regular transaction is in progress."

      Binod wrote:
      > I discussed this with other developers and we think, sailfin is too strict in
      > this case.
      > Can you please file an issue? I will make the code a more flexible.

        Activity

        Hide
        binod added a comment -

        Taking over

        Show
        binod added a comment - Taking over
        Hide
        binod added a comment -

        Starting...

        Show
        binod added a comment - Starting...
        Hide
        binod added a comment -

        Reproduced this locally. Checked in a regression test as well.

        The issue seem to be handling of the CONFIRMED state of INVITE UAC state machine. The container
        doesnt expect any message (other than UPDATE for eg) and replies with 403.

        Testing a potential fix in INVITESession now.

        Show
        binod added a comment - Reproduced this locally. Checked in a regression test as well. The issue seem to be handling of the CONFIRMED state of INVITE UAC state machine. The container doesnt expect any message (other than UPDATE for eg) and replies with 403. Testing a potential fix in INVITESession now.
        Hide
        binod added a comment -

        Checking in src/java/com/ericsson/ssa/sip/INVITESession.java;
        /cvs/sailfin/sip-stack/src/java/com/ericsson/ssa/sip/INVITESession.java,v <-- INVITESession.java
        new revision: 1.50; previous revision: 1.49
        done

        Show
        binod added a comment - Checking in src/java/com/ericsson/ssa/sip/INVITESession.java; /cvs/sailfin/sip-stack/src/java/com/ericsson/ssa/sip/INVITESession.java,v <-- INVITESession.java new revision: 1.50; previous revision: 1.49 done
        Hide
        binod added a comment -

        Updated the version information to reflect where the fix has been checkec in.

        Show
        binod added a comment - Updated the version information to reflect where the fix has been checkec in.
        Hide
        haukex added a comment -

        verified against v2-b28-nightly-13_aug_2009, thanks for the fix!

        Show
        haukex added a comment - verified against v2-b28-nightly-13_aug_2009, thanks for the fix!

          People

          • Assignee:
            binod
            Reporter:
            haukex
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: