googleapis/google-cloud-python

add underlying cause for `TransportError`

Chiusa

#15.181 aperta il 15 dic 2023

Vedi su GitHubGuida contributor
 (2 commenti) (0 reazioni) (0 assegnatari)Python (1481 fork)batch import
help wantedpriority: p3type: feature request

Metriche repository

Star
 (4304 stelle)
Metriche merge PR
 (Merge medio 4g) (158 PR mergiate in 30 g)

Descrizione

Is it possible to save a reference to the last exception thrown at:

https://github.com/googleapis/google-auth-library-python/blob/0afc61a3a9c3d5035c913ad4a7568e8a888e2ec9/google/auth/compute_engine/_metadata.py#L204

and use it to build a more meaningful error message? at:

https://github.com/googleapis/google-auth-library-python/blob/0afc61a3a9c3d5035c913ad4a7568e8a888e2ec9/google/auth/compute_engine/_metadata.py#L217-L220

that way the underlying reason for the TransportError would be available even if DEBUG logging level is not enabled.

Guida contributor

Direzione di ricerca
The issue suggests modifying the TransportError to include the underlying cause. The relevant code is in google/auth/compute engine/ metadata.py around line 204 and 217-220. A contributor should capture the last exception before raising TransportError and use Python's exception chaining (e.g., `raise ... from exc`) to preserve it. The change is small and localized.
Tech stack
python
Dominio
backend
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Nessuno
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80