decentralized-identity / did-jwt

Create and verify DID verifiable JWT's in Javascript
Apache License 2.0
331 stars 70 forks source link

Updating uint8arrays package to lts stable #312

Open mzkrasner opened 3 months ago

mzkrasner commented 3 months ago

Current outdated uint8arrays package versions are conflicting with modern bundlers as we're seeing in certain integrated environments (such as Next.js v14+).

This PR updates uint8arrays to latest stable (v5)

ukstv commented 3 months ago

@mirceanis Could you please have a look at the PR? It updates uint8arrays to the latest stable v5.

stale[bot] commented 3 weeks ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

mirceanis commented 3 weeks ago

Hey folks, We had the version of uint8arrays fixed to 3.1.1 as that was the last version publishing a CommonJS compatible package. Later pure-ESM releases cause issues in commonJS environments. Even though I would prefer to move to ESM I'm not sure how to reconcile these 2 scenarios.

ukstv commented 2 weeks ago

There is a way: move to pure ESM. Nodejs handles ESM pretty well these days, as well as the bundlers.