OpenFeign/feign

feign-kotlin: dismiss404 doesn't work in CoroutineFeign

Fermée

#2 026 ouverte le 26 avr. 2023

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

Métriques du dépôt

Stars
 (9 070 étoiles)
Métriques de merge PR
 (Merge moyen 2j 17h) (71 PRs mergées en 30 j)

Description

Hi! CoroutineBuilder does not set dismiss404 property so it is initialized with false when AsyncFeign is being created. Is this intended?

Guide contributeur

Direction de recherche
The issue is that CoroutineBuilder in CoroutineFeign.java (line 160-181) does not propagate the dismiss404 property to AsyncFeign. To fix, locate CoroutineBuilder and ensure that the dismiss404 field is set on the AsyncFeign.Builder before building. Refer to how other properties like logLevel are passed. Also check existing tests for CoroutineFeign to add a test for this scenario.
Stack technique
javakotlin
Domaine
backendapi
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.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 heures
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
JavaKotlinFeign basics
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75