Skip to content

Add a workflow vulnerability scanner - #61

Merged
dkfellows merged 22 commits into
mainfrom
scan
Sep 3, 2026
Merged

Add a workflow vulnerability scanner#61
dkfellows merged 22 commits into
mainfrom
scan

Conversation

@dkfellows

@dkfellows dkfellows commented May 18, 2026

Copy link
Copy Markdown
Collaborator

This is work in progress...

This adds a workflow vulnerability scanner and fixes the issues it has found. Still to do:

@dkfellows dkfellows self-assigned this May 18, 2026
@dkfellows dkfellows added the enhancement New feature or request label May 18, 2026
@dkfellows dkfellows linked an issue May 18, 2026 that may be closed by this pull request
@github-advanced-security

Copy link
Copy Markdown

You are seeing this message because GitHub Code Scanning has recently been set up for this repository, or this pull request contains the workflow file for the Code Scanning tool.

What Enabling Code Scanning Means:

  • The 'Security' tab will display more code scanning analysis results (e.g., for the default branch).
  • Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results.
  • You will be able to see the analysis results for the pull request's branch on this overview once the scans have completed and the checks have passed.

For more information about GitHub Code Scanning, check out the documentation.

Comment thread .github/workflows/test-impl.yml Fixed
Comment thread .github/workflows/test-impl.yml Fixed
Comment thread .github/workflows/build-exes-impl.yml Fixed
Comment thread .github/workflows/build-exes-impl.yml Fixed
Comment thread .github/workflows/build-exes-impl.yml Fixed
Comment thread .github/workflows/build-exes-impl.yml Fixed
Comment thread .github/workflows/build-exes-impl.yml Fixed
Comment thread .github/workflows/build-exes-impl.yml Fixed
Comment thread .github/workflows/build-exes-impl.yml Fixed
Comment thread .github/workflows/build-exes-impl.yml Fixed
Comment thread .github/workflows/build-exes-impl.yml Fixed
Comment thread .github/workflows/build-exes-impl.yml Fixed
Comment thread .github/workflows/release-impl.yml Fixed
- name: Release
id: release
uses: softprops/action-gh-release@v3
uses: softprops/action-gh-release@b4309332981a82ec1c5618f44dd2e27cc8bfbfda # v3.0.0
Comment thread .github/workflows/test-impl.yml Fixed
Comment thread .github/workflows/test-impl.yml Fixed
dkfellows added 2 commits May 18, 2026 16:54
This needs a little care because what is being cached is tricky
@dkfellows

Copy link
Copy Markdown
Collaborator Author

All remaining issues are... arguably not problems, given that the suggested fixes make things worse in my eyes in the context of what is being done.

@dkfellows
dkfellows marked this pull request as ready for review May 18, 2026 16:06
@dkfellows
dkfellows requested a review from a team as a code owner May 18, 2026 16:06
Comment thread run-in-docker/action.yml Fixed
Comment thread run-in-docker/action.yml Fixed
Comment thread run-in-docker/action.yml Fixed
@dkfellows

Copy link
Copy Markdown
Collaborator Author

All remaining issues are ones I decline to fix. We need to test.

@dkfellows

Copy link
Copy Markdown
Collaborator Author

Tests with hpcflow now pass. I think the other workflows should be OK too (the main tricky bit is that the comparison action is using powershell, which has totally different syntax once you don't use direct substitution).

@dkfellows

Copy link
Copy Markdown
Collaborator Author

Test with matflow passes as well.

@dkfellows
dkfellows merged commit f13c8e6 into main Sep 3, 2026
2 checks passed
@dkfellows
dkfellows deleted the scan branch September 3, 2026 14:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Analyse for workflow weaknesses

2 participants