Skip to content

CI: scan the diff for leaked credentials - #7

Merged
puppe1990 merged 1 commit into
mainfrom
ci/secret-scan
Sep 24, 2026
Merged

puppe1990 merged 1 commit into
mainfrom
ci/secret-scan

Conversation

@puppe1990

Copy link
Copy Markdown
Owner

Por quê

A aplicação key do Backblaze foi commitada por engano num teste (commit f6c6665), corrigida depois com --amend + force-push. Force-push não apaga o objeto no GitHub: o commit antigo continuou acessível pelo SHA neste repo público, e a key só parou de funcionar quando foi revogada no console do Backblaze.

O secret_scanning_push_protection do GitHub já estava ligado, mas ele não conhece chaves de Backblaze — por isso passou. O GitGuardian também não pegou.

O que este PR faz

Adiciona um job de CI (Secrets / Diff scan) que roda em todo push na main e em todo PR, olhando só o diff, procurando:

  • K005… (application key do Backblaze — o formato que vazou)
  • AKIA… (access key ID da AWS)
  • aws_secret_access_key
  • cabeçalhos de chave privada (-----BEGIN … PRIVATE KEY-----)

Testado localmente: pega a key que vazou e não casa com os placeholders que ficaram no teste (b2-key-id / b2-application-key). Os próprios padrões estão partidos na fonte para o arquivo não se auto-detectar.

Rectifying a leak: a Backblaze application key was committed in a test and,
although the commit was amended away, force-pushing does not delete the
object on GitHub, so it stayed reachable by SHA in this public repository
until the key was revoked. GitHub's push protection is enabled but does not
recognise Backblaze keys, so this job scans the diff of every PR and push to
main for B2 keys, AWS access key IDs, aws_secret_access_key assignments and
private key headers.

Co-authored-by: CommandCodeBot <noreply@commandcode.ai>
@puppe1990
puppe1990 merged commit 9805a43 into main Sep 24, 2026
4 checks passed
@puppe1990
puppe1990 deleted the ci/secret-scan branch September 24, 2026 22:55
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