fix(ci): explicit OWASP data.directory + disable irrelevant analyzers - #3
Merged
Conversation
- data.directory now points at the shared, build-tool-agnostic path (~/.dependency-check-data) that reusable-owasp-check.yml caches and restores the centralized nova-devops NVD mirror into. The plugin's implicit default was never verified/documented; previous cache sizes (15-57MB) strongly suggest it did not match what was being cached. - Disabled analyzers for ecosystems that plainly do not exist in this repo (assembly/nuspec/nugetconf/msbuild/.NET, golang, swift, cocoapods, composer/PHP, cpan/Perl, cmake/autoconf, bundleAudit/rubygems, python, dart, retirejs) - zero detection-feature cost since none of these file types are present. Node/nodeAudit analyzers are NOT disabled (package.json is real - commitlint/lefthook devDependencies). - Verified property names against the actually-installed 12.2.2 Gradle plugin jar (javap on AnalyzerExtension.class) rather than assumed from docs - e.g. carthageAnalyzerEnabled does not exist in this version. See docs/java/06-semantic-versioning-en-java.md for the investigation into why OWASP scans were taking 30-50+ min.
End-to-end validation confirmed the same 4 CVEs affect all Nova libraries that use the shared Gradle plugin set (jgit/grgit transitives): - CVE-2026-54428 (7.5) and CVE-2026-54399 (7.5) - DoS in httpcore/httpcore5 - CVE-2025-67030 (8.8) - RCE in plexus-utils - CVE-2025-48734 (8.8) - RCE in commons-beanutils Two layers of fix (same as mask-utils commit f133fa2): 1. scanConfigurations = [compileClasspath, runtimeClasspath] Restricts analysis to what propagates to consumers. 2. resolutionStrategy.eachDependency constraints Patches the vulnerable deps to their CVE-free minimums. Verified locally: 'gradlew dependencyCheckAnalyze' reports 0 vulnerabilities.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Qu\u00e9
Aplica la investigaci\u00f3n de performance de OWASP documentada en docs/java/06-semantic-versioning-en-java.md a este repo:
Por qu\u00e9
OWASP estaba tomando 30-50+ min por PR. Causa ra\u00edz: el cache de GitHub Actions est\u00e1 scoped por rama/PR y nunca se compart\u00eda entre ramas nuevas, forzando un sync completo (~365k registros NVD) en cada PR. Fix completo (cache + mirror centralizado) vive en reusable-owasp-check.yml (nova-devops).
Validaci\u00f3n