crytic/slither

Constants implementing interface getters are marked as `UPPER_CASE_WITH_UNDERSCORES`

Open

#930 aperta il 27 ago 2021

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)Python (886 fork)batch import
enhancementgood first issuehelp wanted

Metriche repository

Star
 (4769 star)
Metriche merge PR
 (Merge medio 54g 9h) (2 PR mergiate in 30 g)

Descrizione

interface I {
    function version() external view returns (uint256);
}

contract C is I {
    uint256 public constant override version = 1;
}

Due to the override specifier, the naming schema should not be checked at C.version.

Guida contributor

Direzione di ricerca
Ispeziona la logica di rilevamento delle convenzioni di denominazione in Slither, in particolare come gestisce il modificatore 'override'. Controlla i casi di test per i getter costanti e verifica se manca una condizione per saltare il controllo di denominazione per le funzioni dell'interfaccia implementate.
Tech stack
python
Dominio
securityblockchain
Tipo issue
Bug
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
SolidityStatic analysis concepts
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65