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

Make the 'instance_id' UA configuration parameter public #49

Closed
ibc opened this issue Jan 23, 2013 · 14 comments
Closed

Make the 'instance_id' UA configuration parameter public #49

ibc opened this issue Jan 23, 2013 · 14 comments
Assignees
Milestone

Comments

@ibc
Copy link
Member

ibc commented Jan 23, 2013

Be able to set a custom URN for the +sip.instance header param of the REGISTER. Currently a random value like "urn:uuid:9374a213-94d6-40b3-8016-81a87e585e5b" is generated. JsSIP must allow using a provided value.

urn:  "uuid:123-123-123-123"

So the param would become:

+sip.instance="<urn:uuid:123-123-123-123>"
@ghost ghost assigned jmillan Jan 23, 2013
@jmillan
Copy link
Member

jmillan commented Jan 23, 2013

Yeah

@saghul
Copy link
Contributor

saghul commented Jan 23, 2013

Can we call that "instance_id" ? The implementation uses a URN, but as a concept it expresses an instance identifier: https://tools.ietf.org/html/rfc5626#section-4.1

Also, should jsSIP verify that the given instance id is a valid urn?

@jmillan
Copy link
Member

jmillan commented Jan 23, 2013

This parameter already exists and its name is instance_id ;-). It is a non public configuration parameter right now.

Of course, it will verify the urn syntax.

@saghul
Copy link
Contributor

saghul commented Jan 23, 2013

José Luis Millán wrote:

This parameter already exists and its name is |instance_id| ;-). It is
a non public configuration parameter right now.

Of course, it will verify the urn syntax.


Reply to this email directly or view it on GitHub
#49 (comment).

Great! I hope you don't planto change it to "urn" ;-)

@jmillan
Copy link
Member

jmillan commented Jan 23, 2013

Not at all. It is The instance_id ;-)

@ibc
Copy link
Member Author

ibc commented Jan 23, 2013

instance_id is a proper name, but +sip.instance Contact header param requires a URN so validation of the field is required (otherwise parsing errors could occur somewhere.

Could you please write an example of such a parameter usage?

@jmillan
Copy link
Member

jmillan commented Jan 23, 2013

It is already implemented internally. I will make the parameter public so it will be validated against the proper syntax as any configuration parameter does.

The following can be seen from a running UA instance.

coolPhone.configuration.instance_id -> 1758eab0-806c-40bb-a88d-710b99503965

The Contact header in a Register request is generated as it needs to:

Contact: <sip:jmillan@kusxwgv55vmw.invalid;transport=ws>;reg-id=1;+sip.instance="<urn:uuid:1758eab0-806c-40bb-a88d-710b99503965>";expires=600

@ibc
Copy link
Member Author

ibc commented Jan 23, 2013

Please, properly document in the API that the value must not contain "urn:uuid:" nor "<" and ">".

Theorically the user provided value should be an entire URN (i.e.: "urn:uuid:xxxx-xxxx-xxxx-xxxx") but we can require just the "xxxx-xxxx-xxxx" fragment if properly documented.

@jmillan
Copy link
Member

jmillan commented Jan 23, 2013

Yes. First I will take it to the public API, and the document.

@ibc
Copy link
Member Author

ibc commented Mar 19, 2013

Updates to this issue please? It is marked for 0.3.x but AFAIK it is not. If so better leaving it for 0.4.x.

@jmillan
Copy link
Member

jmillan commented Mar 19, 2013

No updates at all. Well set to 0.4.x

@jmillan
Copy link
Member

jmillan commented Jun 25, 2013

New UA configuration parameter: 'instance_id':

764dc90

@jmillan jmillan closed this as completed Jun 25, 2013
gavllew pushed a commit to crocodilertc/JsSIP that referenced this issue Aug 16, 2013
(cherry picked from commit 764dc90)

Conflicts:
	src/Grammar/dist/Grammar.min.js
	src/Grammar/src/Grammar.pegjs
	src/UA.js
@walidsynclio
Copy link

HI,
I tried to set instance_id but no luck.
coolPhone.instance_id says undefined.
Even if I add instance_id to configuration it does not show up in the sip request packet. it has some other vlue in sip request packet.
Thanks

@jmillan
Copy link
Member

jmillan commented Nov 19, 2013

Hi @walidsynclio,

The instance_id is a new configuratin parameter in the 'devel' branch. It is not in the current 'master' branch. Are you using the 'devel'?

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

No branches or pull requests

4 participants