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

Fix CVE-2018-17556 #14336

Merged
merged 1 commit into from
Feb 6, 2019
Merged

Fix CVE-2018-17556 #14336

merged 1 commit into from
Feb 6, 2019

Conversation

AgelxNash
Copy link
Contributor

What does it do?

XSS

Why is it needed?

Security reason

Related issue(s)/PR(s)

#14094 #14335

@AgelxNash AgelxNash changed the title Fix #14094 Fix CVE-2018-17556 Feb 6, 2019
@alroniks alroniks self-assigned this Feb 6, 2019
@alroniks alroniks added proposal Proposal about improvement aka RFC. Need to be discussed before start implementation. area-security pr/ready-for-merging Pull request reviewed and tested and ready for merging. labels Feb 6, 2019
@alroniks alroniks added this to the v2.7.1 milestone Feb 6, 2019
@alroniks alroniks merged commit b014e57 into modxcms:2.x Feb 6, 2019
alroniks pushed a commit that referenced this pull request Feb 6, 2019
Fix CVE-2018-17556 #14336
4219e90 
* upstream/pr/14336:
  Fix #14094
@Mark-H Mark-H mentioned this pull request Feb 7, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@alroniks alroniks alroniks approved these changes

@Mark-H Mark-H Mark-H approved these changes

@opengeek opengeek Awaiting requested review from opengeek opengeek is a code owner

Assignees

@alroniks alroniks

Labels
area-security pr/ready-for-merging Pull request reviewed and tested and ready for merging. proposal Proposal about improvement aka RFC. Need to be discussed before start implementation.
Projects
None yet
Milestone
v2.7.1
Development

Successfully merging this pull request may close these issues.
3 participants
@AgelxNash @alroniks @Mark-H