Reading MP3 files whose ID3v2 tags contain multiple PRIV frames about atldotnet

[Numpsy]

I've emailed you a link to a file.

[Zeugma440]

Proper support for PRIV frames will be available on next release ✅

I've updated the dedicated page about non-standard fields to document how.

Stay tuned~

[Numpsy]

Thanks, and I don't have any rush for a release.

I still need a greenlight from you regarding XMP for MP4/M4A

Sorry about that, I've been doing too many things at once recently, I'll give that a test later.

[Numpsy]

To add a bit a extra information to the post from yesterday - One of the files I was testing with contains both an XMP PRIV frame and a PRIV frame with the identifier 'www.amazon.com` (which I gather contains metadata about who bought the track from Amazon for tracking purposes), but only one of those seems to be reported by ATL

[Zeugma440]

Hi there and thanks for your feedback~

To be honest, I never had to focus on the PRIV field until now, that's why support is lacking.

I can definitely solve that, but I'd be more efficient if I had the file in question (the one with two PRIV fields). Could you send it to my e-mail ([github username][at][hot ma il][dot][com]) through a 3rd party hosting site (e.g. dropbox, wetransfer, mega...)?

Thanks in advance~

[Numpsy]

Thanks :-)

[Zeugma440]

Fix is available in today's v5.20

Please close the issue if it works on your side~

[Numpsy]

Thanks, I'll give it a go.

[Numpsy]

I've done a bit of testing and it seems to be reading and writing ok, so I'll close it as the situation with multiple PRIV frames seems to be solved :-)

[Numpsy]

Hmm, maybe I was a bit premature with the comment about writing ok - I was just having a test with reading a file that was updated with ATL using the Adobe (C++) XMP SDK and it didn't seem to work.
Trying to debug through it, it looks like it doesn't expect this text encoding byte to be present inside the PRIV tag before the "XMP\0":

The description of the PRIV frame at https://mutagen-specs.readthedocs.io/en/latest/id3/id3v2.4.0-frames.html#private-frame doesn't mention an encoding byte being present in any case ?

[Zeugma440]

You are right, sorry about that. I misread the specs regarding the encoding of the Owner field.

I've just pushed the changes to the main branch : 78ff27a

Do you need a new release or are you fine with the source?

NB : I still need a greenlight from you regarding XMP for MP4/M4A 5#257)

[Zeugma440]

The fix for PRIV frames has been released with today's v5.21

[Numpsy]

Thanks, It's working for me now

[Zeugma440]

Awesome. Thanks for confirming 😉