Reject NaN timeouts and cap huge ones instead of firing after 1 ms - #54

validateTimeout() only checks timeout <= 0, which misses a couple of cases.

• NaN gets through (NaN <= 0 is false), and then no timer is set at all. With a
  server that never replies, timeout: 2000 gave up after 2s but timeout: NaN
  was still waiting after 4s.

• anything above 2147483647 ms makes Node's setTimeout fire after 1 ms. So
  timeout: Infinity fails almost immediately with RequestTimeoutError, even
  when the server answers fine.

I changed the check to !(timeout > 0) so NaN throws InvalidTimeoutError, and
capped the value at 2147483647 ( 2**32 -1 ). added tests for it in tests/validators_test.ts.

note that NaN used to be silently accepted and now throws. Capping instead of
rejecting big values was my call, so I can change that if you'd rather.

Before Fix

After Fix

You can see changes in above screenshots