Skip to content
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

createBranch for a new SIP Request #269

Closed
GoogleCodeExporter opened this issue Apr 11, 2016 · 3 comments
Closed

createBranch for a new SIP Request #269

GoogleCodeExporter opened this issue Apr 11, 2016 · 3 comments

Comments

@GoogleCodeExporter
Copy link

What steps will reproduce the problem?
1.High possibility of simultaneous SIP requests creating the same branch 
parameter as it uses system.nanotime()

What is the expected output? What do you see instead?
The branch parameter is expected to be UNIQUE for each request.

What version of the product are you using? On what operating system?
MSS 2.0.0.FINAL

Please provide any additional information below.
I am using JBOSS 7 with Mobicents 2.0.0 for the application we are running. We 
have some sort of a heart beat (pinger) mechanism to check on the health of 
ICSCF from our application. To be more clear, there are 2 ICSCFs Primary and 
Secondary and we keep sending OPTIONS every configured seconds (10). The 
OPTIONS pings are sent successively to both Primary and Secondary and based on 
the failure we consider one of them as down and make sure the call is sent 
through the working ICSCF. It looks like the OPTIONS request created for 
Primary and Secondary has the same branch parameter since "System.nanoTime()" 
is what is being used while creating Branch for new requests. On searching 
about this, I do see a fix made in the link 
"https://code.google.com/p/mobicents/issues/detail?id=2278" using 
"ApplicationRoutingHeaderComposer.randomString()". I am assuming that is a 
correct fix but just not sure why it wasn't merged on future versions. Please 
comment and let me know if otherwise.

Original issue reported on code.google.com by Prakoo...@gmail.com on 2 May 2014 at 7:36

@GoogleCodeExporter
Copy link
Author

Original comment by jean.deruelle on 2 May 2014 at 8:09

  • Changed state: Accepted
  • Added labels: TelScale-7.0.2.GA

@GoogleCodeExporter
Copy link
Author

This issue was closed by revision 25e5d7f78e16.

Original comment by jean.der...@telestax.com on 2 May 2014 at 9:03

  • Changed state: Fixed

@GoogleCodeExporter
Copy link
Author

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

  • Added labels: MSS-2.1.0.FINAL
  • Removed labels: MSS-3.0.0.FINAL

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant