OpenFeign/feign

feign-kotlin: dismiss404 doesn't work in CoroutineFeign

Geschlossen

#2.026 geöffnet am 26.04.2023

Auf GitHub ansehenContributor Guide
 (3 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Java (1.911 Forks)batch import
bughelp wanted

Repository-Metriken

Stars
 (9.070 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 2T 17h) (71 gemergte PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
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.
Tech Stack
javakotlin
Domain
backendapi
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
JavaKotlinFeign basics
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75