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

[Gradle] Resolve gradle module from npm package #1368

Merged
merged 4 commits into from
Sep 10, 2024

Conversation

malice00
Copy link
Contributor

@malice00 malice00 commented Sep 9, 2024

When using Expo, several of the gradle modules are actually references to code in npm packages. Unfortunately the purls that are generated from this, are not known by vuln-sources like OSS Index.
This PR can resolve the module to the actual npm package so you get correct data on your components regarding vulns and licenses.

@prabhu
Copy link
Contributor

prabhu commented Sep 9, 2024

Interesting idea! Could you kindly add some repo tests?

@malice00
Copy link
Contributor Author

malice00 commented Sep 9, 2024

I knew I forgot something! 😜 I'll add some, just not sure if they'll work in the build -- the previous ones still don 't work, I think... 😞

- Removed CDXGEN_DEBUG
- Removed non-existing modules from skip

Signed-off-by: Roland Asmann <[email protected]>
@prabhu
Copy link
Contributor

prabhu commented Sep 9, 2024

@aryan-rajoria manually tested the last PR. Even if the test doesn't work in the CI we can use the repos in our manual tests

@malice00
Copy link
Contributor Author

malice00 commented Sep 9, 2024

Darn, I just figured out why the tests don't work -- I made a typo and obviously copied that in all 3 calls! 😭 Should be fixed now.

@prabhu prabhu merged commit 92e1dee into CycloneDX:master Sep 10, 2024
20 of 21 checks passed
@prabhu
Copy link
Contributor

prabhu commented Sep 10, 2024

Thank you so much for a great contribution! Please keep going!

@malice00 malice00 deleted the fix/npm_gradle branch September 10, 2024 08:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants