Issue with album art data about tinytag HOT 5 CLOSED

Hey @candh,

Thanks for providing the samples. Using the samples and your investigation with the binary diffs I knew that there must be a bug in the parser for the image container, because the image header was missing some bytes in the beginning.

Interestingly the parser problem was masked by another bug (that's why it worked most of the time). So we hit two birds with one stone here! 🐦 🐦 🔫 😸

Your issue should now be resolved in master. I will release a new version promptly. If you find any more problems feel free to open another issue.

Thanks again for your help improving tinytag!
Have a piece of cake 🍰

from tinytag.

Hey @candh, thanks for your excellent bug report!

I'd love to solve this, but I still need you to upload the mp3, so I can take a closer look at the data.

from tinytag.

Thanks @devsnd for the quick reply.

I have uploaded both the mp3s here https://drive.google.com/drive/folders/1WM_wQRmZyp3zCWF2UfI1qnSWwUunHNhf

from tinytag.

Sure, no problem! I'll check it out..
Thanks for the great work. 🙂

from tinytag.

I feel like an idiot, because I pip installed tinytag and just hit this same issue and was trying to resolve it (my temp solution was to set self._image_data = content[desc_start_pos:]), and then I realized the row counts between the main scripts were off when I looked at the repository...

I see now that 0.19.0 is the current version while PyPI still has version 0.18.0. You live and you learn, I guess! 😅

from tinytag.