OpenFeign/feign

Allow for 3xx range headers

Open

Aperta il 12 lug 2015

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

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

Tech stack
java
Dominio
backend
Tipo issue
bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 hours
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
stale
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
clear
Prerequisiti
JavaHTTP basics
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70
Direzione di ricerca
The issue is about Feign treating all HTTP status codes above 299 as errors, which incorrectly includes 3xx redirects. The fix involves modifying a single line in SynchronousMethodHandler.java (likely line checking `response.status() >= 200 && response.status() < 300`). After making the change, ensure existing tests pass and add a test for redirect handling. The issue has extensive discussion but no assignee or PR, so it is open for contribution.