Skip to content

Conversation

@nicoloboschi
Copy link
Contributor

@nicoloboschi nicoloboschi commented Mar 9, 2022

Follow-up of #14579.

Motivation

OWASP checker reports this vulnerability
https://nvd.nist.gov/vuln/detail/CVE-2022-24329
for Kotlin < 1.6.x

Currently we import Kotlin 1.4.32 from OkHttp3 (see #13065).
CVE-2022-24329 is rated as mid CVSS level (5.0).
Kotlin is used only by the Kubernetes client runtime lib.

Given that:

  • Pulsar codebase doesn't have a good test coverage for the K8S client
  • The vulnerability is mid level
  • The vulnerability doesn't look relevant for Pulsar

It's safer to add the suppression instead of upgrading it without testing it.

Modifications

  • Add the supression for Kotlin 1.4.32 for the cve CVE-2022-24329

  • no-need-doc

@github-actions github-actions bot added the doc-not-needed Your PR changes do not impact docs label Mar 9, 2022
@nicoloboschi
Copy link
Contributor Author

need to be cherry-picked to 2.9 as well https://github.com/apache/pulsar/runs/5473308768?check_suite_focus=true

@nicoloboschi nicoloboschi changed the title [owasp] add suppressions for Kotlin stdlib CVE-2022-24329 [owasp] add suppression for Kotlin stdlib CVE-2022-24329 Mar 9, 2022
@lhotari
Copy link
Member

lhotari commented Mar 9, 2022

/pulsarbot run-failure-checks

@lhotari lhotari merged commit 4910519 into apache:master Mar 10, 2022
nicoloboschi added a commit to datastax/pulsar that referenced this pull request Mar 10, 2022
nicoloboschi added a commit to datastax/pulsar that referenced this pull request Mar 10, 2022
RobertIndie pushed a commit to RobertIndie/pulsar that referenced this pull request Mar 28, 2022
Nicklee007 pushed a commit to Nicklee007/pulsar that referenced this pull request Apr 20, 2022
nicoloboschi added a commit to datastax/pulsar that referenced this pull request May 5, 2022
@mattisonchao
Copy link
Member

Hi @nicoloboschi Would you mind cherry-pick this PR to branch-2.9?

@nicoloboschi nicoloboschi deleted the suppress-kotlin-owasp branch May 24, 2022 08:02
nicoloboschi added a commit that referenced this pull request May 24, 2022
nicoloboschi added a commit that referenced this pull request May 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants