Skip to content

Fix potential security issues in GitHub actions - #4838

Merged
juliette-derancourt merged 3 commits into
mainfrom
issues/4558
Aug 15, 2025
Merged

Fix potential security issues in GitHub actions#4838
juliette-derancourt merged 3 commits into
mainfrom
issues/4558

Conversation

@juliette-derancourt

@juliette-derancourt juliette-derancourt commented Aug 15, 2025

Copy link
Copy Markdown
Member

I ran zizmor static analysis tool as suggested in #4558 and found several warnings/errors.

Some of them are addressed in this PR:


I hereby agree to the terms of the JUnit Contributor License Agreement.

@marcphilipp marcphilipp left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for taking care of this! 👍

Is there a way of running this check continuously, e.g. as a GitHub Actions workflow?

Comment thread .github/workflows/release.yml Outdated
@juliette-derancourt

juliette-derancourt commented Aug 15, 2025

Copy link
Copy Markdown
Member Author

This usage is also found as an error, but given the nature of the input (an optional string with \ characters), it does not work with this "naive" replacement:

      env:
        ARGUMENTS: ${{ inputs.arguments }}
      run: |
        ./gradlew \
        ... \
        ${ARGUMENTS}

Since the input value comes from a workflow, I think we can leave it as it is?

@juliette-derancourt

Copy link
Copy Markdown
Member Author

Is there a way of running this check continuously, e.g. as a GitHub Actions workflow?

@marcphilipp I was wondering about that as well, but I guess we would need some mechanism to say "this is ignored intentionally" for that to be possible (I haven't checked that yet, will do)

Comment thread .github/workflows/main.yml Outdated
Comment thread .github/workflows/release.yml
Comment thread .github/workflows/release.yml Outdated
@juliette-derancourt

Copy link
Copy Markdown
Member Author

I guess we would need some mechanism to say "this is ignored intentionally" for that to be possible

Seems like there is a way, and a github action as well. I'll add that to #4558 deliverables and set that up in another PR.

@juliette-derancourt
juliette-derancourt force-pushed the issues/4558 branch 2 times, most recently from 21fdb5b to 8216b93 Compare August 15, 2025 16:57
@juliette-derancourt
juliette-derancourt marked this pull request as ready for review August 15, 2025 16:58
@juliette-derancourt
juliette-derancourt merged commit 190716b into main Aug 15, 2025
14 checks passed
@juliette-derancourt
juliette-derancourt deleted the issues/4558 branch August 15, 2025 17:57
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