-
Notifications
You must be signed in to change notification settings - Fork 3.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
add option followAllRedirects to follow post/put redirects #90
Conversation
this looks good but it's a big change that i want to spend some time reviewing. i will get to it, i promise, just be patient :) |
Sure you've gotta lot of stuff on your plate @mikeal, any chance you have some time to revisit this pull request? This would be very helpful to have. |
Sorry, forgot all about this one. This needs a test for maxRedirects added and a line added to the docs to describe the option. Once those are in and the comment by @kitek is addressed it could be merged. |
…o 'GET' This behavior is as described by RFC 2616. This change is in addition to issue request#55 and issue request#90 (pull request)
Ok, took care of the comments raised in this pull request.
Not sure why the |
add option followAllRedirects to follow post/put redirects
This should close #55.