-
Notifications
You must be signed in to change notification settings - Fork 1.2k
[StepSecurity] Apply security best practices #1008
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
base: develop
Are you sure you want to change the base?
Changes from 1 commit
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,6 @@ | ||
| version: 2 | ||
| updates: | ||
| - package-ecosystem: github-actions | ||
| directory: / | ||
| schedule: | ||
| interval: daily |
| Original file line number | Diff line number | Diff line change | ||||||||
|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -29,6 +29,11 @@ jobs: | |||||||||
| timeout-minutes: 35 | ||||||||||
| runs-on: Roboflow-GPU-VM-Runner | ||||||||||
| steps: | ||||||||||
| - name: Harden the runner (Audit all outbound calls) | ||||||||||
| uses: step-security/harden-runner@8d3c67de8e2fe68ef647c8db1e6a09f647780f40 # v2.19.0 | ||||||||||
| with: | ||||||||||
| egress-policy: audit | ||||||||||
|
|
||||||||||
|
Comment on lines
+32
to
+36
|
||||||||||
| - name: Harden the runner (Audit all outbound calls) | |
| uses: step-security/harden-runner@8d3c67de8e2fe68ef647c8db1e6a09f647780f40 # v2.19.0 | |
| with: | |
| egress-policy: audit |
| Original file line number | Diff line number | Diff line change | ||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| @@ -0,0 +1,78 @@ | ||||||||||||||||||
| # For most projects, this workflow file will not need changing; you simply need | ||||||||||||||||||
| # to commit it to your repository. | ||||||||||||||||||
| # | ||||||||||||||||||
| # You may wish to alter this file to override the set of languages analyzed, | ||||||||||||||||||
| # or to provide custom queries or build logic. | ||||||||||||||||||
| # | ||||||||||||||||||
| # ******** NOTE ******** | ||||||||||||||||||
| # We have attempted to detect the languages in your repository. Please check | ||||||||||||||||||
| # the `language` matrix defined below to confirm you have the correct set of | ||||||||||||||||||
| # supported CodeQL languages. | ||||||||||||||||||
| # | ||||||||||||||||||
| name: "CodeQL" | ||||||||||||||||||
|
|
||||||||||||||||||
| on: | ||||||||||||||||||
| push: | ||||||||||||||||||
| branches: ["develop"] | ||||||||||||||||||
| pull_request: | ||||||||||||||||||
| # The branches below must be a subset of the branches above | ||||||||||||||||||
| branches: ["develop"] | ||||||||||||||||||
|
Comment on lines
+16
to
+19
|
||||||||||||||||||
| branches: ["develop"] | |
| pull_request: | |
| # The branches below must be a subset of the branches above | |
| branches: ["develop"] | |
| branches: ["develop", "main", "release/*"] | |
| pull_request: | |
| # The branches below must be a subset of the branches above | |
| branches: ["develop", "main", "release/*"] |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,27 @@ | ||
| # Dependency Review Action | ||
| # | ||
| # This Action will scan dependency manifest files that change as part of a Pull Request, | ||
| # surfacing known-vulnerable versions of the packages declared or updated in the PR. | ||
| # Once installed, if the workflow run is marked as required, | ||
| # PRs introducing known-vulnerable packages will be blocked from merging. | ||
| # | ||
| # Source repository: https://github.com/actions/dependency-review-action | ||
| name: 'Dependency Review' | ||
| on: [pull_request] | ||
|
|
||
| permissions: | ||
| contents: read | ||
|
|
||
| jobs: | ||
| dependency-review: | ||
| runs-on: ubuntu-latest | ||
| steps: | ||
| - name: Harden the runner (Audit all outbound calls) | ||
| uses: step-security/harden-runner@8d3c67de8e2fe68ef647c8db1e6a09f647780f40 # v2.19.0 | ||
| with: | ||
| egress-policy: audit | ||
|
|
||
| - name: 'Checkout Repository' | ||
| uses: actions/checkout@v4 | ||
| - name: 'Dependency Review' | ||
| uses: actions/dependency-review-action@v4 |
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
| @@ -0,0 +1,81 @@ | ||||||
| # This workflow uses actions that are not certified by GitHub. They are provided | ||||||
| # by a third-party and are governed by separate terms of service, privacy | ||||||
| # policy, and support documentation. | ||||||
|
|
||||||
| name: Scorecard supply-chain security | ||||||
| on: | ||||||
| # For Branch-Protection check. Only the default branch is supported. See | ||||||
| # https://github.com/ossf/scorecard/blob/main/docs/checks.md#branch-protection | ||||||
| branch_protection_rule: | ||||||
| # To guarantee Maintained check is occasionally updated. See | ||||||
| # https://github.com/ossf/scorecard/blob/main/docs/checks.md#maintained | ||||||
| schedule: | ||||||
| - cron: '20 7 * * 2' | ||||||
| push: | ||||||
| branches: ["develop"] | ||||||
|
||||||
| branches: ["develop"] | |
| branches: ["develop", "main", "release/*"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
step-security/harden-runneris added to a job matrix that runs on Windows and macOS as well as Ubuntu. The harden-runner action is typically Linux-specific (it relies on runner hardening/egress controls) and may fail or be a no-op on non-Linux runners; consider gating this step with anif: runner.os == 'Linux'(or moving hardening into a Linux-only job) so the Windows/macOS legs don’t break.