ipfs/kubo

Provide sentinel errors in the HTTP API

Open

#7307 aperta il 12 mag 2020

Vedi su GitHub
 (3 commenti) (0 reazioni) (0 assegnatari)Go (2725 fork)batch import
P3help wantedkind/enhancementstatus/readytopic/rpc-api

Metriche repository

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

Descrizione

Matching errors against concrete types would be nice vs. fragile error string matching, e.g.,

if err.Error() == "'from' cid was not recursively pinned already" { ...

Or, maybe using more fine grained error codes would be even better. I believe the above error will come with a code: ErrNormal or similar.

Guida contributor

Direzione di ricerca
Esamina il gestore HTTP API per il pinning. Identifica i punti in cui viene utilizzato un fragile matching di stringhe sugli errori (ad esempio, err.Error() == "'from' cid was not recursively pinned already"). Determina se sono già definiti errori sentinella o tipi di errore personalizzati (come ErrNormal). Quindi sostituisci i confronti di stringhe con errors.Is() o errors.As() utilizzando tali errori sentinella.
Tech stack
go
Dominio
backend
Tipo issue
Refactoring
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.
Abbastanza chiara
Prerequisiti
GoError handling in Go
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60