-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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][sec] Upgrade prometheus client_golang to v1.12.2 to fix CVE-2022-21698 #20579
Conversation
Signed-off-by: ericsyh <[email protected]>
@codelipenghui @freeznet PTAL |
Codecov Report
@@ Coverage Diff @@
## master #20579 +/- ##
=============================================
+ Coverage 31.93% 73.01% +41.08%
- Complexity 11776 31978 +20202
=============================================
Files 1498 1867 +369
Lines 114571 138636 +24065
Branches 12422 15231 +2809
=============================================
+ Hits 36583 101220 +64637
+ Misses 73149 29383 -43766
- Partials 4839 8033 +3194
Flags with carried forward coverage won't be shown. Click here to find out more. |
Merging... Thank you! I wonder what versions are expected to be cherry-picked. |
I think 3.0, 2.11, and 2.10 active branches should cherry-pick this PR. |
…2-21698 (#20579) Signed-off-by: ericsyh <[email protected]> (cherry picked from commit a85e9df)
Fixes #xyz
Master Issue: #xyz
PIP: #xyz
Motivation
Prometheus client_golang v1.11.1 is impacted by the CVE-2022-21698
Modifications
Upgrade the Prometheus client_golang to v1.12.2
Verifying this change
(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)
Does this pull request potentially affect one of the following parts:
If the box was checked, please highlight the changes
Documentation
doc
doc-required
doc-not-needed
doc-complete
Matching PR in forked repository
PR in forked repository:
My local repo CI result:
https://github.com/ericsyh/pulsar/pull/2