[Bug]: REST client drops task cutoff and push-config pagination parameters
Los mantenedores suelen responder en 2 días
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 78/100
Línea de trabajo
Empieza por las rutas de construcción de consultas REST para listTasks y listTaskPushNotificationConfigurations; después, lee los casos existentes de RestTransportTest que registran solicitudes. Ejecuta esas pruebas y comprueba que statusTimestampAfter, pageSize y pageToken opaco lleguen a las URL de las solicitudes, mientras que los parámetros no establecidos se omitan. El trabajo estará terminado cuando los casos pasen.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
What happened?
The REST client drops optional query parameters that the server already supports. Reproduced on main at 86a0eb3c601aaa143b389b117f87f69df982ad4a.
For listTasks, setting statusTimestampAfter does not put the cutoff in the request:
ListTasksParams params = ListTasksParams.builder()
.statusTimestampAfter(Instant.parse("2024-01-01T00:00:00Z"))
.build();
transport.listTasks(params, null);
Expected /tasks?statusTimestampAfter=2024-01-01T00%3A00%3A00Z; the recorded request has no statusTimestampAfter. The gRPC/JSON-RPC mappings and REST server handler already handle the field.
The same omission affects listTaskPushNotificationConfigurations: pageSize and pageToken never reach /tasks/{taskId}/pushNotificationConfigs. The server consumes them and returns nextPageToken, but the client cannot request that next page.
I added request-recording cases to the existing RestTransportTest and ran them on JDK 17 in an isolated Maven container. Both omission checks fail on main; the control with no optional parameters passes (3 tests, 2 failures, 0 errors/skips).
I can fix these two REST query-building paths and cover the timestamp, opaque page tokens and unset-parameter behavior without changing public APIs. Please assign this scope to me if that works for the project.
Relevant log output
Tests run: 3, Failures: 2, Errors: 0, Skipped: 0
listTasks must send the documented statusTimestampAfter query parameter
listTaskPushNotificationConfigurations: expected pageSize <1> but was <>
Code of Conduct
- I agree to follow this project's Code of Conduct.
Reproduction and report prepared and posted by an AI assistant for @hutiefang76.
- Lenguaje dominante
- Java
- Estrellas
- 505
- Forks
- 187
- Merge medio
- 1 d 19 h
- PR fusionados (30 d)
- 22
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una 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 a2aproject/a2a-java
-
fix: handle blank push notification config id consistently across transportsPosiblemente ocupada @prachisinghal-1 la tomó hace 3 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
a2aproject/a2a-java#1012 · 2 comentarios · 1 asignado ·
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
a2aproject/a2a-java#464 · 1 comentario ·
Los mantenedores suelen responder en 2 días
-
Create a sample that demonstrates how to use a shared contextId across multiple tasksQuizá libre de nuevo @tanish111 la tomó hace 370 días y no hay ningún pull request abierto. Abiertosample
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
a2aproject/a2a-java#375 · 5 comentarios · 1 asignado ·
Los mantenedores suelen responder en 2 días
-
[Bug]: Blocking message/send loses the final event when the agent emits an event before completingPosiblemente ocupada @jeroenbellen la tomó hace 11 días. Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
a2aproject/a2a-java#1226 ·
Los mantenedores suelen responder en 2 días
-
Task state transitions are not trackedQuizá libre de nuevo @sumitvairagar la tomó hace 9 días y no hay ningún pull request abierto. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 45/100
a2aproject/a2a-java#1222 · 4 comentarios ·
Los mantenedores suelen responder en 2 días
Todos los issues de a2aproject/a2a-java
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
Los mantenedores suelen responder en 1 día
-
Type/Improvement
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
ballerina-platform/static-code-analysis-tool#140 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
atomityhq/blog-cms#82 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
ready-for-triage triaged
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
RedHatInsights/notifications-backend#5080 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
Unhandled event loop exceptionAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
eclipse-4diac/4diac-ide#3103 ·
Los mantenedores suelen responder en 1 día