Lay foundations for plugin-specific unique session ID in SIP and NoSIP plugins#3607
Merged
Lay foundations for plugin-specific unique session ID in SIP and NoSIP plugins#3607
Conversation
lminiero
added a commit
that referenced
this pull request
Dec 2, 2025
mwalbeck
pushed a commit
to mwalbeck/docker-janus-gateway
that referenced
this pull request
Feb 14, 2026
This PR contains the following updates: | Package | Update | Change | |---|---|---| | [meetecho/janus-gateway](https://github.com/meetecho/janus-gateway) | minor | ` v1.3.3` -> `v1.4.0` | --- ### Release Notes <details> <summary>meetecho/janus-gateway (meetecho/janus-gateway)</summary> ### [`v1.4.0`](https://github.com/meetecho/janus-gateway/blob/HEAD/CHANGELOG.md#v140---2026-02-06) [Compare Source](meetecho/janus-gateway@v1.3.3...v1.4.0) - Generate a pkg-config .pc file when installing Janus \[[PR-3596](meetecho/janus-gateway#3596)] - Implemented compound RTCP reports (thanks [@​addisonpolcyn](https://github.com/addisonpolcyn)!) \[[PR-3585](meetecho/janus-gateway#3585)] - Fixed when to negotiate the rtcp-fb transport-cc SDP attribute \[[PR-3595](meetecho/janus-gateway#3595)] - Strip some WebRTC-specific SDP attributes in SIP and NoSIP plugins - Added RTP forwarders support to SIP and NoSIP plugins (thanks [@​WebTrit](https://github.com/WebTrit)!) \[[PR-3583](meetecho/janus-gateway#3583)] - Fixed subscription renewal possibly going to the wrong host in the SIP plugin (thanks [@​digiboridev](https://github.com/digiboridev)!) \[[PR-3602](meetecho/janus-gateway#3602)] - Laid foundations for plugin-specific unique session ID in SIP and NoSIP plugins \[[PR-3607](meetecho/janus-gateway#3607)] - Fixed 'use-after-free' in SIP session helpers (thanks [@​oleksandr-mihal-zultys](https://github.com/oleksandr-mihal-zultys)!) \[[PR-3613](meetecho/janus-gateway#3613)] - Fixed SIP plugin not offering SRTP in response to offerless INVITEs \[[PR-3618](meetecho/janus-gateway#3618)] - Added new janus-pp-rec option to ignore RTP headers for timing purposes - Other smaller fixes and improvements (thanks to all who contributed pull requests and reported issues!) </details> --- ### Configuration 📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiI0MS4xNDYuMCIsInVwZGF0ZWRJblZlciI6IjQxLjE0Ni4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6W119--> Reviewed-on: https://git.walbeck.it/walbeck-it/docker-janus-gateway/pulls/202 Co-authored-by: renovate-bot <bot@walbeck.it> Co-committed-by: renovate-bot <bot@walbeck.it>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The SIP and NoSIP plugins were so far lacking a way to uniquely identify a session: in the SIP plugin, for instance, we could in theory use the Call-ID, but that property actually identifies a call, not a specific participant in a call: in case both caller and callee are handled by the same Janus instance, or different calls end up using the same Call-ID by accident, this would cause ambiguity. As a result, there was no unique ID we could use to univocally identify a specific participant in the SIP and NoSIP plugins, which could be a very useful feature to have, e.g., to add some synchronous management requests to the plugin. This came up in #3604 (review), for example, but another potential use case would be the RTP forwarders support in #3583, where having these unique IDs would be quite useful too.
This PR adds this functionality, by simply introducing these new IDs but not using them for anything. In the SIP plugin, they're returned as
unique_idas part of theregisteringandregisteredevents, while in the NoSIP plugin they're returned with theprocessedandgeneratedevents: these IDs won't change as long as the Janus handle responsible for that session is alive.Planning to merge soon as this doesn't have an impact on existing functionality (they don't do anything except return a new ID that's not used for anything right now), but it will be useful for future things.