Skip to content
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

Fix *arr url creation #788

Merged
merged 6 commits into from
Oct 18, 2024
Merged

Fix *arr url creation #788

merged 6 commits into from
Oct 18, 2024

Conversation

firrae
Copy link
Contributor

@firrae firrae commented Oct 18, 2024

Seems like at some point normaliseurl changed or someone edited the / off the next line causing all attempts to verify enhanced apps were set up correctly to fail. This should solve that by using the second input of the normaliseurl function.

Radarr was using normaliseurl (which strips the trailing `/`) without passing true to re-add said `/` as needed.
Sonarr as well was using normaliseurl (which strips the trailing `/`) without passing true to re-add said `/` as needed.
Readarr  as well was using normaliseurl (which strips the trailing `/`) without passing true to re-add said `/` as needed.
Lidarr as well was using normaliseurl (which strips the trailing `/`) without passing true to re-add said `/` as needed.
@firrae
Copy link
Contributor Author

firrae commented Oct 18, 2024

This should fix linuxserver/Heimdall#1266.

Bazarr as well was using normaliseurl (which strips the trailing `/`) without passing true to re-add said `/` as needed.
Prowlarr as well was using normaliseurl (which strips the trailing `/`) without passing true to re-add said `/` as needed.
@mvdkleijn mvdkleijn merged commit 633c9f1 into linuxserver:master Oct 18, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

2 participants