security: `Telemetry.maskString` leaves bare `password=value` and `PWD=value` unmasked

Aberta
#1,457 0 comentários 0 reações 0 responsáveis Ver no GitHub

Mantenedores costumam responder em até 1 dia

Ninguém assumiu esta issue ainda.

Avaliação

Dificuldade
3/5
Tempo estimado
1-2 dias
Facilidade para iniciantes
65/100
Tipo de issue
Bug
Clareza
Claramente especificada
Status de atividade
Ativa
Stack de tecnologia
typescript
Domínio
backend, security

Direção de pesquisa

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.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

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.

Linguagem predominante
TypeScript
Estrelas
813
Forks
124
Merge médio
1d 18h
PRs com merge (30d)
70

Preparar o ambiente

Abrir no Codespaces

Inicia o contêiner de desenvolvimento do projeto no navegador, com a sua própria conta do GitHub.

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de AltimateAI/altimate-code

Todas as issues de AltimateAI/altimate-code

Issues semelhantes

Mais issues de TypeScript

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.