OpenFeign/feign

RetryableException requires Request

Open

#1.296 aberto em 27 de out. de 2020

Ver no GitHub
 (1 comment) (0 reactions) (0 assignees)Java (1.911 forks)batch import
help wantedproposal

Métricas do repositório

Stars
 (9.070 stars)
Métricas de merge de PR
 (Mesclagem média 3d 8h) (59 fundiu PRs em 30d)

Description

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!

Guia do colaborador

Direção de pesquisa
Examine a classe RetryableException no código fonte para entender os construtores atuais. Revise as alterações do PR #1039 que tornaram Request obrigatória. Considere adicionar um novo construtor que não receba Request, ou torne Request opcional. Verifique também o método hasRequest() de FeignException para garantir consistência. Crie um PR com as alterações e atualize os testes de acordo.
Pilha de tecnologia
java
Domain
backendapi
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
JavaFeign library
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70