Enhance redirect handling #81

Merged
merged 1 commit into from Sep 28, 2011

Projects

None yet

2 participants

@danmactough
Contributor

Enhance redirect handling: (1) response._redirectsFollowed reports the total number of redirects followed instead of being reset to 0; (2) add response.redirects, an array of the response.statusCode and response.headers.location for each redirect.

@danmactough danmactough Enhance redirect handling: (1) response._redirectsFollowed reports th…
…e total number of redirects followed instead of being reset to 0; (2) add response.redirects, an array of the response.statusCode and response.headers.location for each redirect.
9c897df
@mikeal
Member
mikeal commented Sep 20, 2011

looks good, I'm on planes for the next few days so it might take me a bit to get it merged.

can you verify that all the tests are passing?

@danmactough
Contributor
tests/run.sh
tests/test-body.js
options.requestBodyStream is deprecated, please pass the request object to stream.pipe.
8 tests passed.
tests/test-errors.js
All tests passed.
tests/test-pipes.js
All tests passed.
tests/test-timeout.js
All tests passed.

P.S. Safe travels

@danmactough
Contributor

Bump

@mikeal mikeal merged commit 646c80d into request:master Sep 28, 2011
@mikeal
Member
mikeal commented Sep 28, 2011

thanks for the reminder :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment