Set incoming requesting timeout

[phillipj]

This ensures the incoming request does not timeout before the target's timeout configured with options.proxyTimeout.

As I'm not familiar to the code base, I struggled seeing the best entry to write a test for this. Any thoughts would be appreciated.

Fixes #1113

Edit: I'm looking into the existing timeout test failing

[phillipj]

Looking at the test that fails with these changes added, I wonder if that test is wrong.

It doesn't expect the client request being made to throw an error if an error occurs. At first glance that seems weird, cause a timeout is also an error to the client initiating the request, right?

Additionally there is two very similar tests, whereas the latter succeeds cause it expects timeouts to raise client error:

• failing: it('should proxy the request and handle timeout error (proxyTimeout)', ..)
• similar: it('should proxy the request and handle timeout error', ..)

[indexzero]

Closing as a duplicate of #1273 – if it's not a strict duplicate . from a software design perspective this belongs in the passes not in the root lib/http-proxy/index.js

[indexzero]

Thanks for helping triage this issue and contribute a proposed fix though 😸