- Notifications
You must be signed in to change notification settings - Fork72
Open
Labels
Description
I'd be happy to propose a PR, but I'd recommend rewriting the transport withfetch instead of extending the current approach:
| constreq=base.request(url,opts,(res:Socket)=>{ |
... As I'd have to usesetTimeout(), which would make the code more complicated.
The drawback offetch() is that it'd require Node.js 18+. Which shouldn't be too bad as Node.js 16 is reaching EOL in a few weeks.