Skip to content

fix: security work flow - #80

Merged
voorepreethi merged 5 commits into
newrelic-forks:masterfrom
voorepreethi:NR-587390
Jul 20, 2026
Merged

fix: security work flow#80
voorepreethi merged 5 commits into
newrelic-forks:masterfrom
voorepreethi:NR-587390

Conversation

@voorepreethi

@voorepreethi voorepreethi commented Jul 9, 2026

Copy link
Copy Markdown

Summary
Adds an automated security scanning workflow to improve continuous vulnerability detection and alerting for this repository.
What changed
Added a Trivy filesystem scan focused on HIGH and CRITICAL findings.
Added SARIF upload to GitHub Security where permissions allow it.
Added scheduled and manual execution support.
Added Slack notifications when scheduled/manual scans fail.
Added a dedicated notification path for Dependabot PR events.
Notes
This PR is focused on workflow automation and does not change application runtime code.

@satya-nr

satya-nr commented Jul 9, 2026

Copy link
Copy Markdown

No .github/dependabot.yml changes in this PR, but the workflow has a dependabot-notify job

Comment thread .github/dependabot.yml
- "/v2/cdc-parent/*"
schedule:
interval: cron
cronjob: "0 11 * * 1"

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Schedule style looks different from what I've seen, are you sure this works? If yes, all good

@satya-nr

Copy link
Copy Markdown

Seeing Trivy scan failing on real CVEs, can we fix them in this PR or will take care later !

@satya-nr satya-nr 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.

Looks good to me.

exit-code: '1'
env:
TRIVY_DB_REPOSITORY: public.ecr.aws/aquasecurity/trivy-db
TRIVY_JAVA_DB_REPOSITORY: public.ecr.aws/aquasecurity/trivy-java-db

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

does it have JAVA code?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

yes

"text": ":rotating_light: Trivy detected HIGH/CRITICAL CVEs on ${{ github.ref_name }} in ${{ github.repository }}\nRun: ${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ github.run_id }}\nFindings: ${{ github.server_url }}/${{ github.repository }}/security/code-scanning?query=is%3Aopen+tool%3ATrivy"
}
env:
SLACK_WEBHOOK_URL: ${{ secrets.SLACK_TRIVY_WEBHOOK_URL }}

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Which channel we are using?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

logging-integrations-notifications

… convention

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>

@maya-jha maya-jha 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.

Approved

@bpavan-nr bpavan-nr 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.

Approving to allow merge

@voorepreethi
voorepreethi merged commit 0659db3 into newrelic-forks:master Jul 20, 2026
2 of 3 checks passed
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.

4 participants