Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Request to release an update with updated dependecies #540

Open
sukeshpabolu opened this issue Oct 23, 2023 · 4 comments
Open

Request to release an update with updated dependecies #540

sukeshpabolu opened this issue Oct 23, 2023 · 4 comments

Comments

@sukeshpabolu
Copy link

This project uses @digitalbazaar/http-client from 3.4.1 to 4.0.1. As @digitalbazaar/http-client:3.4.1, uses ky-universal which is causing problems because of top level await.

@davidlehn
Copy link
Member

  • What environment is this causing problems with?
  • We had some problems related to this, but I thought the delayed loading patches fixed that.
  • Doing this update is problematic in the current 8.x release. http-client@4 is a breaking change and will require an updated dependency on node 18+.
  • We will very soon bring in a rdf-canonize breaking change update. We might make everything node 18+ at that point and update http-client and all the ancient dev deps. If we don't do that on the first release with the new lib, it will be very shortly after.

@sukeshpabolu
Copy link
Author

HI,
Thanks for responding. I am using this in my Sveltekit application and using vite + esbuild + svelte-adapter-azure-swa to bundle and deploy the code to Azure SWA

@agustaf9
Copy link

Hello, I have also run into an issue with the older version of rdf-canonize. Emoji's in strings throws "Unsupported U escape". It was fixed in this commit of rdf-canonize: digitalbazaar/rdf-canonize@006a070#diff-c697334692792a5c11dd7b3cc11bfa38e0bc197e06e4656b8f42a187954e3d6f

@davidlehn
Copy link
Member

@agustaf9 That dependency is updated in a branch that will be released soon: #548. Since this will be a major version release, there were a few more cleanups to do such as updating all the other dependencies.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants