nccgroup/sobelow

sobelow should not accept `# sobelow_skip` that are not needed

Open

#159 aberto em 3 de abr. de 2024

Ver no GitHub
 (4 comments) (0 reactions) (0 assignees)Elixir (119 forks)batch import
backlogfeaturegood first issue

Métricas do repositório

Stars
 (1.780 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

I notice in our code an instance of

  # sobelow_skip ["XSS.Raw"]
  def a_function(arg) do
    that_does_not_call_raw()
  end

I believe sobelow should raise an error on these. They do not reflect the code / current intention. Although unlikely, they could allow someone to add raw without it being super apparent in the diff of the resulting PR.

Thanks for sobelow

Guia do colaborador

Direção de pesquisa
Investigue como o sobelow processa comentários de skip e implemente um aviso ou erro quando uma diretiva de skip for desnecessária (por exemplo, a função não chama a função sinalizada).
Pilha de tecnologia
elixir
Domain
securitytooling
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Elixir
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75