-
Notifications
You must be signed in to change notification settings - Fork 18
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
Add support of new native methods. Fix vulnerabily #10
base: master
Are you sure you want to change the base?
Conversation
…h a RegExp in code for old browsers.
Hi. The current version of the dependency does not have a vulnerability, but it is also not hosted in this repository. It's hosted at https://github.com/Trott/trim instead. Unfortunately, I don't have access to close this issue or archive this repository or anything like that. |
A StoryBook |
Thanks for the quick feedback. |
@Trott is there a way to add a pacth for v0.0.3? I'll send a pull request if need it. |
What makes you say it's using 0.0.3? https://github.com/storybookjs/storybook/blob/c745ff687e0dd445e0b9b4b908c1dfe75b3bfa3a/code/yarn.lock says 0.0.1.
And 0.0.3 is not vulnerable to ReDoS while 0.0.1 is. I don't know if GitHub permits pull requests against tags, but if you wanted to open a pull request against 0.0.3, I suppose you could try to open a request against https://github.com/Trott/trim/tree/v0.0.3. Not sure GitHub allows that though. If not, I could create a But I'm not convinced there's an issue to patch in 0.0.3.... |
Hey there!
The lib is a dependecy of the StoryBook project.
Current version of the lib has a vulnerabily so I fixed it and also added support of new native String methods:
Feel free to ask questions