Bug: Metrics constructor mutates the Configuration object passed to it
Mantenedores costumam responder em até 1 dia
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 3/5
- Tempo estimado
- 1-2 dias
- Facilidade para iniciantes
- 45/100
- Tipo de issue
- Bug
- Clareza
- Razoavelmente clara
- Status de atividade
- Estagnada
- Stack de tecnologia
- java
- Domínio
- observability
Direção de pesquisa
Comece lendo src/main/java/dev/openfga/sdk/telemetry/Metrics.java:30-32 e inspecione como Configuration e TelemetryConfiguration são construídas. Confirme que a correção escolhida impede que Metrics modifique a Configuration do chamador, ao mesmo tempo que lida com segurança com uma configuração de telemetria null, e leve em conta que a instância de telemetria compartilha o contexto de PR #290.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
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)
- Linguagem predominante
- Java
- Estrelas
- 54
- Forks
- 26
- Merge médio
- 2d 10h
- PRs com merge (30d)
- 10
Preparar o ambiente
- Sem Dockerfile nem arquivo Docker Compose
- Sem modelo de pull request
- Ler o guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de openfga/java-sdk
-
bug
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 65/100
Mantenedores costumam responder em até 1 dia
-
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 72/100
Mantenedores costumam responder em até 1 dia
-
Dificuldade 5/5 Mais de uma semana Facilidade para iniciantes 25/100
Mantenedores costumam responder em até 1 dia
-
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 35/100
openfga/java-sdk#370 · 2 comentários ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 5/5 Mais de uma semana Facilidade para iniciantes 35/100
openfga/java-sdk#361 · 1 reação ·
Mantenedores costumam responder em até 1 dia
Todas as issues de openfga/java-sdk
Issues semelhantes
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 88/100
Mantenedores costumam responder em até 1 dia
-
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 88/100
apache/arrow-java#1311 ·
Mantenedores costumam responder em até 2 dias
-
bug triage
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 85/100
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 88/100
Mantenedores costumam responder em até 1 dia
-
security
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 65/100
IBM/networking-java-sdk#204 ·