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.

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

Cleanup of Bootstrap XSS Security issue #18

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

code-for-coffee
Copy link

@code-for-coffee code-for-coffee commented Aug 12, 2019

#17 pointed out that some security dependencies are out of date (and was originally an attempt at fixing the Bootstrap XSS issue - https://www.npmjs.com/advisories/891).

This MR is an attempt to clean up some of them. [email protected] resolves the bootstrap problem as does this package.json itself now.

I was curious if there is a good way to test this since there is no contribution.md file in the repository? Thank you so much!

@code-for-coffee code-for-coffee changed the title Cleanup of some security issues Cleanup of Bootstrap XSS Security issue Aug 12, 2019
@maxmurph
Copy link

This merge would really be appreciated. Having to manually patch on the developer's end is quiet cumbersome.

@richardtaylordawson
Copy link

Can we please get this merged?

@davidpelayo
Copy link

Is this going to be merged anytime soon?

@Legutier
Copy link

would really appreciate this merge.

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.

6 participants