Hello!

That error comes from your file having an incomplete UFID frame, which according to the spec should have an owner identifier.

Looking at Symphonia, it looks like the UFID frame is ignored entirely: https://github.com/pdeljanov/Symphonia/blob/c6a14a0405281af555edca98fa56f3c51a46d459/symphonia-metadata/src/id3v2/frames.rs#L338.

Since Lofty is purely a metadata crate, it tries to stick a lot closer to the spec.

Is this a common issue in your library? It may be worth it to add an exception for this with ParsingMode::Relaxed.

Thanks for the blazing quick response!

I wasn't aware of the spec, thanks for the ref. Metadata is such a complex beast!

The library is composed of all filetypes of different origins, but I couldn't reproduce exactly this error with no other file I have.

While looking at the docs for ParsingMode::Relaxed I didn't get a clear grasp on where and how exactly to implement it.
Could you provide me with an example use?

Thanks 🙏

Yeah, that's definitely a weak spot in the docs. :)

You use it in ParseOptions which you can then pass to your Probe.

Like this:

use lofty::{Probe, ParseOptions, ParsingMode};

let options = ParseOptions::new().parsing_mode(ParsingMode::Relaxed);
let tagged_file = Probe::open(path)?.options(options).read()?;

For now ParsingMode::Relaxed will only really affect the output of property readings, but it could be expanded to cover malformed items in otherwise valid tags.

I added ParsingMode::Relaxed to my snippet, but the error persists.

For now ParsingMode::Relaxed will only really affect the output of property readings, but it could be expanded to cover malformed items in otherwise valid tags.

I know too little about the inner workings of Strict vs Relaxed when dealing with malformed items.
Your suggestion however seems very logical to me.

The reading of the probe to not panic when a key, which is required by the spec but just isn't there, would be a nice addition when working with organically grown libraries of questionable origin 😆

Thanks a lot for the help.

I know too little about the inner workings of Strict vs Relaxed when dealing with malformed items.

Yeah, no one really knows unless they read the code. I really need to get to giving ParsingMode some actual documentation. :)

Lofty is a lot stricter than other tag reading libraries by default. I'll see about expanding ParsingMode::Relaxed to cover malformed tag items. It's unfortunate that the entire file is regarded as invalid due to a single frame.

For now, you should either fix or remove the UFID frame in a tag editor.

In 0.14.0 the default parsing mode will be the new ParsingMode::BestAttempt, which will have no problem reading your UFID frame without an owner. :)