Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
a) add license/Copyright header, b) add missing TopLevel Security (on…
…nx#6184) ### Description <!-- - Describe your changes. --> ### Motivation and Context The current analysis done by "openssf scorecard", which could be found at https://api.securityscorecards.dev/projects/github.com/onnx/onnx displays the following points: ``` { "name": "Token-Permissions", "score": 0, "reason": "detected GitHub workflow tokens with excessive permissions", "details": [ "Warn: jobLevel 'contents' permission set to 'write': .github/workflows/auto_update_doc.yml:15", "Info: jobLevel 'actions' permission set to 'read': .github/workflows/codeql.yml:35", "Info: jobLevel 'contents' permission set to 'read': .github/workflows/codeql.yml:36", "Warn: jobLevel 'checks' permission set to 'write': .github/workflows/main.yml:204", "Warn: no topLevel permission defined: .github/workflows/auto_update_doc.yml:1", "Warn: no topLevel permission defined: .github/workflows/check_urls.yml:1", "Info: topLevel 'contents' permission set to 'read': .github/workflows/clang_tidy_review.yml:12", "Warn: topLevel 'checks' permission set to 'write': .github/workflows/clang_tidy_review_post.yml:12", "Info: topLevel 'contents' permission set to 'read': .github/workflows/codeql.yml:24", "Warn: no topLevel permission defined: .github/workflows/dco_merge_group.yml:1", "Info: topLevel 'contents' permission set to 'read': .github/workflows/lint.yml:15", "Info: topLevel 'contents' permission set to 'read': .github/workflows/main.yml:18", "Info: topLevel 'contents' permission set to 'read': .github/workflows/pages.yml:12", "Info: topLevel 'contents' permission set to 'read': .github/workflows/release_linux_aarch64.yml:14", "Info: topLevel 'contents' permission set to 'read': .github/workflows/release_linux_x86_64.yml:14", "Info: topLevel 'contents' permission set to 'read': .github/workflows/release_mac.yml:18", "Info: topLevel 'contents' permission set to 'read': .github/workflows/release_win.yml:18", "Info: topLevel 'contents' permission set to 'read': .github/workflows/reuse.yml:10", "Info: topLevel permissions set to 'read-all': .github/workflows/scorecard.yml:22", "Info: topLevel 'contents' permission set to 'read': .github/workflows/stale.yml:17", "Info: topLevel 'contents' permission set to 'read': .github/workflows/weekly_mac_ci.yml:18", "Info: topLevel 'contents' permission set to 'read': .github/workflows/win_no_exception_ci.yml:14" ], "documentation": { "short": "Determines if the project's workflows follow the principle of least privilege.", "url": "https://github.com/ossf/scorecard/blob/7ce8609469289d5f3b1bf5ee3122f42b4e3054fb/docs/checks.md#token-permissions" } ``` This pull request handles the warnings: `"Warn: no topLevel permission defined" ` in order to improve security and get a higher score ;-) This change is also recommended by: https://github.com/ossf/scorecard/blob/7ce8609469289d5f3b1bf5ee3122f42b4e3054fb/docs/checks.md#token-permissions --------- Signed-off-by: Andreas Fehlner <[email protected]>
- Loading branch information