[PR #3084] [MERGED] Fix #3083: Set the Access-Control-Allow-Origin header to the request … #9338

Closed
opened 2026-02-07 06:00:29 +03:00 by OVERLORD · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/3084
Author: @fhriley
Created: 5/9/2020
Status: Merged
Merged: 5/15/2020
Merged by: @JustAMan

Base: masterHead: fix_allow_origin


📝 Commits (4)

  • 3cb6fd8 Fix #3083: Set the Access-Control-Allow-Origin header to the request origin/host header if possible
  • c70c589 Update Emby.Server.Implementations/HttpServer/HttpListenerHost.cs from review
  • 6990af8 Use simpler dictionary iterator.
  • 9ee10d2 Rename function

📊 Changes

3 files changed (+54 additions, -8 deletions)

View changed files

📝 Emby.Server.Implementations/HttpServer/HttpListenerHost.cs (+31 -4)
📝 Emby.Server.Implementations/HttpServer/ResponseFilter.cs (+15 -4)
📝 MediaBrowser.Controller/Net/IHttpServer.cs (+8 -0)

📄 Description

…origin/host header if possible

Note that to fully fix this requires a couple simple changes in jellyfin-web as well.


🔄 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/3084 **Author:** [@fhriley](https://github.com/fhriley) **Created:** 5/9/2020 **Status:** ✅ Merged **Merged:** 5/15/2020 **Merged by:** [@JustAMan](https://github.com/JustAMan) **Base:** `master` ← **Head:** `fix_allow_origin` --- ### 📝 Commits (4) - [`3cb6fd8`](https://github.com/jellyfin/jellyfin/commit/3cb6fd8a2754837c787213c008ad84a973eb7cab) Fix #3083: Set the Access-Control-Allow-Origin header to the request origin/host header if possible - [`c70c589`](https://github.com/jellyfin/jellyfin/commit/c70c58923667a3c626b4112f783f755f91442d0b) Update Emby.Server.Implementations/HttpServer/HttpListenerHost.cs from review - [`6990af8`](https://github.com/jellyfin/jellyfin/commit/6990af811ad65816a0534f75e889dc9c22632876) Use simpler dictionary iterator. - [`9ee10d2`](https://github.com/jellyfin/jellyfin/commit/9ee10d22c8ccbeb9eb4112b1a9f520d5ed998013) Rename function ### 📊 Changes **3 files changed** (+54 additions, -8 deletions) <details> <summary>View changed files</summary> 📝 `Emby.Server.Implementations/HttpServer/HttpListenerHost.cs` (+31 -4) 📝 `Emby.Server.Implementations/HttpServer/ResponseFilter.cs` (+15 -4) 📝 `MediaBrowser.Controller/Net/IHttpServer.cs` (+8 -0) </details> ### 📄 Description …origin/host header if possible Note that to fully fix this [requires a couple simple changes in jellyfin-web as well](https://github.com/jellyfin/jellyfin-web/pull/1211). --- <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:00:29 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/jellyfin#9338