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

Security Policy violation Binary Artifacts #23

Closed
allstar-app bot opened this issue Nov 18, 2022 · 20 comments · Fixed by #24, #27 or sigstore/.allstar#10
Closed

Security Policy violation Binary Artifacts #23

allstar-app bot opened this issue Nov 18, 2022 · 20 comments · Fixed by #24, #27 or sigstore/.allstar#10
Labels

Comments

@allstar-app
Copy link

allstar-app bot commented Nov 18, 2022

This issue was automatically created by Allstar.

Security Policy Violation
Project is out of compliance with Binary Artifacts policy: binaries present in source code

Rule Description
Binary Artifacts are an increased security risk in your repository. Binary artifacts cannot be reviewed, allowing the introduction of possibly obsolete or maliciously subverted executables. For more information see the Security Scorecards Documentation for Binary Artifacts.

Remediation Steps
To remediate, remove the generated executable artifacts from the repository.

Artifacts Found

  • java/gradle/wrapper/gradle-wrapper.jar

Additional Information
This policy is drawn from Security Scorecards, which is a tool that scores a project's adherence to security best practices. You may wish to run a Scorecards scan directly on this repository for more details.


This issue will auto resolve when the policy is in compliance.

Issue created by Allstar. See https://github.com/ossf/allstar/ for more information. For questions specific to the repository, please contact the owner or maintainer.

@allstar-app allstar-app bot added the allstar label Nov 18, 2022
@haydentherapper
Copy link
Collaborator

@loosebazooka it's not happy about the jar

@loosebazooka
Copy link
Member

It should be okay if the gradle wrapper validation is applied

@loosebazooka
Copy link
Member

Maybe cause it's a condition run. I'll make a separate workflow run

@loosebazooka
Copy link
Member

Actually I think the logic checks if it has run successfully on the last commit or something. @ethanent maybe has the clarification, anyway lemme fix this up.

@ethan7g
Copy link

ethan7g commented Nov 19, 2022

Actually I think the logic checks if it has run successfully on the last commit or something. @ethanent maybe has the clarification, anyway lemme fix this up.

That's right iirc! Hopefully that fix works, but it does seem a little odd because the check containing the Action seems to be passing currently. We'll see!

@allstar-app
Copy link
Author

allstar-app bot commented Nov 20, 2022

Updating issue after ping interval. See its status below.


Project is out of compliance with Binary Artifacts policy: binaries present in source code

Rule Description
Binary Artifacts are an increased security risk in your repository. Binary artifacts cannot be reviewed, allowing the introduction of possibly obsolete or maliciously subverted executables. For more information see the Security Scorecards Documentation for Binary Artifacts.

Remediation Steps
To remediate, remove the generated executable artifacts from the repository.

Artifacts Found

  • java/gradle/wrapper/gradle-wrapper.jar

Additional Information
This policy is drawn from Security Scorecards, which is a tool that scores a project's adherence to security best practices. You may wish to run a Scorecards scan directly on this repository for more details.

@znewman01
Copy link
Contributor

We'll see if that worked...super annoying. Thanks all

@allstar-app allstar-app bot reopened this Nov 21, 2022
@allstar-app
Copy link
Author

allstar-app bot commented Nov 21, 2022

Reopening issue. See its status below.


Project is out of compliance with Binary Artifacts policy: binaries present in source code

Rule Description
Binary Artifacts are an increased security risk in your repository. Binary artifacts cannot be reviewed, allowing the introduction of possibly obsolete or maliciously subverted executables. For more information see the Security Scorecards Documentation for Binary Artifacts.

Remediation Steps
To remediate, remove the generated executable artifacts from the repository.

Artifacts Found

  • java/gradle/wrapper/gradle-wrapper.jar

Additional Information
This policy is drawn from Security Scorecards, which is a tool that scores a project's adherence to security best practices. You may wish to run a Scorecards scan directly on this repository for more details.

