Added timeout with default value of 30s

[Cooke]

Description

The purpose of this pull request is to add a timeout (with default value) which applies to the request for fetching keys.

The introduction of the timeout setting isn't itself breaking but the default value of 30s could be considered breaking, though it should be a sane value that everyone could live with.

References

Issue #131

Testing

No tests have been added.

This change depends on the timeout functionality of the request lib.

Manual verification has been done and auto tests have been executed with npm test.

Checklist

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

[davidpatrick]

Hey @Cooke thanks for the contribution. We are currently migrating off of request because of the libraries deprecation. We will be moving to axios and exposing the axios config options which will allow you to set the timeout. #134

We can move forward with this PR, but its use would break in PR #134. Let me know if you are able to wait.

Thanks

[Cooke]

@davidpatrick we can wait. We are already using our own fork in the meanwhile. Thank you

[apumb]

Hi @davidpatrick, Just wondering if there was a timeframe for moving to axios? This will solve some issues we're facing but the timing will give us a better idea of how we should proceed right now.
Thanks

[davidpatrick]

@apumb working on getting #135 merged asap

@Cooke do you want to update this PR to include the new src/wrappers/request.js https://github.com/auth0/node-jwks-rsa/pull/135/files#diff-c51c2f09575f5eff8f9d311bf261a763

We will need to pass the timeout option through to this new wrapper

[davidpatrick]

^

[Cooke]

@davidpatrick that should hopefully do it. But you should better check it. Also I think I screwed up the ticket references, sorry.

[apumb]

Thanks @davidpatrick, this is awesome!!

[davidpatrick]

@Cooke I've merged in the request dep changes, can you rebase this with master? That should resolve the conflicts