OpenFeign/feign

Allow for 3xx range headers

Ouverte

#249 ouverte le 12 juil. 2015

Voir sur GitHubGuide contributeur
 (28 commentaires) (0 réaction) (0 personne assignée)Java (1 911 forks)batch import
documentationhelp wanted

Métriques du dépôt

Stars
 (9 070 étoiles)
Métriques de merge PR
 (Métriques PR en attente)

Description

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?

Guide contributeur

Direction de recherche
Localisez la ligne `if (response.status() >= 200 && response.status() < 300)` dans SynchronousMethodHandler et modifiez la pour accepter les codes d'état 3xx en changeant la condition en `response.status() >= 200 && response.status() < 400`.
Stack technique
java
Domaine
backend
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
1
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Java
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70