Fix timeout issue when following redirects

[laittg]

When following redirects the timer variable will be created multiple times if open_timeout and/or read_timeout is set (>0).

This makes the script always wait until setTimeout is invoked, thus always emit the timeout event.

Quick fix:

• clearTimeout before following redirects
• also invoke done() inside setTimeout() function on read_timeout to terminate the job correctly

[tomas]

Good catch! Would you add a quick test to ensure the PR fixes the bug?

[laittg]

A unit test is added, @tomas

$ mocha --grep 'follow redirects'


  follow redirects when read_timeout is set
    √ clear timeout before following redirect (3017ms)


  1 passing (3s)

[tomas]

Beautiful! Thanks.