AuthenticationMethod & Config file is now case sensitive #837

Closed
opened 2026-02-20 10:04:09 -05:00 by deekerman · 1 comment
Owner

Originally created by @bakerboy448 on GitHub (Apr 11, 2023).

Originally assigned to: @mynameisbogdan on GitHub.

Is there an existing issue for this?

  • I have searched the existing open and closed issues

Current Behavior

{
  "message": "Requested value \u0027external\u0027 was not found.",
  "description": "System.ArgumentException: Requested value \u0027external\u0027 was not found.\n   at System.Enum.TryParseByName(RuntimeType enumType, ReadOnlySpan\u00601 value, Boolean ignoreCase, Boolean throwOnFailure, UInt64\u0026 result)\n   at System.Enum.TryParse(Type enumType, ReadOnlySpan\u00601 value, Boolean ignoreCase, Boolean throwOnFailure, Object\u0026 result)\n   at System.Enum.Parse(Type enumType, String value)\n   at NzbDrone.Core.Configuration.ConfigFileProvider.GetValueEnum[T](String key, T defaultValue, Boolean persist) in ./Prowlarr.Core/Configuration/ConfigFileProvider.cs:line 262\n   at NzbDrone.Core.Configuration.ConfigFileProvider.get_AuthenticationMethod() in ./Prowlarr.Core/Configuration/ConfigFileProvider.cs:line 194\n   at Prowlarr.Http.Authentication.UiAuthorizationPolicyProvider.GetPolicyAsync(String policyName) in ./Prowlarr.Http/Authentication/UiAuthorizationPolicyProvider.cs:line 31\n   at Microsoft.AspNetCore.Authorization.AuthorizationPolicy.CombineAsync(IAuthorizationPolicyProvider policyProvider, IEnumerable\u00601 authorizeData)\n   at Microsoft.AspNetCore.Authorization.AuthorizationMiddleware.Invoke(HttpContext context)\n   at Microsoft.AspNetCore.Authentication.AuthenticationMiddleware.Invoke(HttpContext context)\n   at Microsoft.AspNetCore.Diagnostics.ExceptionHandlerMiddleware.\u003CInvoke\u003Eg__Awaited|6_0(ExceptionHandlerMiddleware middleware, HttpContext context, Task task)"
}

with a

Expected Behavior

Prowlarr should migrate case sensitive values or should be case agnostic

Steps To Reproduce

N/A

Environment

- OS: agnostic
- Prowlarr: v1.3.3.3115
- Docker Install: agnostic
- Using Reverse Proxy: agnostic
- Browser: agnostic

What branch are you running?

Nightly

Trace Logs?

https://gist.github.com/bakerboy448/fa4f90e092e5e63265a2afaecb637127

Regression from github.com/Prowlarr/Prowlarr@c6ed5d65e0

Trace Logs have been provided as applicable. Reports may be closed if the required logs are not provided.

  • I have followed the steps in the wiki link above and provided the required trace logs that are relevant and show this issue.
Originally created by @bakerboy448 on GitHub (Apr 11, 2023). Originally assigned to: @mynameisbogdan on GitHub. ### Is there an existing issue for this? - [X] I have searched the existing open and closed issues ### Current Behavior ```json { "message": "Requested value \u0027external\u0027 was not found.", "description": "System.ArgumentException: Requested value \u0027external\u0027 was not found.\n at System.Enum.TryParseByName(RuntimeType enumType, ReadOnlySpan\u00601 value, Boolean ignoreCase, Boolean throwOnFailure, UInt64\u0026 result)\n at System.Enum.TryParse(Type enumType, ReadOnlySpan\u00601 value, Boolean ignoreCase, Boolean throwOnFailure, Object\u0026 result)\n at System.Enum.Parse(Type enumType, String value)\n at NzbDrone.Core.Configuration.ConfigFileProvider.GetValueEnum[T](String key, T defaultValue, Boolean persist) in ./Prowlarr.Core/Configuration/ConfigFileProvider.cs:line 262\n at NzbDrone.Core.Configuration.ConfigFileProvider.get_AuthenticationMethod() in ./Prowlarr.Core/Configuration/ConfigFileProvider.cs:line 194\n at Prowlarr.Http.Authentication.UiAuthorizationPolicyProvider.GetPolicyAsync(String policyName) in ./Prowlarr.Http/Authentication/UiAuthorizationPolicyProvider.cs:line 31\n at Microsoft.AspNetCore.Authorization.AuthorizationPolicy.CombineAsync(IAuthorizationPolicyProvider policyProvider, IEnumerable\u00601 authorizeData)\n at Microsoft.AspNetCore.Authorization.AuthorizationMiddleware.Invoke(HttpContext context)\n at Microsoft.AspNetCore.Authentication.AuthenticationMiddleware.Invoke(HttpContext context)\n at Microsoft.AspNetCore.Diagnostics.ExceptionHandlerMiddleware.\u003CInvoke\u003Eg__Awaited|6_0(ExceptionHandlerMiddleware middleware, HttpContext context, Task task)" } ``` with a ### Expected Behavior Prowlarr should migrate case sensitive values or should be case agnostic ### Steps To Reproduce N/A ### Environment ```markdown - OS: agnostic - Prowlarr: v1.3.3.3115 - Docker Install: agnostic - Using Reverse Proxy: agnostic - Browser: agnostic ``` ### What branch are you running? Nightly ### Trace Logs? https://gist.github.com/bakerboy448/fa4f90e092e5e63265a2afaecb637127 Regression from https://github.com/Prowlarr/Prowlarr/commit/c6ed5d65e03e73c7d7754f397259034c7695b497 ### Trace Logs have been provided as applicable. Reports may be closed if the required logs are not provided. - [X] I have followed the steps in the wiki link above and provided the required trace logs that are relevant and show this issue.
Author
Owner

@bakerboy448 commented on GitHub (Apr 11, 2023):

upstream possible fix https://github.com/Sonarr/Sonarr/pull/5547

@bakerboy448 commented on GitHub (Apr 11, 2023): upstream possible fix https://github.com/Sonarr/Sonarr/pull/5547
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/Prowlarr#837
No description provided.