Skip to content

ci: suppress 6 new trivy CVE(s) pending remediation - #169

Open
ik-code-writer[bot] wants to merge 2 commits into
masterfrom
automation/trivy-ignorefile
Open

ik-code-writer[bot] wants to merge 2 commits into
masterfrom
automation/trivy-ignorefile

Conversation

@ik-code-writer

@ik-code-writer ik-code-writer Bot commented Nov 8, 2026 •

Copy link
Copy Markdown

Add temporary Trivy ignore entries for newly reported CVEs.

Jira issue(s): not linked
Workflow run: https://github.com/indykite/developer-hub/actions/runs/42630329542

The entries reference confirmed Jira issues or the workflow run and should be removed after remediation or formal risk acceptance.

@ik-code-writer
ik-code-writer Bot requested review from a team as code owners November 8, 2026 23:22
Copilot AI balanced review requested due to automatic review settings November 8, 2026 23:22

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Changes recommended

Switching to .trivyignore.yaml drops three existing suppressions because Trivy does not merge ignore files, allowing previously ignored findings to fail scans again.

1 open issue
What changed in this PR

Adds six Trivy vulnerability suppressions pending remediation and configures scans to use the YAML ignore file.

Changes:

  • Records six CVE ignore entries with workflow-run references and generation timestamps.
  • Enables .trivyignore.yaml in the Trivy configuration.
File Description
.trivyignore.yaml Adds six CVE suppressions with traceability statements.
.trivy.yaml Selects the YAML ignore file and removes extra blank lines.

Review effort: Balanced

Comment thread .trivyignore.yaml Resolved
@deepsource-io

deepsource-io Bot commented Nov 8, 2026 •

Copy link
Copy Markdown

DeepSource Code Review

We reviewed changes in 13d64df...1f30456 on this pull request. Below is the summary for the review, and you can see the individual issues we found as inline review comments.

See full review on DeepSource ↗

PR Report Card

Overall Grade   Security  

Reliability  

Complexity  

Hygiene  

Code Review Summary

Analyzer Status Updated (UTC) Details
Secrets Nov 10, 2026 7:46p.m. Review ↗
Python Nov 10, 2026 7:46p.m. Review ↗

Important

AI Review is run only on demand for your team. We're only showing results of static analysis review right now. To trigger AI Review, comment @deepsourcebot review on this thread.

@ik-code-writer
ik-code-writer Bot force-pushed the automation/trivy-ignorefile branch from 9dbc84a to 3c001f3 Compare November 9, 2026 13:53
Copilot AI balanced review requested due to automatic review settings November 9, 2026 13:53

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Changes recommended

The new Trivy ignore file omits three existing suppressions, and because it replaces rather than merges with the default file, previously suppressed findings can fail scans again.

2 open issues

Review effort: Balanced

Comment thread .trivyignore.yaml Resolved
@ik-code-writer
ik-code-writer Bot force-pushed the automation/trivy-ignorefile branch from 3c001f3 to fbb5a91 Compare November 9, 2026 14:00
Copilot AI balanced review requested due to automatic review settings November 9, 2026 14:00

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Changes recommended

Switching to .trivyignore.yaml drops the existing suppressions for CVE-2025-8869, CVE-2026-3219, and KSV-0125, allowing previously exempted findings to fail scans unless those entries are preserved.

3 open issues

Review effort: Balanced

Comment thread .trivyignore.yaml Resolved
@cowan-macady
cowan-macady force-pushed the automation/trivy-ignorefile branch from fbb5a91 to 0ba1f70 Compare November 9, 2026 15:05
Copilot AI balanced review requested due to automatic review settings November 9, 2026 15:05

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Changes recommended

Switching Trivy to .trivyignore.yaml drops the existing vulnerability and misconfiguration suppressions, which must be migrated to prevent scans from failing on intentionally excluded findings.

4 open issues

Review effort: Balanced

Comment thread .trivyignore.yaml Resolved
Copilot AI balanced review requested due to automatic review settings November 9, 2026 15:09

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔵 Needs a closer look

The six new CVE suppressions lack affected-package, severity, and remediation details, requiring security-owner review to validate the accepted risk and intended exception duration.

No open issues 🎉

4 resolved since last review

Review effort: Balanced

cowan-macady and others added 2 commits November 10, 2026 20:45
…sions when setting ignorefile'

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@cowan-macady
cowan-macady force-pushed the automation/trivy-ignorefile branch from 198f798 to 1f30456 Compare November 10, 2026 19:46
Copilot AI balanced review requested due to automatic review settings November 10, 2026 19:46

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔵 Needs a closer look

The six repository-wide CVE suppressions lack supporting scan findings and remediation or risk-acceptance evidence in the snapshot, requiring security-owner validation of their scope and justification.

No open issues 🎉

Review effort: Balanced

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants