Results 1383 comments of Vincent Prouillet

I think we could probably just check whether all the timestamp values are sane before?

Your initial patch is much simpler and works, let's go with that ^

We do support some of it but not all. No plan to support everything from it

Should we just link to the line in the Cargo.toml? Keeping them in sync seem like it would never work

Have you seen this PR: https://github.com/Keats/jsonwebtoken/pull/318 ? It's something I was considering

It looks like building this lib on windows is problematic? https://github.com/aws/aws-lc/issues/1477

I don't remember why it's done that way tbh

I'm not sure, would that be considered a breaking change?

I think i'd consider a breaking change but mostly for the end user. If it's included in Display, we will see the source errors twice for apps that did a...