search

nexcra / sipservlets

Automatically exported from code.google.com/p/sipservlets
0 stars 0 forks source link

Container generated 408 for a proxy branch is not treated as a branch response #263

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
What steps will reproduce the problem?
1. Create a proxy, set the proxy1xxTimeout
2. Set the proxy1xxTimeout
3. Proxy the request to downstream
4. Downstream doesn't respond with 1xx
5. Container times out the branch and generates a 408

What is the expected output? What do you see instead?

I would expect this to be treated as the 408 came from a downstream element and 
as such the doBranchResponse should have been invoked.

If you setup the same test but have the downstream element actually generate 
the 408, then doBranchResponse will indeed be invoked.

What version of the product are you using? On what operating system?

Mobicents SIP Container 2.0.0.FINAL on Linux

Original issue reported on code.google.com by jo...@twilio.com on 2 Apr 2014 at 11:22

GoogleCodeExporter commented 9 years ago 
Is this for parallel or sequential ?

Original comment by jean.deruelle on 18 Apr 2014 at 12:48

GoogleCodeExporter commented 9 years ago 
nevermind reproduced it

Original comment by jean.deruelle on 18 Apr 2014 at 1:37

GoogleCodeExporter commented 9 years ago

Original comment by jean.deruelle on 18 Apr 2014 at 1:38

GoogleCodeExporter commented 9 years ago 
This issue was updated by revision f32fc273a502.

 Fix + non regression test

(cherry picked from commit 1e1422004d23e8df3c659697590d52c51f55a1bb)
(cherry picked from commit a1aef4a43d8acee64b588f7dc220044105e37eb0)

Conflicts:

    sip-servlets-test-suite/applications/proxy-sip-servlet/src/main/java/org/mobicents/servlet/sip/testsuite/ProxySipServlet.java

Original comment by jean.der...@telestax.com on 10 May 2014 at 4:41

GoogleCodeExporter commented 9 years ago

Original comment by jean.der...@telestax.com on 10 May 2014 at 4:42

GoogleCodeExporter commented 9 years ago 
This issue was updated by revision edc08859269e.

Fix + non regression test

(cherry picked from commit 1e1422004d23e8df3c659697590d52c51f55a1bb)
(cherry picked from commit a1aef4a43d8acee64b588f7dc220044105e37eb0)

Conflicts:

    sip-servlets-test-suite/applications/proxy-sip-servlet/src/main/java/org/mobicents/servlet/sip/testsuite/ProxySipServlet.java

Original comment by jean.der...@telestax.com on 23 May 2014 at 11:59

GoogleCodeExporter commented 9 years ago

Original comment by jean.der...@telestax.com on 25 Aug 2014 at 10:52