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

fixed xss vulnerability #686

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

fixed xss vulnerability #686

wants to merge 2 commits into from

Conversation

dmarov
Copy link

@dmarov dmarov commented Mar 11, 2022

Severe XSS vulnerability was discovered. More here https://security.snyk.io/vuln/SNYK-JS-TOASTR-2396430.
Probably need to release it with incremented major version.
Changed origin branch from master to develop as discussed here #684

@abhishekori
Copy link

Will this be merged ?

@chris-tuncap
Copy link

chris-tuncap commented May 31, 2022

Looks good, please merge.

@jzamora5
Copy link

Hi. Is there a release for this yet? I still see latest is 2.1.4 in npm and it does not have the fix.

@chris-tuncap
Copy link

Hi. Is there a release for this yet? I still see latest is 2.1.4 in npm and it does not have the fix.

No one seems to be active in merging PRs around here. My team has moved on to using sweetalert2 to avoid the xss vulnerability.

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.

5 participants