Config delete does not trigger the context refresh event

Abierto
#740 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
3/5
Tiempo estimado
1-2 días
Aptitud para principiantes
45/100
Tipo de issue
Error
Claridad
Bastante claro
Estado de actividad
Estancado
Stack tecnológico
java, spring-boot
Área
backend, cloud

Línea de trabajo

Comience con ConfigWatch.java, en la comprobación if alrededor de la línea 162, y trace cómo los cambios de configuración de Consul se convierten en eventos de actualización del contexto. Reproduzca la eliminación de una propiedad o de un archivo de configuración completo y, después, verifique si se emite el evento de actualización esperado y si los valores eliminados vuelven a valores de respaldo o deshabilitan los beans afectados.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

bug

Hello everyone, I would like to know the reason behind this if-check in the ConfigWatch
https://github.com/spring-cloud/spring-cloud-consul/blob/main/spring-cloud-consul-config/src/main/java/org/springframework/cloud/consul/config/ConfigWatch.java#L162

In my view, the situation when some property or whole config file is deleted is perfectly valid, and must trigger a context refresh event to switch off some beans or switch to default property values. But now this is not possible, because such events are ignored because of aforementioned if-check. I think it is possible to remove it without breaking things to support cases when properties are deleted. Please let me know if I am wrong.

Lenguaje dominante
Java
Estrellas
821
Forks
540
Merge medio
13 h 41 min
PR fusionados (30 d)
17

Preparar el entorno

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de spring-cloud/spring-cloud-consul

Todos los issues de spring-cloud/spring-cloud-consul

Issues similares

Más issues de Java

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.