Fixes missing host header on retried request when using forever agent #380

Merged
merged 1 commit into from Nov 29, 2012

Projects

None yet

2 participants

@mac-
mac- commented Nov 28, 2012

Fixes #349

@mikeal
Member
mikeal commented Nov 29, 2012

i'm trying to remember why this line is there in the first place. can anyone think of a reason we need to cleanup the host header on error. once we hit an error (if we aren't using forever agent) there wont be additional requests?

@mac-
mac- commented Nov 29, 2012

If there was a reason to have it, we can always just move it below the forever agent condition instead of removing it. However, when a request errors out and the forever agent isn't being used, an error is just returned to the callback, so is it really necessary to modify anything on the request object?

@mikeal mikeal merged commit 000ee26 into request:master Nov 29, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment