Skip to content
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

Fix issue #1038 #1446

Merged
merged 5 commits into from Feb 27, 2015
Merged

Fix issue #1038 #1446

merged 5 commits into from Feb 27, 2015

Conversation

ghost
Copy link

@ghost ghost commented Feb 23, 2015

I think this should fix the problem but I'm new to Node, so please forgive me if it's not good.

@ghost
Copy link
Author

ghost commented Feb 24, 2015

cc @bmarini

@simov
Copy link
Member

simov commented Feb 25, 2015

@guimonz that looks nice, can you add tests for this new behavior?

@ghost
Copy link
Author

ghost commented Feb 25, 2015

Sure @simov!
I'll add them tonight!
:-)

@simov
Copy link
Member

simov commented Feb 26, 2015

Looking good, @guimonz can you pull simov@bbcff41 here? Only a minor indentation fixes, other than that 👍 from me.

@ghost
Copy link
Author

ghost commented Feb 26, 2015

Here you go, @simov!

@simov
Copy link
Member

simov commented Feb 26, 2015

Thanks @guimonz I'll merge this shortly.

@ghost
Copy link
Author

ghost commented Feb 26, 2015

👍

simov added a commit that referenced this pull request Feb 27, 2015
Missing HTTP referer header with redirects Fixes #1038
@simov simov merged commit a2c9ec7 into request:master Feb 27, 2015
@ghost
Copy link
Author

ghost commented Feb 27, 2015

cc @bmarini

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants