ipfs/kubo

Audit CoreAPI for cases where the context should be canceled internally

Open

#7200 aperta il 23 apr 2020

Vedi su GitHub
 (2 commenti) (0 reazioni) (0 assegnatari)Go (2725 fork)batch import
P3exp/intermediatehelp wantedkind/bug

Metriche repository

Star
 (13.906 star)
Metriche merge PR
 (Merge medio 4g 2h) (28 PR mergiate in 30 g)

Descrizione

Version information:

v0.5.0-rc3

Description:

There are several places in the CoreAPI (e.g., Ls) where we're assuming the context will be canceled when the caller is done. If that doesn't happen, we won't cleanup some resources.

Luckily, we are canceling these contexts. However, we shouldn't require that.

Guida contributor

Direzione di ricerca
Esamina ogni funzione CoreAPI (es. Ls) e assicurati che le goroutine e le risorse interne vengano pulite anche se il chiamante non cancella il contesto. Usa context.WithCancel o context.WithTimeout internamente.
Tech stack
go
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
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
Gocontext
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70