diff options
| author | Joshua M. Boniface <joshua@boniface.me> | 2025-04-05 14:44:42 -0400 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2025-04-05 14:44:42 -0400 |
| commit | 2c9c9f591dce26a420ded044a7c9a17ea373c5c9 (patch) | |
| tree | 7de3e16abf66791c6ff0a384a3ad71a98b56e0de | |
| parent | 7d705249cac50ab878f41b180b822488158c5900 (diff) | |
| parent | a2ef0e4abec6ae78bd43089c46064e1e0943d8a5 (diff) | |
Merge commit from fork
Fix not trusting all sources for forward headers if none are configured
| -rw-r--r-- | Jellyfin.Server/Extensions/ApiServiceCollectionExtensions.cs | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/Jellyfin.Server/Extensions/ApiServiceCollectionExtensions.cs b/Jellyfin.Server/Extensions/ApiServiceCollectionExtensions.cs index c6c3f21fe..b04e55baa 100644 --- a/Jellyfin.Server/Extensions/ApiServiceCollectionExtensions.cs +++ b/Jellyfin.Server/Extensions/ApiServiceCollectionExtensions.cs @@ -119,15 +119,15 @@ namespace Jellyfin.Server.Extensions // https://github.com/dotnet/aspnetcore/blob/master/src/Middleware/HttpOverrides/src/ForwardedHeadersMiddleware.cs // Enable debug logging on Microsoft.AspNetCore.HttpOverrides.ForwardedHeadersMiddleware to help investigate issues. - options.ForwardedHeaders = ForwardedHeaders.XForwardedFor | ForwardedHeaders.XForwardedProto | ForwardedHeaders.XForwardedHost; - if (config.KnownProxies.Length == 0) { + options.ForwardedHeaders = ForwardedHeaders.None; options.KnownNetworks.Clear(); options.KnownProxies.Clear(); } else { + options.ForwardedHeaders = ForwardedHeaders.XForwardedFor | ForwardedHeaders.XForwardedProto | ForwardedHeaders.XForwardedHost; AddProxyAddresses(config, config.KnownProxies, options); } |
