Fix cookie and redirect bugs and add auth support for HTTPS tunnel #203

Merged
merged 5 commits into from Mar 14, 2012

Projects

None yet

4 participants

@milewise

Fixed: The proxy.auth option wasn't being passed through to the https tunnel. This prevented the https tunnel from being used with proxy servers that require authentication.

Fixed: Due to a bug in the composable jar code, cookies weren't being set properly in redirects.

Fixed: Cookie attributes should be case insensitive. For example, the cookie parser fails when Path=/ is used instead of path=/

Fixed: When following non-GET redirects, remove any content in the body of the request. Otherwise, servers will complain about the GET request being malformed.

@mikeal
Member
mikeal commented Mar 14, 2012

would appreciate a node on this one from @isaacs

@isaacs
Contributor
isaacs commented Mar 14, 2012

LGTM. Doesn't appear to break npm. It would be nice if there was a test for the basic auth HTTPS tunneling.

@mikeal mikeal merged commit 7184e8c into request:master Mar 14, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment