oracle/oci-cloud-controller-manager

Need to check whether the variable is nil

Open

#340 aperta il 7 gen 2021

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)Go (104 fork)auto 404
good first issue

Metriche repository

Star
 (157 star)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

According to line 89, resq may be nil, and resp.Body in line 96 will cause a crash. By reading the previous code, resq cannot be nil, so we can remove the comparison in line 89. https://github.com/oracle/oci-cloud-controller-manager/blob/220ab897fd3e8b1bd4b6b6d13a072f477453c073/pkg/oci/instance/metadata/instance_metadata.go#L89-L96

Guida contributor

Direzione di ricerca
Tech stack
Dominio
Tipo issue
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Prerequisiti
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.