Skip to content

TEL-892 Adding to_user_override field - #1717

Open
genseric-ghiro wants to merge 1 commit into
mainfrom
genseric/adding-to_user_override
Open

TEL-892 Adding to_user_override field#1717
genseric-ghiro wants to merge 1 commit into
mainfrom
genseric/adding-to_user_override

Conversation

@genseric-ghiro

Copy link
Copy Markdown

Summary

  • Adding to_user_override field in CreateSIPParticipantRequest proto
  • This will allow to change the user part of the To header in outbound INVITEs.
  • The alternative, directly adding a new To header, led to two To headers in the request which created a malformed request

@genseric-ghiro
genseric-ghiro requested a review from a team August 14, 2026 21:35
@genseric-ghiro genseric-ghiro self-assigned this Aug 14, 2026
@changeset-bot

changeset-bot Bot commented Aug 14, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 58d13d1

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
github.com/livekit/protocol Patch
@livekit/protocol Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Comment thread livekit/sip_validation.go
"refer-to": true, // rfc3515
"reply-to": true,
"to": true, // Set the user part with CreateSIPParticipantRequest.to_user_override
"via": true, // Multiple Via are legal, but a caller's own breaks response routing

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Might help to do a quick check in DD to see if anyone is actually passing these in as part of CreateSipParticipant.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants