OpenFeign/feign

feign-kotlin: dismiss404 doesn't work in CoroutineFeign

Chiusa

#2026 aperta il 26 apr 2023

Vedi su GitHubGuida contributor
 (3 commenti) (0 reazioni) (0 assegnatari)Java (1911 fork)batch import
bughelp wanted

Metriche repository

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

Descrizione

Hi! CoroutineBuilder does not set dismiss404 property so it is initialized with false when AsyncFeign is being created. Is this intended?

Guida contributor

Direzione di ricerca
The issue is that CoroutineBuilder in CoroutineFeign.java (line 160-181) does not propagate the dismiss404 property to AsyncFeign. To fix, locate CoroutineBuilder and ensure that the dismiss404 field is set on the AsyncFeign.Builder before building. Refer to how other properties like logLevel are passed. Also check existing tests for CoroutineFeign to add a test for this scenario.
Tech stack
javakotlin
Dominio
backendapi
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
JavaKotlinFeign basics
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75