Version 5.2.3 breaks reading of mp3 picture.data for some files about node-taglib-sharp

[benrr101]

@digimezzo thanks for reporting this - yeah, the last release was supposed to fix the way that GEOB frames were handled.
Looking into the file, I can see it has a GEOB frame that is of size 1394 bytes w/description "RealJukebox:Metadata". GEOB frames (General Encapsulated Object) are included in the Pictures property (at this point Picture is a bit of a misnomer since it's generally any kind of attachment to the file). It sounds like it might be necessary to tweak your usage of the picture data in your app such that PictureType.NotAPicture is filtered out of Pictures.

I puzzled over this for a long while... I have a small utility program that dumps tags, and on debugging that file, it wouldn't show the GEOB as part of Pictures. But looking at the TagLib# source, Pictures simply gets all frames that inherit from AttachmentFrame. Upon closer inspection, the version of TagLib# my utility program was built against is using an older version of TagLib# where only AttatchedPictureFrame was being returned. So at some point, the TagLib# behavior was updated to include GEOB frames in Pictures.

Let me know your thoughts

[benrr101]

@digimezzo any more thoughts on this, or can I close it?

[digimezzo]

@benrr101 Sorry, I meant to give this issue some time but lost track of it due to other tasks. I just tried your recommendation concerning PictureType.NotAPicture. Filtering out Pictures with PictureType.NotAPicture solves the problem. As far as I'm concerned, this issue can be closed. Thank you for the help.

[benrr101]

Thanks for confirming. I'll go ahead and close out, but as always please let me know if there's other issues!