search

amitpahune / jain-sip

Automatically exported from code.google.com/p/jain-sip
0 stars 0 forks source link

No Route header on UAC BYE when 200 OK contains a record route #66

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
What steps will reproduce the problem?
1. answer a call with a 200 OK with record route headers
2. end call

What is the expected output? What do you see instead?
BYE sent by the UAC does not contain any route header. 

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

Please provide any additional information below.
Probably realated also with issue 65.

Original issue reported on code.google.com by sergio.g...@gmail.com on 18 Feb 2013 at 11:37

GoogleCodeExporter commented 8 years ago

Original comment by laurent....@gmail.com on 19 Feb 2013 at 5:21

GoogleCodeExporter commented 8 years ago

Original comment by laurent....@gmail.com on 20 Feb 2013 at 8:47

GoogleCodeExporter commented 8 years ago 
Hello Sergio, thanks for the feedback, can you verified the fix?

Original comment by laurent....@gmail.com on 20 Feb 2013 at 8:49

GoogleCodeExporter commented 8 years ago 
Still same problem logs attached

Original comment by sergio.g...@gmail.com on 20 Feb 2013 at 4:11

Attachments:

GoogleCodeExporter commented 8 years ago

Original comment by laurent....@gmail.com on 20 Feb 2013 at 4:32

GoogleCodeExporter commented 8 years ago 
Fixed, additional bugs have been found:
1- in the JAIN SIP JS stack: addRoute() was not called in 
SipProviderImpl.prototype.getNewServerTransaction()
2: Bad JAIN SIP API usage to create BYE request, works better by using 
SIPDialog.createRequest("BYE").
Has been tested (BYE from caller and callee) with Kamailo SIP Server which 
inject Record-Route header.

Original comment by laurent....@gmail.com on 22 Feb 2013 at 3:22

GoogleCodeExporter commented 8 years ago

Original comment by laurent....@gmail.com on 22 Feb 2013 at 3:22

GoogleCodeExporter commented 8 years ago 
I am using the old MobicentsWebRTCPhone.js and just updating the jain-sip.js 
didn't solved the issue. What changes are needed in MobicentsWebRTCPhone.js 
also?

Original comment by sergio.g...@gmail.com on 23 Feb 2013 at 12:56

GoogleCodeExporter commented 8 years ago 
Changed the xxxRequest.createByeRequest(true) calls to 
xxxDialog.CreateRequest("BYE") on MobicentsWebRTCPhone.js and it is working 
correctly now.

Thnxs!!

Original comment by sergio.g...@gmail.com on 24 Feb 2013 at 2:53