Skip to content

chore(gha): remove benchmark github workflows#6322

Merged
mcollina merged 2 commits intomainfrom
sec-p
Sep 18, 2025
Merged

chore(gha): remove benchmark github workflows#6322
mcollina merged 2 commits intomainfrom
sec-p

Conversation

@Eomm
Copy link
Copy Markdown
Member

@Eomm Eomm commented Sep 18, 2025

Even tho these workflow are triggered manually, we must remove them 😢

This is not a farewell, to add them back, we need another workflow:

image

https://scorecard.dev/viewer/?uri=github.com%2Ffastify%2Ffastify

Checklist

@github-actions github-actions bot added the github actions Github actions related label Sep 18, 2025
@mcollina
Copy link
Copy Markdown
Member

Why?

@Eomm
Copy link
Copy Markdown
Member Author

Eomm commented Sep 18, 2025

This is the full documentation:
https://github.com/ossf/scorecard/blob/40576783fda6698350fcbbeaea760ff827433034/docs/checks.md#dangerous-workflow

Since we are checking out external code, a malicious user may run external code in our org GHA context.

While this is almost impossible because:

  • we must add a label to run the workflow
  • and the label is automatically removed after the benchmark run
  • SO, a user can't commit an un-checked code while we are adding the label

Nevertheless, this is considered a potential security threat and is flagged as critical by the OpenSSF working group.
Reading the documentation there is not a why to mark this record as "invalid" or "we know it but it can't happen"

Our scope is to reduce any potential issue to zero, even tho it may require some extra work to move the GHA into a repo that could be exploited without harm

@Fdawgs
Copy link
Copy Markdown
Member

Fdawgs commented Sep 18, 2025

Happy for them to be removed for now to reduce any immediate potential nastiness.

We can revisit them and do what is suggested in https://securitylab.github.com/resources/github-actions-preventing-pwn-requests/ and split the workflows.

Note that https://github.com/fastify/workflows/blob/main/.github/workflows/plugins-benchmark-pr.yml also has the same codeql alerts.

@Uzlopak Uzlopak changed the title chore(gha): removing benchmark scripts chore(gha): removing benchmark github workflows Sep 18, 2025
@Uzlopak Uzlopak changed the title chore(gha): removing benchmark github workflows chore(gha): remove benchmark github workflows Sep 18, 2025
Copy link
Copy Markdown
Contributor

@Uzlopak Uzlopak left a comment

Choose a reason for hiding this comment

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

LGTM

@mcollina mcollina merged commit 39d2e10 into main Sep 18, 2025
35 checks passed
@mcollina mcollina deleted the sec-p branch September 18, 2025 16:13
@Eomm Eomm mentioned this pull request Sep 21, 2025
2 tasks
Fdawgs added a commit to fastify/light-my-request that referenced this pull request Oct 18, 2025
mcollina pushed a commit to fastify/light-my-request that referenced this pull request Oct 18, 2025
@github-actions
Copy link
Copy Markdown

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 21, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

github actions Github actions related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants