-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
fix: fix quadratic execution in em mask #2818
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Nice! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh wow how did you notice this?
@calculuschild: This was one of the first things I learned about when joining Marked on its adventure. Sometimes folks discover vulnerabilities, let the maintainers know, then the maintainers fix the issue. This is how we met @davisjam in fact. (Mainly because I didn't handle it well - see comment history from a few years ago...I learned a lot about OSS in a very short time.) In this case someone from Snyk sent notification in accordance with the security notice: https://github.com/markedjs/marked/blob/master/SECURITY.md @UziTech being the awesome human he is, responded in a timely fashion with proposed solution. Snyk offered feedback to improve. And here we are. Hope that helps (and is an accurate representation). |
## [5.0.3](v5.0.2...v5.0.3) (2023-05-26) ### Bug Fixes * fix quadratic execution in em mask ([#2818](#2818)) ([a37fe8e](a37fe8e))
Marked version: v5.0.2
Description
Fix quadratic execution in em mask regexp
Contributor
Committer
In most cases, this should be a different person than the contributor.