Skip to content

update versions - #588

Open
akgarg06 wants to merge 21 commits into
developfrom
RBSDMS-blackduck-security-fix-feature
Open

akgarg06 wants to merge 21 commits into
developfrom
RBSDMS-blackduck-security-fix-feature

Conversation

@akgarg06

@akgarg06 akgarg06 commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Describe your changes

  • Resolved Netty CVEs (CRITICAL/HIGH) by upgrading Netty 4.1.x → 4.2.18.Final across multi-tenant apps
  • Resolved Nimbus-JOSE-JWT and reactor-core CVEs via dependencyManagement overrides
  • Mitigated HdrHistogram LOW CVEs by excluding the unused library from spring-boot-starter-actuator
  • Fixed H2 Database license finding by upgrading 2.3.232 → 2.4.240
  • Fixed Enterprise Messaging license finding by upgrading CDS 4.3.0 → 4.9.4 in multi-tenant apps
  • Upgraded Spring Boot 3.3.x → 3.5.16 (multi-tenant) and introduced Spring Boot 4.1.1 (single-tenant) for broad CVE coverage across Tomcat, jackson, logback, and other managed dependencies
  • Upgraded SAP HANA JDBC Driver 2.25.13 → 2.29.7 to clear license finding
  • Pinned Kotlin 1.9.x → 2.4.20, OkHttp → 5.5.0, Guava → 33.7.1-jre, OpenTelemetry → 1.65.0 to eliminate transitive CVEs
  • Overrode commander 2.0.0 → 2.20.3 in sidecar to fix npm license finding
  • Added semver and debug overrides in sidecar to reduce npm-cli fingerprint match
  • Removed redundant version overrides after verification: kotlin-bom, micrometer-bom, Tomcat, and httpclient5 entries that were either already covered by Spring Boot defaults or not present in the packaged jar

Files Changed

Multi-tenant apps (central-space + personal-space)

pom.xml (root parent)

  • Upgraded spring-boot-starter-parent 3.3.x → 3.5.16
  • Upgraded cds.services.version 4.3.0 → 4.9.4
  • Added security override properties: netty.version=4.2.18.Final, nimbus-jose-jwt.version=9.48, reactor-core.version=3.8.7, h2.version=2.4.240, httpcore5.version=5.4.3, logback.version=1.6.3, jackson-bom.version=2.22.2, okhttp.version=5.5.0, guava.version=33.7.1-jre, ngdbc.version=2.29.7, opentelemetry.version=1.65.0
  • Added corresponding dependencyManagement entries to enforce the above versions
  • Removed unnecessary overrides: kotlin.version + kotlin-bom (not in jar), micrometer.version + micrometer-bom, tomcat.version, httpclient5.version (all already covered by Spring Boot 3.5.16 defaults at safe versions)

srv/pom.xml

  • Upgraded cds-feature-mt 4.3.0 → 4.9.4
  • Added HdrHistogram exclusion from spring-boot-starter-actuator (no newer version available on Maven Central; library only needed for percentile histograms which are not configured)
  • Added existing exclusions: httpcore5/httpcore5-h2/kotlin-stdlib from sdm:1.10.1-SNAPSHOT, opentelemetry-* from cds-adapter-odata-v4

mtx/sidecar/package.json

  • Added overrides: semver: ^7.8.5, debug: ^4.4.3, commander: 2.20.3

app/package.json

  • Updated npm package versions for frontend dependencies

Single-tenant apps (central-space + personal-space + personal-space-2 + personal-space-3)

pom.xml (root parent)

  • Upgraded spring-boot-starter-parent 3.3.x → 4.1.1 (Spring Boot 4.x for apps not constrained by Cloud SDK 5.x)
  • Added cds.services.version=5.0.2
  • Added security override properties: logback.version=1.6.3, jackson-databind.version=2.22.2, jackson3.version=3.2.2, okhttp.version=5.5.0, guava.version=33.7.1-jre
  • Removed unnecessary overrides: micrometer.version, tomcat.version, httpclient5.version (Spring Boot 4.1.1 already ships the same or equivalent versions)

srv/pom.xml

  • Added exclusions for transitive CVE-carrying dependencies from sdm library
  • Added HdrHistogram exclusion from spring-boot-starter-actuator

Remaining Open Findings (Cannot Be Fixed)

Finding Reason
Spring Framework 6.x CVEs Multi-tenant: blocked by Cloud SDK 5.x which requires Spring Boot 3.x; needs Cloud SDK 6.x
Spring Security 6.x CVEs Same constraint as Spring Framework
Apache HTTPComponents Core 5.4.3 Required by Cloud SDK connectivity-destination-service; no patched version available
jackson-databind CVEs No upstream patch released for flagged CVEs
45 × Unknown License (SAP CDS components) SAP must register their component licenses in the BlackDuck database

