OpenFeign/feign

Allow for 3xx range headers

Open

#249 aperta il 12 lug 2015

Vedi su GitHub
 (28 commenti) (0 reazioni) (0 assegnatari)Java (1911 fork)batch import
documentationhelp wanted

Metriche repository

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

Descrizione

Everything above 299 is considered an error, which is plainly incorrect.

To get around this I had to:

  1. add an error decoder
  2. throw a special exception for 3xx range
  3. when catching runtime errors, check if it is the special 300 error
  4. convert to a valid response entity

Will investigate replacing this line:

if (response.status() >= 200 && response.status() < 300) { in SynchronousMethodHandler and PR'ing it. I just wonder why nobody else has come across this issue.

Should I fix it?

Guida contributor

Direzione di ricerca
Trova la riga `if (response.status() >= 200 && response.status() < 300)` in SynchronousMethodHandler e modificala per accettare anche i codici di stato 3xx cambiando la condizione in `response.status() >= 200 && response.status() < 400`.
Tech stack
java
Dominio
backend
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
1
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.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Java
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70