mirror of
https://github.com/RandomNinjaAtk/arr-scripts.git
synced 2026-03-02 22:57:35 -05:00
[BUG] - Lidarr - Regex characters in album name cause album to be not found #45
Labels
No labels
Needs Triage
Not Reproducible
Upstream Issue
User Error
bug
documentation
enhancement
good first issue
help wanted
invalid
lidarr
lidarr
question
radarr
readarr
sabnzbd
sonarr
synology (host)
unraid (host)
waiting for logs
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/arr-scripts#45
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @calebj on GitHub (Aug 27, 2023).
Originally assigned to: @RandomNinjaAtk on GitHub.
Application
Lidarr
Host platform
Docker on Linux
Script
Lidarr audio script
Script Version
2.12
Describe the bug
Regex characters in album title are not escaped, causing
jqto error and album to be marked not found.To Reproduce
Steps to reproduce the behavior:
+)Expected behavior
Special characters in an album name are not treated as regex
Logs/Screenshots
Additional context
For what it's worth, the album is titled "Classics.exe" (no +) in Deezer, which might cause issues by itself, but it does appear in search results. In this case, however, jq simply doesn't like the invalid pattern being given to test().
The problem is happening here:
github.com/RandomNinjaAtk/arr-scripts@9e2b4f0774/lidarr/Audio.service.bash (L1452-L1454)Manual reproduction:
outputs
@RandomNinjaAtk commented on GitHub (Aug 28, 2023):
This is a good find, although I'm not sure what the best course of action would be to fix it... If you have any thoughts or ideas, I'm open to suggestions...
@RandomNinjaAtk commented on GitHub (Sep 7, 2023):
@calebj
Should be fixed in latest version...