starred/jellyfin
1
0
Fork 0
mirror of https://github.com/jellyfin/jellyfin.git synced 2026-07-16 05:43:54 +03:00
Code Issues 822 Packages Projects Releases 17 Wiki Activity

[PR #4035] [MERGED] Fix apidoc routes with base url #9834

New Issue
Closed
opened 2026-02-07 06:09:20 +03:00 by OVERLORD · 0 comments
OVERLORD commented 2026-02-07 06:09:20 +03:00
Owner
Copy Link

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/4035
Author: @crobibero
Created: 9/1/2020
Status: ✅ Merged
Merged: 9/4/2020
Merged by: @joshuaboniface

Base: master ← Head: api-doc-base-url

📝 Commits (9)

  • 0b38ac9 Fix apidoc routes with base url
  • 1feee6f Properly host static files and set base url
  • 21a5692 Merge remote-tracking branch 'upstream/master' into api-doc-base-url
  • 7504f06 fix merge
  • 4e52fe1 Wrap application in baseurl
  • ca6dfd7 move metrics to end of pipeline
  • c404660 Remove double listener
  • 9f97c47 Merge remote-tracking branch 'upstream/master' into api-doc-base-url
  • 021e218 Remove MvcRoutePrefix

📊 Changes

9 files changed (+75 additions, -218 deletions)

View changed files

📝 Jellyfin.Api/Controllers/DashboardController.cs (+2 -100)
➖ Jellyfin.Api/MvcRoutePrefix.cs (+0 -56)
📝 Jellyfin.Server/Extensions/ApiApplicationBuilderExtensions.cs (+10 -3)
📝 Jellyfin.Server/Extensions/ApiServiceCollectionExtensions.cs (+2 -7)
📝 Jellyfin.Server/Middleware/BaseUrlRedirectionMiddleware.cs (+1 -5)
📝 Jellyfin.Server/Program.cs (+1 -1)
📝 Jellyfin.Server/Startup.cs (+58 -36)
📝 MediaBrowser.Common/Configuration/IApplicationPaths.cs (+1 -4)
📝 MediaBrowser.Model/Configuration/ServerConfiguration.cs (+0 -6)

📄 Description

Removes the baseUrl from the openapi endpoints if a baseurl is set.
#4029

image

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/jellyfin/jellyfin/pull/4035 **Author:** [@crobibero](https://github.com/crobibero) **Created:** 9/1/2020 **Status:** ✅ Merged **Merged:** 9/4/2020 **Merged by:** [@joshuaboniface](https://github.com/joshuaboniface) **Base:** `master` ← **Head:** `api-doc-base-url` --- ### 📝 Commits (9) - [`0b38ac9`](https://github.com/jellyfin/jellyfin/commit/0b38ac9a8a141149dfe5531c4f96b4d462939aaa) Fix apidoc routes with base url - [`1feee6f`](https://github.com/jellyfin/jellyfin/commit/1feee6f95e00cf579ab16c7ca004947534545d9b) Properly host static files and set base url - [`21a5692`](https://github.com/jellyfin/jellyfin/commit/21a5692626fa7389b4b37a91050496b2fe7a0260) Merge remote-tracking branch 'upstream/master' into api-doc-base-url - [`7504f06`](https://github.com/jellyfin/jellyfin/commit/7504f067efd304e5563935ff49bce11ceeb87573) fix merge - [`4e52fe1`](https://github.com/jellyfin/jellyfin/commit/4e52fe106088a4e425652831cbba0c373ddbc14a) Wrap application in baseurl - [`ca6dfd7`](https://github.com/jellyfin/jellyfin/commit/ca6dfd7c45a8a95cf66b8965053cf91a25f40673) move metrics to end of pipeline - [`c404660`](https://github.com/jellyfin/jellyfin/commit/c404660f67e62db6a58faea2be22609ce8f54988) Remove double listener - [`9f97c47`](https://github.com/jellyfin/jellyfin/commit/9f97c47936587369603caf565d6da24461274c32) Merge remote-tracking branch 'upstream/master' into api-doc-base-url - [`021e218`](https://github.com/jellyfin/jellyfin/commit/021e218f754af8a1cf7f35e2a1d8286e813884e1) Remove MvcRoutePrefix ### 📊 Changes **9 files changed** (+75 additions, -218 deletions) <details> <summary>View changed files</summary> 📝 `Jellyfin.Api/Controllers/DashboardController.cs` (+2 -100) ➖ `Jellyfin.Api/MvcRoutePrefix.cs` (+0 -56) 📝 `Jellyfin.Server/Extensions/ApiApplicationBuilderExtensions.cs` (+10 -3) 📝 `Jellyfin.Server/Extensions/ApiServiceCollectionExtensions.cs` (+2 -7) 📝 `Jellyfin.Server/Middleware/BaseUrlRedirectionMiddleware.cs` (+1 -5) 📝 `Jellyfin.Server/Program.cs` (+1 -1) 📝 `Jellyfin.Server/Startup.cs` (+58 -36) 📝 `MediaBrowser.Common/Configuration/IApplicationPaths.cs` (+1 -4) 📝 `MediaBrowser.Model/Configuration/ServerConfiguration.cs` (+0 -6) </details> ### 📄 Description Removes the baseUrl from the openapi endpoints if a baseurl is set. #4029 ![image](https://user-images.githubusercontent.com/24963659/91893989-33473700-ec52-11ea-9dc4-cb2ae3b2fea5.png) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
OVERLORD added the pull-request label 2026-02-07 06:09:20 +03:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
EFjellyfin.db
area:database
awaiting-feedback
backend
blocked
breaking change: web api
bug
build
ci
confirmed
discussion needed
dotnet future
downstream
duplicate
enhancement
feature
future
github-actions
good first issue
hdr
help wanted
invalid
investigation
librarydb
live-tv
lyrics
media playback
music
needs testing
nuget
performance
platform
pull-request
Mirrored from GitHub Pull Request
 question
regression
release critical
requires-web
roadmap
security
security
stale
support
syncplay
ui & ux
upstream
wontfix
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
OVERLORD
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/jellyfin#9834
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?