security: `Telemetry.maskString` leaves bare `password=value` and `PWD=value` unmasked
Maintainer antworten meist innerhalb von 1 Tag
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Anfängerfreundlichkeit
- 65/100
- Issue-Typ
- Bug
- Klarheit
- Klar beschrieben
- Aktivitätsstatus
- Aktiv
- Tech-Stack
- typescript
Rechercherichtung
Find the implementation and tests for Telemetry.maskString; the issue also names altimate/native/connections/reconnect-log.ts as a caller. First inspect the existing masking cases and tests, then check how the proposed delimiters apply to unquoted values. Done means the listed bare password=, PWD=, and pass= cases are masked while already masked tokens and quoted-password behavior remain covered.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Found during the v0.12.7 release review's adversarial tests. Deferred: maskString is shared by telemetry and every log line that masks errors, so a change needs its own review. Related: #1287.
password=hunter2 -> password=hunter2 (unmasked)
PWD=hunter2; -> PWD=hunter2; (unmasked; ODBC-style connection strings)
pass=hunter2 -> pass=hunter2 (unmasked)
password: "hunter2" -> password: ? (masked)
token=sk-ant-api03-… -> token=sk-*** (masked)
Callers include warehouse-connect connect failed and, new in v0.12.7, reconnect failed (altimate/native/connections/reconnect-log.ts). Snowflake's own auth errors don't quote the password, so no known leak today, but a driver or SDK error that echoes a connection string would be logged as is.
Proposed: mask the value of unquoted (password|passwd|pwd|pass|secret|token)= up to the next ;, &, whitespace or end.
- Vorherrschende Sprache
- TypeScript
- Sterne
- 812
- Forks
- 123
- Ø Merge
- 1 T. 16 Std.
- Gemergte PRs (30 T.)
- 81
Entwicklungsumgebung
Startet den Dev-Container des Projekts im Browser, mit Ihrem eigenen GitHub-Konto.
- Kein Dockerfile und keine Docker-Compose-Datei
- Hat eine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus AltimateAI/altimate-code
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
AltimateAI/altimate-code#1593 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
AltimateAI/altimate-code#1591 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
AltimateAI/altimate-code#1590 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 69/100
AltimateAI/altimate-code#1588 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 76/100
AltimateAI/altimate-code#1573 ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in AltimateAI/altimate-code
Ähnliche Issues
-
bug needs-triage
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 76/100
apex-dev-tools/apex-log-parser#199 ·
Maintainer antworten meist innerhalb von 1 Tag
-
area/frontend/library-panels automated-triage type/bug type/ux
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 78/100
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
PostHog/posthog.com#21607 ·
Maintainer antworten meist innerhalb von 1 Tag
-
enhancement
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 73/100
ajv-validator/ajv#2689 ·
-
[Bug]: IME confirmation Enter can submit the SSH port-forward dialogEvtl. vergeben @seijikohara hat das heute übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 88/100
Maintainer antworten meist innerhalb von 1 Tag