RandomNinjaAtk / docker-lidarr-extended

lidarr-extended :: Lidarr application packaged with multiple scripts to provide additional functionality
GNU General Public License v3.0
275 stars 24 forks source link

[Bug]: cannot auto add artists from deezer lists and deezer extended audio download fails #120

Closed BackedUpBooty closed 1 year ago

BackedUpBooty commented 1 year ago

Describe the bug No 'Top 10' artists from deezer (the only list enabled) can be auto-added, and Extended Audio Download fails.

To Reproduce Steps to reproduce the behavior:

  1. Docker-compose.yml has the following env variables:
    - arlToken=$ARLTOKEN
    - audioLyricType=both
    - dlClientSource=deezer
    - addDeezerTopArtists=true
    - addDeezerTopAlbumArtists=false
    - addDeezerTopTrackArtists=false
    - topLimit=10
    - addRelatedArtists=false
  2. Spin up container
  3. Watch logs until error happens (on all 10 artists)
  4. Continue to watch logs until Extended Audio error occurs
  5. Try updating the arl token as suggested, no change to Extended Audio

Expected behavior Top 10 artists from Deezer are added to lidarr, and Extended Audio doesn't error out

Log Data https://gist.github.com/BackedUpBooty/8c683c06bd05f47dacd90262d22242c0 (deezer top 10 artist add error) https://gist.github.com/BackedUpBooty/3eb20e66cb2a9aa2d739bdc098b7dfaf (Extended Audio Download failure)

Docker Info:

Additional context I have a free US deezer account, but I am not currently in the US, so my lidarr-extended container network is going through another VPN container. I've confirmed however that the IP of that container is in the US:

root@6ae324623d75:/config# curl ipinfo.io/json
{
  "ip": "xx.xx.xxx.xx", [redacted]
  "city": "San Jose",
  "region": "California",
  "country": "US",
  "loc": "37.3394,-121.8950",
  "org": "AS35913 DediPath",
  "postal": "95103",
  "timezone": "America/Los_Angeles",
  "readme": "https://ipinfo.io/missingauth"
RandomNinjaAtk commented 1 year ago

Not able to replicate this issue:

2022-11-07 11:42:56 :: Extended Audio :: 1.0.271 :: Finding Top Artists...
2022-11-07 11:42:56 :: Extended Audio :: 1.0.271 :: 1 Top Artists Found...
2022-11-07 11:42:57 :: Extended Audio :: 1.0.271 :: 1 of 1 :: Kanye West :: Searching Musicbrainz for Deezer artist id (230)
2022-11-07 11:43:02 :: Extended Audio :: 1.0.271 :: 1 of 1 :: Kanye West :: Adding Kanye West to Lidarr (164f0d73-1234-4e2c-8743-d77bf2191051)...
2022-11-07 11:43:02 :: Extended Audio :: 1.0.271 :: STATUS :: LIDARR BUSY :: Pausing/waiting for all active Lidarr tasks to end...
2022-11-07 11:44:05 :: Extended Audio :: 1.0.271 :: FINDING MISSING ALBUMS :: sorted by date

No errors and working as expected in my system/configuration.

BackedUpBooty commented 1 year ago

Not sure why this is an invalid report. The error is clearly there in the lidarr API on my system. Does Deezer need to be a paid account?

RandomNinjaAtk commented 1 year ago

As you can see from my testing it works and if I can’t reproduce it, there is no hope for me to help fix it. It honestly doesn’t require an account at all to work, accounts are only required for download execution.

So if your experiencing the issue, it’s likely something unique to your particular setup/configuration.

Edit: Also your download error indicates an issue with your ARL token, you may have just miss configured it, added spaces or possibly something else.

Edit 2: You may want to setup a 2nd test instance that is a clean install to see if the issue still persists.

RandomNinjaAtk commented 1 year ago

So I ran into a similar bug, but was my own doing with recent changes.

After reading yours more closely, what is the configured root folders in your lidarr setup? Are you using more than one? What are the full paths?

It looks like its barfing due to your root folder/path configuration.

BackedUpBooty commented 1 year ago

Hey thanks for circling back to this.

My docker-compose volumes are mapped like this which essentially matches my Sonarr/Radarr setups:

volumes:
      - ./lidarr:/config
      - $MEDIADIR:/downloads-lidarr-extended
      - $MEDIADIR:/media
      - $MEDIADIR/media/music videos:/music-videos

There are no folder mappings required in the lidarr gui to make sure downloads are found and can be renamed/moved etc.

MEDIADIR=/volume1/docker/libraries. The lidarr config is in /volume1/docker/compose/media/lidarr where the docker-compose.yml is in /volume1/docker/compose/media. So I'm guessing that having the container's /downloads-lidarr-extended and /media mapped to the same MEDIADIR folder doesn't help?

RandomNinjaAtk commented 1 year ago

No, what I'm asking about is under Lidarr Web UI: Settings -> Media management -> Root Folders

If you have more than one root folder, that could likely be the issue...

BackedUpBooty commented 1 year ago

Ah gotcha. Yes I have a root folder for music and one for music videos. Is the music videos one not required?

RandomNinjaAtk commented 1 year ago

You shouldn't be mapping the root folder for videos in Lidarr Web UI, Lidarr can do nothing with them, so it has no purpose. You should remove that and see if it works.

BackedUpBooty commented 1 year ago

Looks like that got it, thanks for the help.