Skip to content
This repository was archived by the owner on Mar 3, 2025. It is now read-only.

🐛 Disable HTTP/2 by Default for Webhooks to Mitigate CVE Risks - #487

Closed
camilamacedo86 wants to merge 1 commit into
operator-framework:release-v1.0from
camilamacedo86:patch-release
Closed

🐛 Disable HTTP/2 by Default for Webhooks to Mitigate CVE Risks#487
camilamacedo86 wants to merge 1 commit into
operator-framework:release-v1.0from
camilamacedo86:patch-release

Conversation

@camilamacedo86

Copy link
Copy Markdown
Contributor

Cherry-pick: #484

Ensure HTTP/2 is disabled by default for webhooks. Disabling HTTP/2 mitigates vulnerabilities associated with:
  - HTTP/2 Stream Cancellation (GHSA-qppj-fm5r-hxr3)
  - HTTP/2 Rapid Reset (GHSA-4374-p667-p6c8)

While CVE fixes exist, they remain insufficient; disabling HTTP/2 helps reduce risks. For details, see: kubernetes/kubernetes#121197
@camilamacedo86
camilamacedo86 requested a review from a team as a code owner December 18, 2024 15:33
@codecov

codecov Bot commented Dec 18, 2024

Copy link
Copy Markdown

Codecov Report

Attention: Patch coverage is 0% with 11 lines in your changes missing coverage. Please review.

Project coverage is 37.89%. Comparing base (adf64a0) to head (cd464ee).

Files with missing lines Patch % Lines
cmd/manager/main.go 0.00% 11 Missing ⚠️
Additional details and impacted files
@@               Coverage Diff                @@
##           release-v1.0     #487      +/-   ##
================================================
- Coverage         38.23%   37.89%   -0.35%     
================================================
  Files                15       15              
  Lines              1224     1235      +11     
================================================
  Hits                468      468              
- Misses              706      717      +11     
  Partials             50       50              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@camilamacedo86

Copy link
Copy Markdown
Contributor Author

/hold

@openshift-ci openshift-ci Bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Dec 18, 2024
@camilamacedo86

Copy link
Copy Markdown
Contributor Author

we will wait the full release

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant