feat: use teeny-request instead of deprecated request dependency

[wvanderdeijl]

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

• Make sure to open an issue as a bug/issue before writing your code! That way we can discuss the change, evaluate designs, and agree on the general idea
• Ensure the tests and linter pass
• Code coverage does not decrease (if any source code was changed)
• Appropriate docs were updated (if necessary)

Fixes #1584

This is my suggested fix to #1584. Having a dependency on request is not wise as it is deprecated and has security issues. But it also poses an issue with bundlers (such as webpack) that will try to include request that is not installed in node_modules. Hence this solution with an explicit dependency to teeny-request

[wvanderdeijl]

retry-requestsince 7.0.0 itself already has a dependency on teeny-request. So, we're not even introducing a new dependency here.
The documentation of retry-request itself now also suggests to use teeny-request and no longer rely on the deprecated request

[wvanderdeijl]

If you want to keep the require statement conditional, I could revert to doing this around line 44 in if (opts.request === undefined)

[wvanderdeijl]

This error will be thrown by retry-request itself if opts.request is empty

[nicole0707]

Is anyone available to review the PR? @leahecole 🙏. It's blocking us for a while, Thanks!

[leahecole]

thanks for the ping! I want @sofisl to take a look at this first but Sofia, ping me if you want to chat about it

[leahecole]

Just had a brain blast, wasn't @danielbankhead doing work to migrate us from teeny-request to gaxios? Wondering if we may want to use that instead here.

[danielbankhead]

@leahecole yep! We will want to migrate away from both retry-request and teeny-request to gaxios.

[sofisl]

Since gaxios is used in auth, it won't make the package any bigger :) I think this is in the next phase of @danielbankhead's project so we should be working on it soon. As an FYI, I am planning on deprecating teeny and retry within the next few months, will put up a deprecation notice on the README to alert users long before we deprecate so that they can start preparing.

[leahecole]

Putting a do not merge on this given all of the shifting that's going on with teeny-request and gaxios right now, and because @danieljbruce may end up consolidating streamingRetryRequest into streaming.ts, rendering it obsolete. I'll make a note to look into this after that happens though - we do need to remove the request dependency!