Skip to content
This repository was archived by the owner on Dec 19, 2023. It is now read-only.

Fixed XSS #2#3

Open
Aravindha1234u wants to merge 14 commits into
418sec:masterfrom
Aravindha1234u:master
Open

Fixed XSS #2#3
Aravindha1234u wants to merge 14 commits into
418sec:masterfrom
Aravindha1234u:master

Conversation

@Aravindha1234u
Copy link
Copy Markdown

Fixed XSS #2

Issues: Cross-Site Scripting allows people to inject malicious javascript code to steal user data.

Please check if the PR fulfills these requirements

  • It's submitted to right branch according to our branching model
  • It's right issue type on title
  • When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)
  • The commit message follows our guidelines
  • Tests for the changes have been added (for bug fixes/features)
  • Docs have been added/updated (for bug fixes/features)
  • It does not introduce a breaking change or has description for the breaking change

Description

Bountry URL: https://www.huntr.dev/bounties/2-npm-tui-grid/
Issue: XSS
Fixed

Payload: ss'<img src=x onerror=alert(1)>
paste payload on any custom editor field and click outside to view javascript execution

image


Thank you for your contribution to TOAST UI product. 🎉 😘 ✨

joebordes and others added 14 commits April 15, 2021 10:22
* docs: add coreBOS to used by

* docs: change URL of coreBOS to home site
* fix: copy the data without encoding

* chore: innerHTML => textContent

* chore: apply code review
…n#1312)

* fix: duplcated component key

* fix: wrong offsetTop, offsetLeft style

* chore: add dummy story(show the cell with scroll)

* chore: add integration test for dummy rows and remove story

* chore: fix broken test case
Bumps [y18n](https://github.com/yargs/y18n) from 3.2.1 to 3.2.2.
- [Release notes](https://github.com/yargs/y18n/releases)
- [Changelog](https://github.com/yargs/y18n/blob/master/CHANGELOG.md)
- [Commits](https://github.com/yargs/y18n/commits)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* fix: add select box max-height

* fix: set max-height to checkbox editor
* fix: add i18n for filter select option

* chore: add filter button i18n

* chore: remove unnecessary blank line
* fix: Remove empty values from select filter

* fix: Nil value handling when selectAll is selected

* chore: add test case(filter empty value in select filter)

Co-authored-by: js87zz <js87zz.lee@nhn.com>
… is smaller than the filter layer left position (nhn#1327)
* feat: add draggableRow

* chore: wrong comment for grid instance

* feat: add floating row css for draggable

* feat: add draggable functionality

* chore: add test case(D&D move row)

* chore: apply code review

* feat: tree drag and drop (nhn#1324)

* feat: add tree drag operation

* refactor: tree floating cell

* refactor: add tree parent-cell class name

* feat: add drag event

* feat: move tree row

* fix: add '_isLeaf' prop for checking leaf node properly

* feat: change moveRow API for moving the tree row

* chore: add test case(move tree row)

* chore: add drag event test case

* docs: add event description

* chore: fix broken test case

* chore: fix floating row css

* feat: add custom draggable renderer

* chore: apply review

* chore: apply code review

* chore: apply code review

* feat: add appended property in drop event

* fix: wrong property(isLeaf => leaf)

* chore: fix the move to last row

* chore: add z-index to floating-line css

* feat: move floating row as X position
* fix: emit script error when copying unobservable rows

* chore: apply code review
@huntr-helper
Copy link
Copy Markdown

👋 Hello, @js87zz. @Aravindha1234u has opened a PR to us with a fix for a potential vulnerability in your repository. To view the vulnerability, please refer to the bounty URL in the first comment, above. If you want this fix in your repository, a PR will automatically open once you comment:

@huntr-helper - LGTM


☎️ Need further support?

Come and join us on our community Discord!


@js87zz - want more fixes like this?

Copy this snippet into your README.md for more vulnerability fixes in the future:

[![huntr](https://cdn.huntr.dev/huntr_security_badge_mono.svg)](https://huntr.dev)

huntr

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants