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

SIP plugin: SIP MESSAGE out of dialog #2616

Merged
merged 5 commits into from
Apr 7, 2021
Merged

SIP plugin: SIP MESSAGE out of dialog #2616

merged 5 commits into from
Apr 7, 2021

Conversation

ihusejnovic
Copy link
Contributor

Hello,

In this pull request, I added the option to send a SIP MESSAGE "out of dialog". The message will be sent out of dialog if you pass the new parameter called "uri" with the SIP URI of the peer. If the "uri" parameter is not set, the message will be sent in the active dialog as it is in the current implementation. In order to send a message out of dialog, the user must be registered.

I tested it with the Kamailio and it works fine.

Copy link
Member

@lminiero lminiero left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Added a couple of notes inline.

plugins/janus_sip.c Outdated Show resolved Hide resolved
plugins/janus_sip.c Outdated Show resolved Hide resolved
Copy link
Member

@lminiero lminiero left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! I've added a couple of notes inline.

plugins/janus_sip.c Show resolved Hide resolved
plugins/janus_sip.c Outdated Show resolved Hide resolved
@ihusejnovic
Copy link
Contributor Author

@lminiero I fixed all the things you suggested in the comments. Let me know if there is anything else that needs to be done. Thank you.

@lminiero
Copy link
Member

lminiero commented Apr 7, 2021

Thanks, looks good to me now! Merging 👍

@lminiero lminiero merged commit f79b41c into meetecho:master Apr 7, 2021
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.

None yet

2 participants