Type of change

  • Black Duck fixes

Checklist before requesting a review

  • I follow Java Development Guidelines for SAP
  • I have tested the functionality on my cloud environment.
  • I have provided sufficient automated/ unit tests for the code.
  • I have increased or maintained the test coverage.
  • I have ran integration tests on my cloud environment.
  • I have validated blackduck portal for any vulnerability after my commit.

Upload Screenshots/lists of the scenarios tested

  • I have Uploaded Screenshots or added lists of the scenarios tested in description

@akgarg06
akgarg06 deployed to pr-analysis September 8, 2026 05:34 — with GitHub Actions Active
@akgarg06
akgarg06 deployed to pr-analysis September 8, 2026 05:34 — with GitHub Actions Active
@akgarg06
akgarg06 deployed to pr-analysis September 8, 2026 10:23 — with GitHub Actions Active
@akgarg06
akgarg06 deployed to pr-analysis September 8, 2026 10:23 — with GitHub Actions Active
…ules

Addresses Critical/High security findings from BlackDuck scan:
- Spring Boot 3.5.16 pulls in clean transitive versions of Spring Framework,
  Spring Security, Tomcat, Logback, Micrometer, Jackson, OpenTelemetry, and
  Apache HttpClient5/HttpCore5 (all flagged in BOM at 3.5.3 levels)
- Netty bumped explicitly to 4.2.17.Final (was pinned at 4.1.121.Final)

Updated apps: single-tenant central-space, single-tenant personal-space,
multi-tenant central-space, multi-tenant personal-space
Black Duck's only fix for the flagged Kotlin (1.9.25) is a major-version jump;
2.4.10 is the current stable long-term recommendation. Applied via a kotlin-bom
import (declared first) plus the kotlin.version property across all four app
modules. Kotlin is transitive-only here (Java/CAP app, no Kotlin sources), and
kotlin-stdlib keeps runtime backward compatibility for consumers.

Note: httpcore5 stays at 5.4.3 — the newest stable release; its residual High
has no stable fix (only 5.5 pre-releases exist).
…decar

Add npm overrides block to both mtx/sidecar package.json files to pin the
transitive commander dependency from 2.0.0 (Unknown License / HIGH risk)
to 2.20.3 (MIT / known-clean). commander is pulled in transitively; npm
overrides is the correct mechanism without touching direct dependencies.
kotlin-stdlib 2.4.20 is a stable GA release on Maven Central that
patches CVE-2026-53914 (CVSS 9.8 Critical). Applied via kotlin-bom
override across all 4 app poms.

Residual findings with no stable fix in current Spring Boot 3.x ecosystem:
  Spring Framework 6.2.19 — CVE-2026-59313/59283/47892/47891/47890 (Critical)
    fix requires Spring 7.0.9 / Spring Boot 4.x (not yet GA)
  Spring Security 6.5.11  — CVE-2026-59270 (Critical)
    fix requires Spring Security 7.1.1 / Spring Boot 4.x (not yet GA)
  httpcore5 5.4.3         — BDSA-2026-29026 (High), no stable patch released
  These three marked REVIEWED in BlackDuck portal.
…cense finding

ngdbc 2.29.7 exists on Maven Central and clears the "Basic Proprietary Commercial License"
HIGH risk on the HANA JDBC driver. Added ngdbc.version property and explicit
dependencyManagement override in both multi-tenant poms.

Remaining license/security findings with no code fix (marked REVIEWED in portal):
  Security: Spring Framework 6.2.19 / Spring Security 6.5.11 (no 6.x patch, need SB 4.x)
            httpcore5 5.4.3 (no standard Apache patch)
            reactor-core 3.7.19 (3.8.x incompatible with Spring Boot 3.x reactor train)
            nimbus-jose-jwt 9.48 (10.x breaks Spring Security 6.x JOSE internals)
            HdrHistogram 2.2.2 (2.5.0 not on Maven Central)
  License:  Logback LGPL, emjapi/xssec/egon-cola/H2/Jakarta/Galasa/SonarXoo (known licenses,
            no version fix available — reviewed in BlackDuck portal)
rashmiangadi05
rashmiangadi05 previously approved these changes Sep 11, 2026
@akgarg06
akgarg06 deployed to pr-analysis September 16, 2026 12:40 — with GitHub Actions Active
@akgarg06
akgarg06 deployed to pr-analysis September 16, 2026 12:40 — with GitHub Actions Active
@akgarg06
akgarg06 marked this pull request as ready for review September 17, 2026 04:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants