OpenFeign/feign

RetryableException requires Request

Open

#1,296 建立於 2020年10月27日

在 GitHub 查看
 (1 留言) (0 反應) (0 負責人)Java (1,911 fork)batch import
help wantedproposal

倉庫指標

Star
 (9,070 star)
PR 合併指標
 (平均合併 2天 17小時) (30 天內合併 71 個 PR)

描述

The constructors for RetryableException were updated in #1039 to require the original Request object. In some cases the Request may contain sensitive information and never be used from the RetryableException, so it would be nice to have an option to create one without the Request to limit surface area.

Additionally, FeignException already exposes a hasRequest() method, so clients would be able to check whether RetryableException has a Request before attempting to access it.

I'd be more than happy to open a PR if this sounds like a reasonable use case. Thanks!

貢獻者指南