Skip to content

Add centralized Snyk security scanning#10

Merged
smartinellibenedetti merged 2 commits intomasterfrom
snyk-scan
Oct 3, 2025
Merged

Add centralized Snyk security scanning#10
smartinellibenedetti merged 2 commits intomasterfrom
snyk-scan

Conversation

@fdevans
Copy link
Contributor

@fdevans fdevans commented Sep 16, 2025

This PR adds the centralized Snyk security scanning workflow to enable automated vulnerability detection.

Changes

  • Added .github/workflows/snyk-scan.yml workflow
  • Configured to run on push and pull requests to main/master branches
  • Uses the reusable workflow from rundeck-plugins/.github repository

Benefits

  • Automated security vulnerability scanning
  • Consistent security configuration across all plugins
  • Early detection of security issues

Testing

The workflow will be triggered automatically on:

  • Push to main/master branches
  • Pull requests to main/master branches
  • Manual workflow dispatch

Dependencies

  • Requires SNYK_TOKEN and SNYK_ORG_ID secrets to be configured in the repository

Copilot AI review requested due to automatic review settings October 2, 2025 22:38
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Introduces a reusable Snyk security scanning workflow to automate vulnerability detection across pushes, PRs, and manual dispatches.

  • Adds GitHub Actions workflow invoking centralized reusable Snyk scan
  • Configures triggers for main/master branches and manual dispatch
  • Passes required Snyk secrets to the reusable workflow

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment on lines +1 to +15
name: Snyk Scan

on:
push:
branches: [ main, master ]
pull_request:
branches: [ main, master ]
workflow_dispatch:

jobs:
security:
uses: rundeck-plugins/.github/.github/workflows/snyk-scan-reusable.yml@main
secrets:
SNYK_TOKEN: ${{ secrets.SNYK_TOKEN }}
SNYK_ORG_ID: ${{ secrets.SNYK_ORG_ID }} No newline at end of file
Copy link

Copilot AI Oct 2, 2025

Choose a reason for hiding this comment

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

The workflow does not declare an explicit permissions block, so it will inherit the repository's default GITHUB_TOKEN permissions; explicitly scoping permissions helps follow least-privilege principles. Consider adding a minimal permissions section (e.g., 'contents: read' and 'security-events: write' if the reusable workflow uploads SARIF) above 'jobs:' to reduce risk of unintended token capabilities.

Copilot uses AI. Check for mistakes.
@smartinellibenedetti smartinellibenedetti merged commit 14c220e into master Oct 3, 2025
2 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.

2 participants