These commits allow for configuration to also follow redirects (302) when issuing POST requests. The default is still not to follow them (why?) so it should be safe. I also added a test case which tests the functionality. Thanks of a great library! Keep u #98

Merged
merged 2 commits into from May 17, 2012

Conversation

Projects
None yet
2 participants
Contributor

irony commented May 16, 2012

No description provided.

Contributor

chriso commented May 17, 2012

Thanks. The request library node.io uses changes its API often. When I incorporated it originally this was the default behaviour :/

@chriso chriso added a commit that referenced this pull request May 17, 2012

@chriso chriso Merge pull request #98 from Iteam1337/master
These commits allow for configuration to also follow redirects (302) when issuing POST requests. The default is still not to follow them (why?) so it should be safe. I also added a test case which tests the functionality. Thanks of a great library! Keep u
4968447

@chriso chriso merged commit 4968447 into node-js-libs:master May 17, 2012

Contributor

irony commented May 17, 2012

I understand, no problem. Do you want me to ask @mikeal to merge this fix as well?

Contributor

chriso commented May 20, 2012

No, it's not an issue on that end, just a matter of locking in a version of request and watching out for changes.

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