-
Notifications
You must be signed in to change notification settings - Fork 510
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
BUG: Apache 2.0 license not recognized #3333
Comments
@raghavkaul this seems to be from a gap in the GitLab client. It seems like there's a way of getting the license as part of the project endpoint (in the REST at least), not sure how easily extendible this is, or if the license type is return as any sort of standardized way: https://stackoverflow.com/a/74685995
|
Note: We don't have any good way of getting the SPDX info for local repos either, but that sounds harder since it's something we'd need to detect ourselves |
Yes, we'd need the license's SPDXId. For GitHub, we use the API: scorecard/clients/githubrepo/licenses.go Line 84 in e82d15d
We don't have an equivalent for GitLab, but for some reason it doesn't work to simply add scorecard/clients/gitlabrepo/licenses.go Line 73 in e82d15d
I can take a look further, assigning to self. Wrt. SPDX Id for local repos, can we do some kind of fuzzy matching on License name? |
Does the license scorecard/clients/gitlabrepo/project.go Lines 40 to 43 in e82d15d
|
Describe the bug
When having the Apache 2.0 license placed at the root of a GitLab repo, it is not recognized as an "FSF or OSI recognized license"
Reproduction steps
Steps to reproduce the behavior:
export SCORECARD_EXPERIMENTAL=1
export GITLAB_AUTH_TOKEN=<token>
scorecard --repo https://gitlab.com/<namespace>/<repo> --commit <commit id> --checks=License,Code-Review,Maintained,Security-Policy,Binary-Artifacts,Vulnerabilities,Pinned-Dependencies --show-details
Expected behavior
Getting a 10/10 rating with no complaints.
Additional context
Scorecard version: v4.11.0
Reproduced on: Debian 11.x x86 architecure, macOS Ventura ARM architecture
The text was updated successfully, but these errors were encountered: