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

Proper handling of User on SIP URI #49

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Andrepuel
Copy link

One of the added tests show how user would be incorrectly inferred because the original code supposed that everything until @ was the user part. The other test shows a username with a period being considered a hostname (including the at symbol). Changed how auth part of the SIP URI is parsed by using the characters as expressed by the SIP-URI part of the grammar on the section 25.1 of the SIP protocol RFC (RFC 3261).

One of the added tests show how user would be incorrectly inferred because the original code supposed that everything until @ was the user part.
The other test shows a username with a period being considered a hostname (including the at symbol).
Changed how auth part of the SIP URI is parsed by using the characters as expressed by the SIP-URI part of the grammar on the section 25.1 of the SIP protocol RFC (RFC 3261).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant