BaseCloudEvent.getExtensionNames() breaks cloud event encapsulation

Abierto
#581 2 comentarios 0 reacciones 1 asignado Ver en GitHub

@mhaider97 ya está trabajando en esto.

Desde el 2/10/2023.

Evaluación

Este issue todavía no se ha evaluado.

Descripción

enhancement help wanted

please replace

@Override
public Set<String> getExtensionNames() {
    return this.extensions.keySet();
}

with something, like

@Override
public Set<String> getExtensionNames() {
    return Collections.unmodifiableSet (this.extensions.keySet());
}

to avoid breaking encapsulation.

More details can be found here

https://docs.oracle.com/javase/8/docs/api/java/util/HashMap.html#keySet--
Returns a Set view of the keys contained in this map. The set is backed by the map, so changes to the map are reflected in the set, and vice-versa.

Lenguaje dominante
Java
Estrellas
446
Forks
172
Sin PR fusionados en 30 d

Guía de contribución

Abrir la guía de contribución

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 cloudevents/sdk-java

Todos los issues de cloudevents/sdk-java

Issues similares

Más issues de Java

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.