-
-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
bump x/text to 0.3.8 #2305
bump x/text to 0.3.8 #2305
Conversation
Some information about that problem also.
fix was https://go-review.googlesource.com/c/text/+/442235/2/language/parse.go |
So I am trying to understand how Echo is affected by it though these dependencies so far if you have something like in your code you are affected:
|
@madmuffin1 what do you think? Echo is not directly/indirectly affected because there are no code that links to that problematic function? |
my take on this: echo is not directly affected, but should not have a vulnerable dependency as implementing projects might refer to the affected method. our company forbids any library with vulnerable dependencies, so all our echo projects will have to explicitly bump x/text, rather than updating this dependency. |
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.
LGTM
as |
see https://go.dev/issue/56152, https://ossindex.sonatype.org/vulnerability/CVE-2022-32149?component-type=golang&component-name=golang.org%2Fx%2Ftext&utm_source=nancy-client&utm_medium=integration&utm_content=1.0.41