Bug: Metrics constructor mutates the Configuration object passed to it
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 45/100
- Tipo di issue
- Bug
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Ferma
- Stack tecnologico
- java
- Ambito
- observability
Direzione di ricerca
Inizia leggendo src/main/java/dev/openfga/sdk/telemetry/Metrics.java:30-32 e verifica come vengono costruite Configuration e TelemetryConfiguration. Conferma che la correzione scelta impedisca a Metrics di modificare la Configuration del chiamante, gestendo al contempo in modo sicuro una configurazione di telemetria null, e considera che l'istanza di telemetria condivide il contesto di PR #290.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Summary
In src/main/java/dev/openfga/sdk/telemetry/Metrics.java:30-32, the Metrics constructor silently mutates the caller's Configuration:
if (this.configuration.getTelemetryConfiguration() == null) {
this.configuration.telemetryConfiguration(new TelemetryConfiguration());
}
Impact
- Side effect in constructor — Constructors should not modify their inputs.
- Shared Configuration objects are unsafe — If a
Configurationis shared across contexts, one path creatingMetricswill silently add aTelemetryConfigurationthat affects all other users of that object.
Proposed Fix
Instead of mutating the configuration, handle the null case defensively in the methods that read telemetry config. Alternatively, ensure a non-null default TelemetryConfiguration is set during Configuration construction so the null check is never needed.
Related
- PR #290 (telemetry instance sharing fix)
- Lingua principale
- Java
- Stelle
- 54
- Fork
- 26
- Merge medio
- 2g 10h
- PR unite (30g)
- 10
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di openfga/java-sdk
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
I maintainer di solito rispondono entro 1 giorno
-
Make client-credentials token refresh buffer and jitter configurableForse già presa Una pull request collegata a questa issue è aperta o già unita. Aperta
Difficoltà 3/5 1-2 giorni Idoneità per principianti 72/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 25/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
openfga/java-sdk#370 · 2 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
openfga/java-sdk#361 · 1 reazione ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di openfga/java-sdk
Issue simili
-
new feature
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
apache/rocketmq-dashboard#5594 ·
I maintainer di solito rispondono entro 3 giorni
-
bug pkg:sdk
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
aws/aws-durable-execution-sdk-java#773 ·
I maintainer di solito rispondono entro 1 giorno
-
[C#]:主页联网更新的提示投稿横幅指向错误Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
PCL-Community/PCL-CE#3652 ·
I maintainer di solito rispondono entro 1 giorno
-
TaskSecret.vue: replace explicit `any` with real typesForse già presa @prayas-bit l’ha presa oggi. Apertaarea/frontend good first issue kind/cooldown
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
kestra-io/kestra#20352 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno