Skip to content

Preserve base paths in server API URLs - #178

Merged
biglittlebigben merged 1 commit into
livekit:mainfrom
piotr-dydycz-whirly:fix/api-base-path
Oct 29, 2026
Merged

biglittlebigben merged 1 commit into
livekit:mainfrom
piotr-dydycz-whirly:fix/api-base-path

Conversation

@piotr-dydycz-whirly

@piotr-dydycz-whirly piotr-dydycz-whirly commented Sep 22, 2026 •

Copy link
Copy Markdown
Contributor
  • Preserve configured base paths when building server API requests
  • Use relative Retrofit endpoint paths across all services
  • Keep root URL behavior unchanged

This enables path-prefixed API deployments, including those exposed through a reverse proxy or load balancer.

Use relative Retrofit endpoint paths so API requests retain path prefixes configured in the server URL. This enables path-prefixed API deployments, including those exposed through a reverse proxy or load balancer.
@changeset-bot

changeset-bot Bot commented Sep 22, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: db66968

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

This PR includes changesets to release 1 package
Name Type
server-sdk-kotlin Minor

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

@piotr-dydycz-whirly
piotr-dydycz-whirly marked this pull request as ready for review September 22, 2026 09:35
@piotr-dydycz-whirly
piotr-dydycz-whirly requested a review from a team as a code owner September 22, 2026 09:35
@dydyczp

dydyczp commented Oct 29, 2026

Copy link
Copy Markdown

Do you see any contraindications to merging this?

@biglittlebigben
biglittlebigben merged commit eb624db into livekit:main Oct 29, 2026
3 checks passed
@davidliu davidliu mentioned this pull request Oct 29, 2026
@biglittlebigben

Copy link
Copy Markdown
Contributor

Thanks

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.

4 participants