Skip to content

fix(deps): node-forge 1.4.2 + Audit-Allowlist (decode-uri-component) - #374

Merged
S540d merged 2 commits into
testingfrom
fix/audit-node-forge-2026-11
Nov 3, 2026
Merged

S540d merged 2 commits into
testingfrom
fix/audit-node-forge-2026-11

Conversation

@S540d

@S540d S540d commented Nov 3, 2026 •

Copy link
Copy Markdown
Owner

Neue High-Advisories seit 2026-10-29 ließen das Audit-Gate scheitern und blockierten den Release-PR #375.

  • node-forge 1.4.0 → 1.4.2 (nur Build-Tooling via expo -> @expo/cli), gezielt per npm update node-forge (6 Zeilen Lockfile-Diff statt ~1.400 bei npm audit fix). Behebt GHSA-7wch-p9px-956x, GHSA-jc66-7q78-782g, GHSA-8hrf-8g6f-7xwp, GHSA-hqq3-7jhf-c5jx, GHSA-5x38-2vff-x55p und das bisher zugelassene GHSA-86w9-cpqp-85rv (Allowlist-Eintrag entfernt).
  • GHSA-qr28-fpjm-wq7f (decode-uri-component via expo-router -> query-string@7) befristet bis 2026-12-31 zugelassen (Entscheidung des Nutzers). Ausnahme von der Allowlist-Regel: Das Paket liegt im App-Bundle. Der Fix erfordert decode-uri-component 0.5.1 (ESM-only, inkompatibel) bzw. expo-router 58 (SDK-Major). Das Risiko ist gering, weil nur clientseitiges URL-Parsen betroffen ist. Mit dem nächsten SDK-Upgrade auflösen.

Audit-Gate lokal: ✅ (2 akzeptierte, 0 offene high+). Tests: 482 grün.

🤖 Generated with Claude Code

Behebt GHSA-7wch-p9px-956x, GHSA-jc66-7q78-782g, GHSA-8hrf-8g6f-7xwp,
GHSA-hqq3-7jhf-c5jx, GHSA-5x38-2vff-x55p. Nur Build-Tooling
(expo -> @expo/cli), gezielt per npm update statt audit fix.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions

github-actions Bot commented Nov 3, 2026 •

Copy link
Copy Markdown

🔀 Mergeability-Report

✅ Konfliktfrei mergebar (unstable).
✅ Ziel-Branch testing ist policy-konform.

CI-Checks: ✅ keine fehlgeschlagenen Checks

⏳ mergeability / mergeability
⏳ ⚙️ Workflow Linting / actionlint
⏳ 🔧 Platform Compatibility
⏳ 🔍 Code Quality & Linting
⏳ 🔒 Security Audit
⏳ 🔐 Docs Privacy Check
⏳ 🔑 Keystore & Credential Scan
⏳ Analyze (actions)
⏳ Analyze (javascript-typescript)

Merge-Checkliste

  • CI-Checks grün
  • Keine Merge-Konflikte
  • Ziel-Branch korrekt
  • (Optional) tiefes KI-Review via /review aus Claude Code (Pro-Abo, ohne API-Kosten)

Kostenlos generiert von reusable-mergeability.yml · keine API-Kosten · review-gate: success

@S540d S540d mentioned this pull request Nov 3, 2026
2 of 5 tasks
…e-forge-Eintrag entfernen

GHSA-qr28-fpjm-wq7f: Fix nur in decode-uri-component 0.5.1 (ESM-only,
inkompatibel mit query-string@7 aus expo-router) bzw. expo-router 58
(SDK-Major). Befristet bis 2026-12-31, mit dem nächsten SDK-Upgrade
auflösen. GHSA-86w9-cpqp-85rv ist durch node-forge 1.4.2 behoben.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@S540d S540d changed the title fix(deps): node-forge 1.4.0 → 1.4.2 (5 high Advisories) fix(deps): node-forge 1.4.2 + Audit-Allowlist (decode-uri-component) Nov 3, 2026
@S540d
S540d merged commit 35aae85 into testing Nov 3, 2026
14 checks passed
@S540d
S540d deleted the fix/audit-node-forge-2026-11 branch November 3, 2026 09:01
S540d added a commit that referenced this pull request Nov 3, 2026
Release-PR für **v1.9.0 / versionCode 71**: Play Store steht auf 1.8.0 /
vc69, vc70 wurde nie veröffentlicht.

> ⚠️ **Nicht mergen ohne ausdrückliche Freigabe.** Merge auf `main` löst
sofort den GitHub-Pages-Deploy aus (`deploy.yml`). Die Web-Demo geht
damit zu 100 % live, unabhängig vom gestaffelten Play-Store-Rollout.

## Inhalt

Vollständige Liste im `CHANGELOG.md` unter `[1.9.0]`. Die wichtigsten
Punkte:

**Neu:** Pause-Vorschlag alle 5 Level, Impressum/Datenschutz im
Über-Dialog (7 Sprachen), Micro-Animationen, Web-Auffindbarkeit und
Play-Store-Attribution
**Geändert:** neues App-Icon/Splash (Haus-Motiv), Design-System Blau
statt Violett, Einstellungen gruppiert, Startseite entschlackt, ruhiger
Merke-Timer, Replay- und Web-Canvas-Performance
**Behoben:** Konfetti/Abzeichen bei gleicher Sternzahl,
Darkmode-Sichtbarkeit, Galerie-Status beim Neustart, Web-Fills bei
gebrochener Canvas-Breite, PWA-Tags/Icons
**Security:** diverse `npm audit fix`, Audit-Gate mit befristeter
Allowlist

## Status vor Merge

- [x] `package.json` / `app.json`: 1.9.0, versionCode 71
- [x] Lokal: `tsc` ✅, Lint ✅, 52 Suites / 482 Tests ✅
- [ ] **Audit-Gate:** neue High-Advisories seit 2026-10-29
  - node-forge → Fix in #374 (gegen `testing`)
- decode-uri-component (GHSA-qr28-fpjm-wq7f, via expo-router →
query-string) → Entscheidung offen
- [ ] CHANGELOG-Datum `[1.9.0] - TBD` setzen
- [ ] Dependabot-PR #373 (zod/expo, gegen `main`) schließen; zod ist
bereits auf `testing` adressiert

## Nach dem Merge

1. AAB aus `main` bauen (production, vc71)
2. Play Console: Produktion, **gestaffelte Einführung 10 %**
3. Tag `v1.9.0` setzen
4. Android Vitals einige Tage beobachten, dann Anteil erhöhen

🤖 Generated with [Claude Code](https://claude.com/claude-code)

---------

Co-authored-by: S540d <S540d@users.noreply.github.com>
Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>
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.

1 participant