-
-
Notifications
You must be signed in to change notification settings - Fork 171
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
detection of license for ua-parser-js is MIT instead of AGPL-3.0-or-later #1324
Comments
Interesting! It is taking the last data from npmjs instead of package.json. In enterprise environments, most package.json files might have Unlicense or some other unreliable id, so we generally do not rely on that file. Is this a matter of having a new release with the new license? Also, I am unsure if the existing published versions could be retroactively changed from MIT, but I am not a lawyer. |
The existing published is indeed MIT (@prabhu thank you for spotting that). In this case we are more interested in the repository as the source of truth. For example assume the repo Update: Looks like even with |
The repository is unreleased main branch mostly, while the license data in the sbom is specific to the released version. Have you tried scancode to see if it returns uptodate information? |
I think the difference is app/project level vs. third party dependency. In the root of a repo the app/project doesn't need to use the registry to get info as it may not yet be published, where as it's third party dependencies and transitives are. For app/project stuff I don't think the default should be to look to the registry and instead should look to its files available. |
When cloning https://github.com/faisalman/ua-parser-js/
then
and running
I get a generated sbom that says the license is MIT but the license is AGPL-3.0-or-later
The text was updated successfully, but these errors were encountered: