Skip to content

Upgrade Bootstrap Tags Input#8672

Closed
alecslupu wants to merge 2 commits intodecidim:developfrom
i-need-another-coffee:fix/js-bootstrap-tagsinput
Closed

Upgrade Bootstrap Tags Input#8672
alecslupu wants to merge 2 commits intodecidim:developfrom
i-need-another-coffee:fix/js-bootstrap-tagsinput

Conversation

@alecslupu
Copy link
Copy Markdown
Contributor

@alecslupu alecslupu commented Jan 1, 2022

🎩 What? Why?

Upgrade Bootstrap Tags Input to mitigate Cross-site Scripting (XSS)
More details: https://security.snyk.io/vuln/npm:bootstrap-tagsinput:20160720

♥️ Thank you!

@alecslupu alecslupu changed the title upgrade Bootstrap Tags Input Upgrade Bootstrap Tags Input Jan 2, 2022
@alecslupu alecslupu marked this pull request as ready for review January 2, 2022 08:59
@alecslupu alecslupu mentioned this pull request Jan 2, 2022
12 tasks
@andreslucena
Copy link
Copy Markdown
Member

andreslucena commented Jan 5, 2022

More info about the new package: https://www.npmjs.com/package/bootstrap-tagsinput-2021

The official Version v0.8.0 is actually not supported anymore!

Use the fixed Version bootstrap-tagsinput-2021 instead. XSS vulnerable Version fixed -> npm install bootstrap-tagsinput-2021

I don't like it, as it doesn't seem to be getting much traction (100 weekly downloads 3 months after the fork/release vs 16K of the original NPM package), and as far as I see the organization and the author doesn't have much public record in GitHub.

Seeing that the CVE is from 2016, I prefer to open a discussion to search for alternatives: #8683

@alecslupu alecslupu deleted the fix/js-bootstrap-tagsinput branch January 5, 2022 21:46
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