-
Notifications
You must be signed in to change notification settings - Fork 233
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
Can't match types definition in @types/[email protected] #393
Comments
Looking at the source code for the passport integration, it appears that the Edit: see below. I have opened a PR that changes this parameter to |
@zackdotcomputer Yes, I think the unknown type is better if it is not used. 👍 |
Checklist
Description
It works on
@types/[email protected]
, but the build fails on@types/[email protected]
due to type changes.This is caused by changes in the following parts.
https://github.com/DefinitelyTyped/DefinitelyTyped/pull/67897/files#diff-b71c8d6ec401ffed807cc36d147e6eda88a0a5eb1a17175172ce9004026d8b79R38-L41
The following patch can be used as a simple change.
https://github.com/auth0/node-jwks-rsa/blob/master/index.d.ts#L68-L74
Reproduction
Additional context
DefinitelyTyped/DefinitelyTyped#67897
jwks-rsa version
v3.1.0
Node.js version
v20.10.0
The text was updated successfully, but these errors were encountered: