-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
[Feature]: Improvement of OpenSSF Scorecard Score #5815
Labels
Comments
@harshitasao thank you for assistance. I have no objection to these items, except for fuzzing. |
Sure, I will make the PRs ASAP. |
yurishkuro
pushed a commit
that referenced
this issue
Aug 18, 2024
## Which problem is this PR solving? fixed the pinned dependencies issue reported by [scorecard](https://scorecard.dev/viewer/?uri=github.com/jaegertracing/jaeger). Part of #5815 ## Description of the changes - pinned the github actions by hash. ## How was this change tested? - ## Checklist - [X] I have read https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md - [X] I have signed all commits - [ ] I have added unit tests for the new functionality - [ ] I have run lint and test steps successfully - for `jaeger`: `make lint test` - for `jaeger-ui`: `yarn lint` and `yarn test` Signed-off-by: harshitasao <harshitasao@gmail.com>
This was referenced Aug 22, 2024
JaredTan95
pushed a commit
to JaredTan95/jaeger
that referenced
this issue
Aug 28, 2024
## Which problem is this PR solving? fixed the pinned dependencies issue reported by [scorecard](https://scorecard.dev/viewer/?uri=github.com/jaegertracing/jaeger). Part of jaegertracing#5815 ## Description of the changes - pinned the github actions by hash. ## How was this change tested? - ## Checklist - [X] I have read https://github.com/jaegertracing/jaeger/blob/master/CONTRIBUTING_GUIDELINES.md - [X] I have signed all commits - [ ] I have added unit tests for the new functionality - [ ] I have run lint and test steps successfully - for `jaeger`: `make lint test` - for `jaeger-ui`: `yarn lint` and `yarn test` Signed-off-by: harshitasao <harshitasao@gmail.com> Signed-off-by: Jared Tan <jian.tan@daocloud.io>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Requirement
Hi, I'm Harshita. I’m working with CNCF and the Google Open Source Security Team for the GSoC 2024 term. We are collaborating to enhance security practices across various CNCF projects. The goal is to improve security for all CNCF projects by both using OpenSSF Scorecards and implementing its security improvements.
Problem
As this project already has scorecard action, I'm here to increase the final score by going over each check. I've listed all of the checks where work needs to be done, in order of its criticality. I plan to submit each PR for each fix. Please let me know what you think and for which ones a PR is welcome that I will submit it ASAP.
Proposal
Current Score: 8.1
Scorecard report: https://scorecard.dev/viewer/?uri=github.com/jaegertracing/jaeger
Here's a few checks we can work on to improve the project's security posture:
Token-Permissions: Score = 0
Signed-Releases: Score = 8
Vulnerabilities: Score = 9
Affected package: github.com/aws/aws-sdk-go (https://osv.dev/vulnerability/GO-2022-0646)(Already fixed)osv-scanner.toml
to mark some of these as not impacting/ignored.Pinned-Dependencies: Score = 6
Fuzzing: Score = 0Integrating the project with OSS-Fuzz by following the instructions here. The most difficult one on the list, maintainers help, is highly appreciated./cc @joycebrum @diogoteles08 @pnacht @nate-double-u
Open questions
No response
The text was updated successfully, but these errors were encountered: