New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add credentials: "include" back to Radarr and Sonarr #291
Add credentials: "include" back to Radarr and Sonarr #291
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch @mcclurec, sorry about that. Thanks for fixing this.
I think this reintroduced the CORS errors that were previously resolved. See #250 |
Yep! It worked for me until the last update :/ |
Damned, sorry for that. |
I'm running Homer, Sonarr, Radarr in separate docker containers just on localhost, with their own assigned ports. i.e.
No reverse proxy or auth layers for these services, no headers added to Sonarr/Radarr. |
🤔 Could you guys paste some error messages you're getting in browser? Are you seeing something like |
Sonarr doesn't specifically set that header, so that may be the root issue |
The error is:
|
Thanks. I'm whipping up a PR to make the |
@bastienwirtz Opened PR 301 to fix |
Description
credentials: "include"
was mistakenly removed from the Sonarr and Radarr components when migrating the API key from a header to a query param. This PR adds thefetch
option back to support set ups with SSO providers like Authelia and others. Similar to PR-258Fixes # (issue)
Possibly #251. That assumes they're running an SSO like Authelia, and the JSON parse data is because it's getting the login page HTML instead of the API JSON. Can't tell without more info on the Issue.
Type of change
Checklist:
config.yml
file