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

[fix][ci] Fix OWASP dependency check suppressions #20486

Merged
merged 1 commit into from
Jun 5, 2023

Conversation

lhotari
Copy link
Member

@lhotari lhotari commented Jun 5, 2023

Motivation

OWASP dependency check fails because of false positives.

Modifications

  • jetcd shouldn't match etcd or grpc CVEs
    • add proper suppression rules that cover this
  • keep OWASP Dependency Check version up-to-date
    • Upgrade to 8.2.1 version

Documentation

  • doc
  • doc-required
  • doc-not-needed
  • doc-complete

- jetcd shouldn't match etcd or grpc CVEs
- keep OWASP Dependency Check version up-to-date
@lhotari lhotari self-assigned this Jun 5, 2023
@github-actions github-actions bot added the doc-not-needed Your PR changes do not impact docs label Jun 5, 2023
@poorbarcode poorbarcode added this to the 3.1.0 milestone Jun 5, 2023
@tisonkun
Copy link
Member

tisonkun commented Jun 5, 2023

Merging...

Thank you!

@tisonkun tisonkun merged commit 3b862ae into apache:master Jun 5, 2023
lhotari added a commit that referenced this pull request Jun 6, 2023
(cherry picked from commit 3b862ae)

# Conflicts:
#	src/owasp-dependency-check-suppressions.xml
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ci doc-not-needed Your PR changes do not impact docs ready-to-test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants