BaseCloudEvent.getExtensionNames() breaks cloud event encapsulation

Aperta
#581 2 commenti 0 reazioni 1 assegnatario Vedi su GitHub

@mhaider97 ci sta già lavorando.

Dal 2/10/2023.

Valutazione

Questa issue non è ancora stata valutata.

Descrizione

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.

Lingua principale
Java
Stelle
446
Fork
172
Nessuna PR unita negli ultimi 30g

Guida per i contributori

Apri la guida per i contributori

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

Tutte le issue di cloudevents/sdk-java

Issue simili

Altre issue su Java

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.