Retry-After parser accepts invalid delay-seconds values
Los mantenedores suelen responder en 2 días
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 82/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- java
- Área
- backend-api-design
Línea de trabajo
Empieza por RetryUtil.retryAfterNanos y sus pruebas existentes en io.opentelemetry.exporter.internal.RetryUtilTest. Comprueba cómo maneja el analizador +30, -0 y ٣٠, y luego ejecuta esa clase de pruebas. Se considera terminado cuando cada valor mal formado devuelve OptionalLong.empty() y los valores ASCII válidos de segundos de espera se siguen analizando.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Describe the bug
The internal RetryUtil.retryAfterNanos parser accepts +30 as a 30-second delay and -0 as zero. Long.parseLong also accepts non-ASCII decimal digits. These values do not match the HTTP Retry-After delay-seconds grammar (1*DIGIT, ASCII 0–9).
The JDK and OkHttp HTTP senders use this parser. An accepted malformed value overrides their ordinary retry backoff instead of being ignored.
Steps to reproduce
In the existing io.opentelemetry.exporter.internal.RetryUtilTest, pass +30, -0, or ٣٠ to RetryUtil.retryAfterNanos(value, Instant.EPOCH).
What did you expect to see?
OptionalLong.empty(), as for other malformed values, so the ordinary retry policy applies.
What did you see instead?
OptionalLong.of(30000000000L) for +30; OptionalLong.of(0L) for -0; non-ASCII decimal digits are also parsed as a delay.
What version and what artifacts are you using?
Artifacts: exporters/common (used by the JDK and OkHttp senders)
Version: source checkout at f02bddc
How referenced: direct source checkout
Environment
Compiler: Temurin 25.0.5
OS: macOS 15.5
Additional context
The HTTP grammar is defined in RFC 9110 section 10.2.3.
- Lenguaje dominante
- Java
- Estrellas
- 2.5k
- Forks
- 1k
- Merge medio
- 4 d 3 h
- PR fusionados (30 d)
- 49
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de open-telemetry/opentelemetry-java
-
PrometheusComponentProvider promotes all resource attributes when resource_constant_labels is empty ({})Posiblemente ocupada @rooks-ns la tomó hace 1 día. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
open-telemetry/opentelemetry-java#9014 ·
Los mantenedores suelen responder en 2 días
-
MetricReusableDataMarshaler passes MetricData count instead of data point count to otel.sdk.exporter.metric_data_point.{inflight,exported}Posiblemente ocupada @goutamadwant la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
open-telemetry/opentelemetry-java#9011 · 1 comentario ·
Los mantenedores suelen responder en 2 días
-
Fix `otel.sdk.log.created` counting semanticsPosiblemente ocupada @anuraaga la tomó hace 94 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
open-telemetry/opentelemetry-java#8688 ·
Los mantenedores suelen responder en 2 días
-
Add experimental view_matching_mode (composable metric views) supportPosiblemente ocupada @rooks-ns la tomó hoy. Abierto
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
open-telemetry/opentelemetry-java#9013 ·
Los mantenedores suelen responder en 2 días
-
Span/log processors and PeriodicMetricReader use static ComponentId, causing all instances to share otel.component.name "<type>/0"Posiblemente ocupada @goutamadwant la tomó hoy. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 68/100
open-telemetry/opentelemetry-java#9012 ·
Los mantenedores suelen responder en 2 días
Todos los issues de open-telemetry/opentelemetry-java
Issues similares
-
hermes-queue
Dificultad 1/5 Menos de una hora Aptitud para principiantes 72/100
chrisalunlloyd2-sudo/mindpalace#349 ·
Los mantenedores suelen responder en 1 día
-
bug priority: medium Review effort [1-5]: 2 status: needs-triage type: bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
carlos-emr/carlos#5818 ·
Los mantenedores suelen responder en 1 día
-
bug good first issue help wanted tooling
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
appmonster007/kmip-adapter#100 ·
-
[Documentation]: Remove non-existent CHANGELOG reference from StateBackendProperties JavadocAbiertoaccepted good first issue java
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
Los mantenedores suelen responder en 1 día
-
Marketplace time-zone issueAbiertobug main ui
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
openhab/openhab-webui#4671 ·
Los mantenedores suelen responder en 1 día