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

Clicking "Save" in Radarr/Sonarr settings resets base URL #1904

Closed
dglava opened this issue Jul 13, 2022 · 9 comments
Closed

Clicking "Save" in Radarr/Sonarr settings resets base URL #1904

dglava opened this issue Jul 13, 2022 · 9 comments

Comments

@dglava
Copy link

dglava commented Jul 13, 2022

Describe the bug
Clicking "Save" after modifying Radarr/Sonarr settings, the base URL gets reverted back to "bazarr".

To Reproduce

  1. Enter custom base URL
  2. Click on "Save"
  3. The base URL changes to "bazarr"

Expected behavior
The base URL shouldn't change.

Screenshots
Here's a video showing the issue: https://webm.red/view/CWNe.webm (the API key was deleted just for the video, it's displaying the same behaviour with a valid API key).

Software

  • Bazarr: 1.1.0
  • Radarr: 4.1.0.6175
  • Sonarr: 3.0.8.1507
  • OS: Debian 11
@LASER-Yi LASER-Yi self-assigned this Jul 14, 2022
LASER-Yi added a commit that referenced this issue Jul 15, 2022
@LASER-Yi
Copy link
Collaborator

Sorry for the inconvenience, this issue should be fixed in the upcoming beta version

@dglava
Copy link
Author

dglava commented Jul 16, 2022

No worries, thanks for fixing it so quickly.

@motoridersd
Copy link

I'm seeing the same issue, even on v1.1.1-beta.6

@morpheus65535
Copy link
Owner

I'm seeing the same issue, even on v1.1.1-beta.6

Can you confirm how you fixed it as stated on Discord?

@motoridersd
Copy link

motoridersd commented Jul 24, 2022

I noticed the config.ini file had the base url for both Sonarr and Radarr as "//sonarr" and "//radarr" respectively. I edited the file manually to remove the extra / and restarted the service. This fixed it. (This was on Ubuntu with v1.1.1.-beta.6)

This is what the bad config looked like:

[sonarr]
ip = 127.0.0.1
port = 8989
base_url = //sonarr

I know the GUI doesn't like you leaving the / out when editing the config in the GUI, but it seems that if you add it, it's getting added twice into the config file.

I was also comparing the config.ini to an older instance I have installed on another server, and that's how I noticed that the base_url entry was incorrect

@mvanbaak
Copy link
Contributor

I also still have the issue on 1.1.1-beta.6.
I checked the config file, i do NOT have the double / for the base url: base_url = /radarr

@morpheus65535 morpheus65535 reopened this Jul 26, 2022
@morpheus65535
Copy link
Owner

@LASER-Yi we can't use the same BaseUrlModification constant for all 3 base url (Bazarr/Sonarr/Radarr) if you point it to settings.general.base_url.

@LASER-Yi
Copy link
Collaborator

Should be fixed in the upcoming beta version

@mvanbaak
Copy link
Contributor

Can confirm this one is fixed in beta.7

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants