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

curl error calling url in Sonarr - there is a slash missing in the URL #57

Closed
jrcalvert opened this issue Oct 14, 2023 · 8 comments · Fixed by #58
Closed

curl error calling url in Sonarr - there is a slash missing in the URL #57

jrcalvert opened this issue Oct 14, 2023 · 8 comments · Fixed by #58
Assignees
Labels
bug Something isn't working

Comments

@jrcalvert
Copy link

Hello,
I am running Sonarr v4 and are seeing these errors when the script is run - there is a missing "/" after the port number and the Url Base. This is under docker, the LinuxServer.io image ghcr.io/linuxserver/sonarr:develop with striptracks installed as a "mod"

[Error] striptracks-eng.sh: Error|[3] curl error when calling: "http://localhost:8989sonarr/api/v3/command" with data {"name":"RescanSeries","seriesId":xxxx}
[Error] striptracks-eng.sh: Error|Web server returned:
[Error] striptracks-eng.sh: Error|The 'RescanSeries' API with seriesId xxxx failed.
[Error] striptracks-eng-jpn.sh: Error|[3] curl error when calling: "http://localhost:8989sonarr/api/v3/system/status"
[Error] striptracks-eng-jpn.sh: Error|Web server returned:
[Error] striptracks-eng-jpn.sh: Error|[3] curl error when calling: "http://localhost:8989sonarr/api/v3/languageprofile"
[Error] striptracks-eng-jpn.sh: Error|Web server returned:
[Error] striptracks-eng-jpn.sh: Warn|Unable to retrieve language codes from 'languageprofile' API (curl error or returned a null name).

Thanks

@TheCaptain989 TheCaptain989 self-assigned this Oct 14, 2023
@TheCaptain989 TheCaptain989 added the bug Something isn't working label Oct 14, 2023
@TheCaptain989
Copy link
Owner

Hmmm. I haven't tested it with v4 yet, but it looks like it could be a simple fix.

I'll investigate.

P.S. - Thanks for the logs!

TheCaptain989 added a commit that referenced this issue Oct 14, 2023
- Addresses #57
@TheCaptain989
Copy link
Owner

TheCaptain989 commented Oct 14, 2023

Okay, I think I fixed this. Could you please test by using the develop version by setting your DOCKER_MODS variable to:
DOCKER_MODS=thecaptain989/radarr-striptracks:develop ?

@jrcalvert
Copy link
Author

jrcalvert commented Oct 15, 2023 via email

@jrcalvert
Copy link
Author

Hello. I am now seeing this error - Can that be ignored?
"[Error] striptracks-eng-jpn.sh: Warn|The 'episode' API with id 106558 returned a false hasFile."

@TheCaptain989
Copy link
Owner

TheCaptain989 commented Oct 18, 2023

That used to be normal in earlier versions of the script. That would happen in Radarr when the file extension changed (like .avi to .mkv). I wouldn't expect it now with the new way I'm handling things. Did the remuxed file appear in Sonarr after script completion?

@TheCaptain989
Copy link
Owner

Either way, I think this signals progress! Could you check the striptracks.txt log file for the actual script logs?

@jrcalvert
Copy link
Author

jrcalvert commented Oct 20, 2023 via email

@TheCaptain989
Copy link
Owner

Great news. I'll try to create a PR this weekend.

TheCaptain989 added a commit that referenced this issue Oct 21, 2023
- Fix for populated urlbase (Addresses #57)
- Added loop and wait on some functions when database is locked
- Fixed for multiline message on API error
- Improved mkvmerge error handling
TheCaptain989 added a commit to TheCaptain989/docker-mods that referenced this issue Dec 17, 2023
- Added command-line option for log file
- Added checks for log file and path
- Added loop and wait on some functions when database is locked
- Changed temporary file name generation to avoid possible race condition with Radarr/Sonarr refresh
- Shows temp filename in debug output
- Fix for TheCaptain989/radarr-striptracks#61
- Fix for populated urlbase (Addresses TheCaptain989/radarr-striptracks#57)
- Fix for multiline message on API error
- Improved mkvmerge error handling
- Edited README and SECURITY
- Update help
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants