jdbi/jdbi

investigate making more classes final

Aperta

#2515 aperta il 19 ott 2023

Vedi su GitHubGuida contributor
 (1 commento) (0 reazioni) (0 assegnatari)Java (324 fork)batch import
help wantedimprovementinvestigation needed

Metriche repository

Star
 (1745 stelle)
Metriche merge PR
 (Merge medio 1g 10h) (6 PR mergiate in 30 g)

Descrizione

spotbugs added https://spotbugs.readthedocs.io/en/stable/bugDescriptions.html#ct-constructor-throw, which is an unlikely attack vector that can be mitigated by making classes final.

With bytecode mocking on the way out anyway and mocking as one of the few use cases to extend some of the base classes, we could investigate making more classes (e.g. Handle) final.

Guida contributor

Direzione di ricerca
Indaga quali classi nella codebase di jdbi potrebbero essere rese final per mitigare il vettore di attacco CT CONSTRUCTOR THROW. Inizia esaminando classi come Handle e altre che sono comunemente estese. Controlla se esistono sottoclassi o framework di mocking che dipendono dall'estensibilità. Rivedi la documentazione di SpotBugs per CT CONSTRUCTOR THROW per comprendere il rischio. Considera di creare un elenco di classi candidate e discuti nell'issue prima di implementare le modifiche.
Tech stack
java
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
JavaMocking
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
40