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

Close open redirects to reduce phishing risks #467

Merged
merged 2 commits into from Dec 29, 2016

Conversation

2 participants
@Changaco
Member

Changaco commented Dec 26, 2016

Issue reported by @EdOverflow via email.

@Changaco Changaco requested a review from EdOverflow Dec 27, 2016

@Changaco Changaco merged commit 64f4323 into master Dec 29, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@Changaco Changaco deleted the open-redirects branch Dec 29, 2016

@EdOverflow

This looks good to me. I will test this once you deploy it and will give you a definitive response.

@Changaco

This comment has been minimized.

Show comment
Hide comment
@Changaco

Changaco Dec 29, 2016

Member

@EdOverflow You can test it now, I deployed it 20 minutes ago. ;-)

Member

Changaco commented Dec 29, 2016

@EdOverflow You can test it now, I deployed it 20 minutes ago. ;-)

@EdOverflow

This comment has been minimized.

Show comment
Hide comment
@EdOverflow
Member

EdOverflow commented Dec 29, 2016

@Changaco Changaco referenced this pull request Jan 4, 2017

Merged

Fix XSS on payment pages #488

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