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.

Sign up for GitHub

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

Jump to bottom

Fixed HTTPoxy vulnerability ref #251 #252

Closed
wants to merge 2 commits into from
Closed

Fixed HTTPoxy vulnerability ref #251 #252

wants to merge 2 commits into from

Conversation

kkomelin
Copy link
Contributor

@kkomelin
Copy link
Contributor Author

Accidentally included redundant commit, closing this PR in order to create a correct one.

@kkomelin kkomelin closed this Jul 19, 2016
@rfay
Copy link

rfay commented Jul 19, 2016

Just FYI in the future you can force-push into your PR to update it before it's pulled.

@kkomelin
Copy link
Contributor Author

Thanks @rfay. This is the first time I deal with such collision. Could you please refer to documentation?

@kkomelin kkomelin deleted the kkomelin-httpoxy branch July 19, 2016 23:59
@kkomelin
Copy link
Contributor Author

kkomelin commented Jul 20, 2016 •
edited
Loading

@rfay No worries. I have sorted it out.
Here is the correct PR #253

@rfay
Copy link

rfay commented Jul 20, 2016

git commit --amend (or rebase, or whatever)
git push --force (into your own fork, which is where the PR lives)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@kkomelin @rfay