5.0.0 doesn't work in Node environment (e.g. with node-fetch)

[tessro]

👋🏼 Hi!

We noticed strange errors when attempting to upgrade 5.0.0, throughout our unit tests. Anywhere we invoked a fetch-retry-wrapped fetch, we got this error:

ReferenceError: Request is not defined

It traces to this PR: #59. Request isn't part of Node's global environment, so this check causes a ReferenceError. This applies regardless of fetch implementation, so I think it will unfortunately break all projects using Node.

I think a typeof Request !== 'undefined' prior to the instanceof check might be enough to fix things for Node. I would normally put up a PR but I'm swamped at the moment -- I can follow up on this when I have more time if that would be helpful.

[knotthere]

Hi @paulrosania - I will create a PR to support the node environment.

[tessro]

Thank you!

[jonbern]

Many thanks for reporting the issue, and for suggesting a fix. I have merged @knotthere's PR and published a new release.

[tessro]

Thanks both! 5.0.1 is working for us. :)