Repository navigation
perf: extractKeyValue faz uma conversão para []byte desnecessária para encontrar '=' #39
Description
Problema
secretscanner.extractKeyValue converte a linha (string) para []byte apenas para chamar bytes.IndexByte e encontrar o =, alocando e copiando a linha inteira sem necessidade.
Correção sugerida
Usar strings.IndexByte(line, '='), que encontra o mesmo índice sem a alocação/cópia.
Contexto
- Encontrado durante o code review da branch
feat/secret-scanning(commit 40a4708). - Arquivo:
internal/secretscanner/secretscanner.go:149
Activity
[-]perf: extractKeyValue does an unnecessary []byte conversion to find '='[/-][+]perf: extractKeyValue faz uma conversão para []byte desnecessária para encontrar '='[/+]joaooncode commentedon Sep 12, 2026
Corrigido no PR #43 (mesclado em main):
extractKeyValueagora usastrings.IndexByte(line, '=')em vez debytes.IndexByte([]byte(line), '='), eliminando a alocação/cópia desnecessária.