Bump google.golang.org/protobuf from 1.28.1 to 1.33.0 #825
Conversation
![dependabot[bot]](https://avatars.githubusercontent.com/in/29110?s=60&v=4){kind=small}
Bumps google.golang.org/protobuf from 1.28.1 to 1.33.0. --- updated-dependencies: - dependency-name: google.golang.org/protobuf dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com>
dependabot
bot
added
dependencies
|
When is this going to be approved and merged? Is it safe to bump it in modules.txt and go.mod locally until it is? |
|
It doesn't sound important, because it problem in a part of protobuf library that carbonapi doesn't use. So I probably would bump all dependencies at once when I will be preparing a new release. |
|
@Civil thanks |
Carbonapi uses protobuf library, just not the part that had a security bug. Bug that depenadbot wanted to fix is in protobuf's json parsing library (protojson.Unmarshal to be precise) and carbonapi doesn't use that functionality of the library (and there are some extras that needs to be used to trigger the DoS). Protobufs are used heavily by carbonapi (main message format for backend communications), but we only use binary protocol. Dependabot is somewhat stupid - it parses go.mod, see reference to google.golang.org/protobuf potentially vulnurable version and bumps it. It doesn't check if any of the code actually uses vulnurable codepaths. |
|
Also about the question you've asked before:
Also about that - it should ok to update all modules within the same major version. In last 6-7 years I don't remember any events when even bumping all modules to latest version broke anything. |
|
@Civil thanks! is there going to be an artifact for this available in the RHEL9 repo? |
|
Autobuilds for RHEL9 should work for some time. And release will be added after I tag a new release. But honestly, it should be ok to use RHEL8 builds, carbonapi only uses cairo and libc from system libraries. |
|
@Civil sorry I don't understand - there is no carbonapi in the RHEL8 or 9 repo. |
|
I can see carbonapi for el/7, 8 and 9 in autobuilds: And I can see stable 0.16.0 for rhel8: |
|
Hm... There should've been 0.16.1 in stable, but I guess my autoclean script removed it by mistake... Well, I think 0.17.0 is coming anyway soon, so I won't bother rebuilding 0.16.1 again. |
|
Also #831 should update protobuf to 1.33.0 as part of huge vendored dependencies update. |
|
@Civil thanks I just wget'd the rpm down. fwiw the package definitely isn't available from the repo via yum. not sure what the deal is. thanks! |
Bumps google.golang.org/protobuf from 1.28.1 to 1.33.0.
You can trigger a rebase of this PR by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot mergewill merge this PR after your CI passes on it@dependabot squash and mergewill squash and merge this PR after your CI passes on it@dependabot cancel mergewill cancel a previously requested merge and block automerging@dependabot reopenwill reopen this PR if it is closed@dependabot closewill close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)You can disable automated security fix PRs for this repo from the Security Alerts page.