-
Notifications
You must be signed in to change notification settings - Fork 15
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
Mopidy tracks are not being scrobbled correctly to Maloja #110
Comments
Thanks for the detailed info.
Did you replace any of this info (EX substituting |
I replaced that information, I just didn't want for the exact details what I was listening to to distract from the issue at hand 😂 |
Also, to avoid confusion, I noticed the issue title says "Spotify tracks", when it should actually day "Mopidy tracks". It was already late at night fore when I created it andust have gotten it wrong. I edited the title to reflect the correct service. |
Should be using mapped artist string name rather than model but cannot test due to mopidy being difficult. Hopefully this works?
I'll be honest mopidy has been the most infuriating application to setup I've ever had the displeasure of working with. I run arch linux, docker, virtual machines, the works. I've debugged esoteric java webapplets that were easier to figure out than mopidy. So far mopidy, native, has failed to work with pulseaudio or pipewire. It doesn't work in docker with three different iris/built-in implementations I've tried. It worked at some point in the past but I spent the last 3 hours trying to get anything to play with constant errors. I'm using the official mopidy js implementation and have made a change that may fix your issue but i cannot test it myself. Please use |
@disi33 have you had a chance to test this? |
Thank you for addressing this, very much appreciated! |
hey @FoxxMD apologies again for the long wait - of course my homelab server died! Thanks for looking into this - very much appreciated! And sorry again for leaving you hanging there for a while! |
Describe the bug
I just created the following setup:
Now when I listen to a song via Mopidy, I do see the multi-scrobbler Mopidy integration kicking in and attempting to scrobble the track, however, it is failing with an Internal Server Error at Maloja.
This is the error log from my multi-scrobbler instance:
This is the error log from my Maloja instance:
I believe the issue lies within the formatting of the payload that multi-scrobbler is sending to Maloja.
Notice how the data of the album_artists property seems to be a list of dicts:
"albumartists":[{"__model__":"Artist","name":"<artistname>","uri":"spotify:artist:ABCDEFG123456"}]
The respective code within Maloja that is responsible for parsing/sanitizing this data does not seem to be equipped to handle this case:
cleanup.py
multi-scrobbler version used: v0.6.2
Maloja version used: v3.2.1
To Reproduce
Set up an environment as described above.
Expected behavior
I would expect this data to be properly sanitized into a format that Maloja is expecting. The Maloja API specification is defined here,
Logs
See log snippets above.
Versions (please complete the following information):
Provide version information for any related sources/clients.
Additional context
Additionally, during my I noticed that even when I try to blacklist spotify URIs from being scrobbled through Mopidy using the configuration below, multi-scrobbler still attempts to scrobble any track listened to via the Mopidy-Spotify integration. Not sure if this issue is related.
The text was updated successfully, but these errors were encountered: