Add support for request timeout in HTTP transport

I'd be happy to propose a PR, but I'd recommend rewriting the transport with fetch instead of extending the current approach:

Line 49 in db60220

const req = base.request(url, opts, (res: Socket) => {

... As I'd have to use setTimeout(), which would make the code more complicated.

The drawback of fetch() 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.

@gnarea

@gnarea feel free to submit a PR 👍

looks like the exact date it the 11th of September for node 16 EOL, so probably wouldn't merge until then

This issue is stale because it has been open 30 days with no activity.

Navigation Menu