Skip to content
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

Workflows are referencing vulnerable actions #1

Closed
igibek opened this issue Jan 30, 2022 · 1 comment
Closed

Workflows are referencing vulnerable actions #1

igibek opened this issue Jan 30, 2022 · 1 comment

Comments

@igibek
Copy link

igibek commented Jan 30, 2022

Hello, there!

As part of the university research we are currently doing regarding the security of Github Actions, we noticed that one or many of the workflows that are part of this repository are referencing vulnerable versions of the third-party actions. As part of a disclosure process, we decided to open issues to notify GitHub Community.

Please note that there are could be some false positives in our methodology, thus not all of the open issues could be valid. If that is the case, please let us know, so that we can improve on our approach. You can contact me directly using an email: ikoishy [at] ncsu.edu

Thanks in advance

  1. The workflow run-tests.yml is referencing action shivammathur/setup-php using references v1. However this reference is missing the commit 7163319 which may contain fix to the vulnerability.

The vulnerability fix that is missing by actions' versions could be related to:
(1) CVE fix
(2) upgrade of vulnerable dependency
(3) fix to secret leak and others.
Please consider updating the reference to the action.

If you end up updating the reference, please let us know. We need the stats for the paper :-)

@igibek
Copy link
Author

igibek commented Feb 3, 2022

Hello, there!

It turns out to be a false positive. The aforementioned version v1 actually contains the commit which was cherry-picked to include it, here: 11e062bf. Unfortunately, this commit was missed by our tool.

Sorry for the incorrect reporting.

I am closing this issue because it is safe to use the shivammathur/setup-php@v1 in the workflows.

Have a great day!

@igibek igibek closed this as completed Feb 3, 2022
This issue was closed.
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

No branches or pull requests

1 participant