[FIPS] connectivity: jks-js pulls non-FIPS crypto (MD5, SHA-1, DES) into all consumers

Aperta
#6,959 2 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

@davidkna-sap ci sta già lavorando.

Dal 30/10/2026.

  • #7061 di @davidkna-sap — aperta

Valutazione

Difficoltà
5/5
Tempo stimato
Più di una settimana
Idoneità per principianti
35/100
Tipo di issue
Refactoring
Chiarezza
Da chiarire
Stato di attività
Attiva
Stack tecnologico
typescript
Ambito
backend, security

Direzione di ricerca

Iniziate esaminando la catena di dipendenze di @sap-cloud-sdk/connectivity e il supporto per JKS introdotto in #6153, incluso il commit ab96aff. Confrontate la sostituzione di jks-js con un parser compatibile con FIPS con la possibilità di rendere facoltativo il supporto per JKS, quindi confermate la direzione preferita e definite il completamento come la rimozione dei pacchetti non compatibili con FIPS dai consumer che non utilizzano JKS.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

feature request

Summary

@sap-cloud-sdk/connectivity unconditionally depends on jks-js, which brings node-forge and node-rsa into the production dependency tree of every consumer. These three packages implement non-FIPS algorithms (MD5, SHA-1, DES, 3DES, AES-ECB, RIPEMD-160, RSA+MD5, RSA+SHA-1) in pure JavaScript, bypassing the OpenSSL FIPS provider. Any consumer that must pass a FIPS compliance scan is blocked.

Dependency chain

@sap-cloud-sdk/[email protected]
  → [email protected]          (MD5, SHA-1, DES via node-forge)
    → [email protected]    (MD5, SHA-1, DES, 3DES, AES-ECB, RIPEMD-160)
    → [email protected]      (RSA+MD5, RSA+SHA-1)

jks-js was introduced in #6153 (commit ab96aff) to add JKS keystore support. Verified against latest @sap-cloud-sdk/[email protected] — still present.

Impact

Consumers that only use PEM certificates never reach the JKS code path at runtime, but the non-FIPS packages remain in node_modules and are flagged by FIPS compliance scanners. Example: a consumer using ClientCertificateAuthentication with type: 'pem' exclusively — no JKS use at runtime — is still blocked.

Suggested fix

Escalate to @sap-cloud-sdk maintainers to either:

  1. Replace jks-js with a FIPS-compatible JKS keystore parser, or
  2. Make JKS support opt-in so services that don't use it avoid the dependency entirely.

Ask

  • Confirm which direction the team prefers.
  • Provide a rough timeline so downstream consumers can plan.

cc: @davidkna-sap @KavithaSiva @emincihangeri

Lingua principale
TypeScript
Stelle
221
Fork
79
Merge medio
13h 27m
PR unite (30g)
22

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di SAP/cloud-sdk-js

Tutte le issue di SAP/cloud-sdk-js

Issue simili

Altre issue su TypeScript

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.