EnableRetry is enabled un-conditionally

Aperta
#766 3 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
1/5
Tempo stimato
Meno di un'ora
Idoneità per principianti
45/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Ferma
Stack tecnologico
java, spring
Ambito
backend, cloud

Direzione di ricerca

Iniziate in spring-cloud-consul-core/src/main/java/org/springframework/cloud/consul/ConsulAutoConfiguration.java, nella configurazione ConsulAutoConfiguration collegata, e analizzate la relativa condizione spring.cloud.consul.retry.enabled. Verificate il comportamento di retry quando la proprietà viene omessa e quando viene impostata esplicitamente, usando come contesto il report collegato di Spring Cloud Kubernetes; il lavoro è completato quando Consul non abilita più retry incondizionatamente.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

bug

We have this report of a bug in spring cloud kubernetes here. The underlying issue is that if spring consul is on the classpath, it automatically enable spring-retry, as seen in this configuration.

That class has @ConditionalOnProperty(value = "spring.cloud.consul.retry.enabled", matchIfMissing = true), but the problem is in matchIfMissing = true.

To give some details: for example someone has both consul and kubernetes dependencies on the classpath. Because consul enables spring retry un-conditonally, it means it is enabled in kubernetes also - though users might not want that. The linked issue has all the needed details. To me, matchIfMissing = true should be dropped.

wdyt?

Lingua principale
Java
Stelle
821
Fork
540
Merge medio
20h 1m
PR unite (30g)
14

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di spring-cloud/spring-cloud-consul

Tutte le issue di spring-cloud/spring-cloud-consul

Issue simili

Altre issue su Java

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.