OpenFeign/feign

RetryableException requires Request

Open

#1296 aperta il 27 ott 2020

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)Java (1911 fork)batch import
help wantedproposal

Metriche repository

Star
 (9070 star)
Metriche merge PR
 (Merge medio 2g 17h) (71 PR mergiate in 30 g)

Descrizione

The constructors for RetryableException were updated in #1039 to require the original Request object. In some cases the Request may contain sensitive information and never be used from the RetryableException, so it would be nice to have an option to create one without the Request to limit surface area.

Additionally, FeignException already exposes a hasRequest() method, so clients would be able to check whether RetryableException has a Request before attempting to access it.

I'd be more than happy to open a PR if this sounds like a reasonable use case. Thanks!

Guida contributor

Direzione di ricerca
Esamina la PR #1039 per comprendere le modifiche al costruttore, quindi aggiungi un parametro Request opzionale o un costruttore sovraccaricato per RetryableException, assicurando la coerenza con il metodo hasRequest().
Tech stack
java
Dominio
api
Tipo issue
Funzionalità
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.
Chiara
Prerequisiti
GitJava
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75