8000 Fix xss bug in multiple select box by carakas · Pull Request #3501 · forkcms/forkcms · GitHub
[go: up one dir, main page]

Skip to content

Fix xss bug in multiple select box#3501

Merged
carakas merged 1 commit intoforkcms:masterfrom
justcarakas:fix-xss-bug-in-multiple-selectbox
Mar 24, 2022
Merged

Fix xss bug in multiple select box#3501
carakas merged 1 commit intoforkcms:masterfrom
justcarakas:fix-xss-bug-in-multiple-selectbox

Conversation

@carakas
Copy link
Member
@carakas carakas commented Mar 23, 2022

Type

  • Security

Pull request description

Multiple select box was vulnerable to xss strings

@carakas carakas added this to the 5.11.1 milestone Mar 23, 2022
@carakas carakas requested a review from a team as a code owner March 23, 2022 19:22
@codecov
Copy link
codecov bot commented Mar 23, 2022

Codecov Report

Merging #3501 (8e87608) into master (c9d0f23) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #3501   +/-   ##
=========================================
  Coverage     27.87%   27.87%           
  Complexity     8150     8150           
=========================================
  Files           575      575           
  Lines         30705    30705           
=========================================
  Hits           8559     8559           
  Misses        22146    22146           
Flag Coverage Δ
functional 23.78% <ø> (ø)
installer 3.84% <ø> (ø)
unit 7.69% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c9d0f23...8e87608. Read the comment docs.

@carakas carakas force-pushed the fix-xss-bug-in-multiple-selectbox branch from 2f87f46 to 8e87608 Compare March 23, 2022 19:50
@carakas carakas merged commit 45695f2 into forkcms:master Mar 24, 2022
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.

3 participants

0