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

chore: require 1.3.2 for indirect gogo/protobuf #30

Merged
merged 1 commit into from
Aug 26, 2022

Conversation

jdstrand
Copy link
Contributor

@jdstrand jdstrand commented Mar 30, 2022

To address CVE-2021-3121:

$ go mod edit -require github.com/gogo/[email protected]
$ go mod tidy

I've checked the commits since 1.3.1 in https://github.com/gogo/protobuf/commits/v1.3.2 and this should be safe.

To address CVE-2021-3121:

  $ go mod edit -require github.com/gogo/[email protected]
  $ go mod tidy
@jdstrand jdstrand requested review from WesleyDavid and lisale0 March 31, 2022 20:42
@jdstrand
Copy link
Contributor Author

@lisale0 - thanks for the review! Can you (or @WesleyDavid) comment on if this is safe to commit now? IIRC, landing changes in wirey needs to be timed well in relation to customer rollouts, etc (please correct me if I am misremembering).

@lisale0
Copy link

lisale0 commented Aug 26, 2022

Hi @jdstrand, you're good to merge these changes

@jdstrand-dontuse jdstrand-dontuse merged commit ec76a43 into master Aug 26, 2022
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.

3 participants