nccgroup/sobelow

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

Open

#159 aperta il 3 apr 2024

Vedi su GitHub
 (4 commenti) (0 reazioni) (0 assegnatari)Elixir (119 fork)batch import
backlogfeaturegood first issue

Metriche repository

Star
 (1780 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

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

Guida contributor

Direzione di ricerca
Indaga come sobelow elabora i commenti di skip e implementa un avviso o un errore quando una direttiva di skip non è necessaria (ad esempio, la funzione non chiama la funzione contrassegnata).
Tech stack
elixir
Dominio
securitytooling
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Elixir
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75