update GetVerificationKey type to match node-jwks-rsa

kylerm42 commented 2 years ago

Description

A PR was recently merged to auth0/node-jwks-rsa that changed the type of GetVerificationKey, which breaks the interaction with express-jwt. This PR updates the type here to match the change there.

References

[Issue that led to node-jwks-rsa change](https://github.com/auth0/node-jwks-rsa/issues/328
PR of the actual change
Fixes #304

Testing

None, does this need a test?

Checklist

[x] I have added documentation for new/changed functionality in this PR or in auth0.com/docs
[x] All active GitHub checks for tests, formatting, and security are passing
[x] The correct base branch is being used, if not the default branch

manast commented 2 years ago

Whats holding this PR to be merged?

jfromaniello commented 1 year ago

Thank you, published in v8.1.0.

Note v8 is a major because of a major upgrade in jsonwebtoken, but it should not affect applications on most cases.

jfromaniello commented 1 year ago

And sorry about the delay 🙏

auth0 / express-jwt