Closed gyaneshgouraw-okta closed 1 month ago
@nandan-bhat or @frederikprijck is there a plan to release this? I'm having the same trouble and it sounds like this fix exists and are approved but haven't been released 10+ days later
Hey @stevekanter, we are planning to get this PR merged and released by this week.
Changes
Please describe both what is changing and why this is important. Include:
undici-types
to fix the issue #1007undici
which also contains it's type was part of dev dependecyundici
dependecy was running into ts errorundici-types
as dependecy will not be required once@types/node
is updated to20.*.*
@types/node
to20.*.*
, which explicitly hasundici-types
however this will be a major change, and should be only done when this sdk is qualified to support nodeJS20.*.*
undici
to dependency list from dev-dependency, however this will give an impression that we are extensively usingundici
currently, which is not the case, so just opted forundici-types
References
Please include relevant links supporting this change such as a:
1007
Testing
Please describe how this can be tested by reviewers. Be specific about anything not tested and reasons why. If this library has unit and/or integration testing, tests should be added for new functionality and existing tests should complete without errors.
Step 1 - auth0-node
npm run build
dist
folderStep 2 - ClientApp setup
node_modules
->auth0-> dist with copieddist
from step1dist
folder error will be seen[ ] This change adds unit test coverage
[ ] This change adds integration test coverage
Checklist