Conversation
* Add prefix to nodejs dependencies to support running in Node and Cloudflare environments https://developers.cloudflare.com/workers/runtime-apis/nodejs/
|
@jordanfinners - thanks for raising this, and for waiting for a response. The change looks good; we are going to bundle it into an upcoming major version release at same time as dropping support for older end-of-life Node versions. Note that we'll also be including #978 in the upcoming major version, which removes dependency on As a heads up, CI is currently failing on this PR due to older Node versions, so will hold off on merging until we have shipped a change shortly to remove the outdated Node versions from CI - will merge and approve once that's done. |
|
@CharlesRea oh awesome! Thank you! For the PRs in |
By submitting a PR to this repository, you agree to the terms within the Auth0 Code of Conduct. Please see the contributing guidelines for how to create and submit a high-quality PR for this repo.
Description
Add prefix to nodejs dependencies to support running in Node and Cloudflare environments
References
https://developers.cloudflare.com/workers/runtime-apis/nodejs/
Also changed this in the dependencies:
auth0/node-jwa#50
auth0/node-jws#110
Testing
This will break older node versions, which do not support this syntax. This goes back to Node 14
Checklist