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

If MusicBrainz Returns an empty XML (404) whole task fails and no results displayed #10796

Closed
mixxxbot opened this issue Aug 23, 2022 · 4 comments

Comments

@mixxxbot
Copy link
Collaborator

Reported by: fatihemreyildiz
Date: 2022-07-31T13:47:50Z
Status: Confirmed
Importance: High
Launchpad Issue: lp1983206


I have been working on DlgTagFetcher (I mentioned on the previous bug). I realized that there are some tracks has related recording found by AcoustID (for example 16), then MusicBrainzTask starts. After a while of getting responds with found Recording ID's (after 8th Recording ID), there is one Recording ID starts and get network replies and the tasks stops with failed.

I went to response link and the link is this:
https://musicbrainz.org/ws/2/recording/09a4f166-78ad-4043-838f-e3c4a55a63fe?inc=artists+artist-credits+releases+release-groups+media

So basically it returns 404 and the XML is empty. I think that if this link wasn't on the list we could get the related metadata. That's probably related with the MusicBrainz, but can we handle this situation in order to get the metadata found? Since we get all the links from previous task, would it cause a bug or big problems if we pass 404 and start the next MusicBrainzTask and if that's the last link just we send signal the process is success?

If you can approve this is a bug and if you say we should handle the 404, I would like to fix this issue too in order to get more metadata.

@mixxxbot
Copy link
Collaborator Author

Commented by: fatihemreyildiz
Date: 2022-08-17T20:41:31Z


In order to solve this bug, I have added a block of code in the #4871, it basically checks if the task is looping, and if it is looping and the response is empty XML that task is passed to the next task.

@mixxxbot mixxxbot transferred this issue from another repository Aug 24, 2022
@daschuer
Copy link
Member

daschuer commented Sep 9, 2022

Fix in progress here: #10875

@luzpaz
Copy link
Contributor

luzpaz commented Mar 6, 2023

Is this fixed?

@ronso0
Copy link
Member

ronso0 commented Mar 6, 2023

Yes, #10875 was merged and is now available in Mixxx 2.3.4

@ronso0 ronso0 closed this as completed Mar 6, 2023
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

4 participants