OpenFeign/feign

RetryableException requires Request

オープン

#1,296 opened on 2020/10/27

 (1 件のコメント) (0 件のリアクション) (0 人の担当者)Java (1,911 件のフォーク)batch import
help wantedproposal

Repository metrics

Stars
 (9,070 個のスター)
PR merge metrics
 (PR metrics pending)

説明

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!

コントリビューターガイド