@loosebazooka
Copy link
Member

loosebazooka commented Nov 22, 2022

I hate you all star. Maybe matching of gradle wrapper is limited to the root? no that's not it.

It looks like the constraint checker only allows tagged versions? https://github.com/ossf/scorecard/blob/ef79b9487d8f8bf6fca7b0bafc8c55049d925403/checks/raw/binary_artifact.go#L248, hash matching versions are not checked in @ethanent ?

@loosebazooka
Copy link
Member

It looks like the constraint checker only allows tagged versions? https://github.com/ossf/scorecard/blob/ef79b9487d8f8bf6fca7b0bafc8c55049d925403/checks/raw/binary_artifact.go#L248, hash matching versions are not converted and compare @ethanent ?

@ethan7g
Copy link

ethan7g commented Nov 22, 2022

It looks like the constraint checker only allows tagged versions? https://github.com/ossf/scorecard/blob/ef79b9487d8f8bf6fca7b0bafc8c55049d925403/checks/raw/binary_artifact.go#L248, hash matching versions are not converted and compare @ethanent ?

Ah yeah, that seems right. I forgot that that limitation was present in the Scorecard implementation of Action comparison

@loosebazooka
Copy link
Member

so, maybe I'll just use version for now and add a TODO to change this.

@cpanato
Copy link
Member

cpanato commented Nov 22, 2022

i found this issue today 😓
we can add a exeption for this if this binary is needed for the test/ci or development workflow, please let me know then i can update the rules

@loosebazooka @znewman01

@znewman01
Copy link
Contributor

@cpanato: If it's easy—I'm inclined to add an exception for now until ossf/scorecard#2477 is fixed, rather than unpin the gradle-wrapper-validation action, which feels like a step backwards. That'd be great

@loosebazooka
Copy link
Member

I feel like I'd rather unpin the gradle/wrapper-validation-action. The exception in binary validation is for all binaries

@znewman01
Copy link
Contributor

Ahh okay I see. Yeah everything kinda sucks here. I'll approve your PR

@cpanato
Copy link
Member

cpanato commented Nov 22, 2022

will ping you both in the slack to conitnue and agree

@allstar-app
Copy link
Author

allstar-app bot commented Oct 18, 2023

Reopening issue. See its status below.


Project is out of compliance with Binary Artifacts policy: binaries present in source code

Rule Description
Binary Artifacts are an increased security risk in your repository. Binary artifacts cannot be reviewed, allowing the introduction of possibly obsolete or maliciously subverted executables. For more information see the Security Scorecards Documentation for Binary Artifacts.

Remediation Steps
To remediate, remove the generated executable artifacts from the repository.

Artifacts Found

  • java/gradle/wrapper/gradle-wrapper.jar

Additional Information
This policy is drawn from Security Scorecards, which is a tool that scores a project's adherence to security best practices. You may wish to run a Scorecards scan directly on this repository for more details.

@allstar-app allstar-app bot reopened this Oct 18, 2023
@loosebazooka
Copy link
Member

loosebazooka commented Oct 19, 2023

I think the wrapper validation script probably failed. weirdly not the case, this is a bit confusing

@allstar-app
Copy link
Author

allstar-app bot commented Oct 20, 2023

Updating issue after ping interval. See its status below.


Project is out of compliance with Binary Artifacts policy: binaries present in source code

Rule Description
Binary Artifacts are an increased security risk in your repository. Binary artifacts cannot be reviewed, allowing the introduction of possibly obsolete or maliciously subverted executables. For more information see the Security Scorecards Documentation for Binary Artifacts.

Remediation Steps
To remediate, remove the generated executable artifacts from the repository.

Artifacts Found

  • java/gradle/wrapper/gradle-wrapper.jar

Additional Information
This policy is drawn from Security Scorecards, which is a tool that scores a project's adherence to security best practices. You may wish to run a Scorecards scan directly on this repository for more details.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment