OpenFeign/feign

Allow for 3xx range headers

Open

#249 创建于 2015年7月12日

在 GitHub 查看
 (28 评论) (0 反应) (0 负责人)Java (1,911 fork)batch import
documentationhelp wanted

仓库指标

Star
 (9,070 star)
PR 合并指标
 (平均合并 2天 17小时) (30 天内合并 71 个 PR)

描述

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?

贡献者指南

研究方向
在 SynchronousMethodHandler 中找到 `if (response.status() >= 200 && response.status() < 300)` 这一行，并将其修改为接受 3xx 状态码，即将条件改为 `response.status() >= 200 && response.status() < 400`。
技术栈
java
领域
backend
议题类型
缺陷
难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。
1
预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。
1 小时以内
活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。
活跃
清晰度议题是否清楚说明期望改动、验收标准和下一步。
清晰
前置要求
Java
新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。
70