[Bug]: REST client drops task cutoff and push-config pagination parameters
维护者通常 1 天内回复
评估
调研方向
先查看 listTasks 和 listTaskPushNotificationConfigurations 的 REST 查询构建路径,然后阅读现有的 RestTransportTest 请求记录用例。运行这些测试,并检查 statusTimestampAfter、pageSize 和不透明的 pageToken 是否传递到请求 URL 中,同时未设置的参数保持省略。用例通过后即完成。
由索引模型根据 Issue 内容生成。
描述
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.
- 主要语言
- Java
- 星标
- 505
- 派生
- 187
- 平均合并
- 1 天 17 小时
- 30 天内合并 PR
- 24
环境准备
- 没有 Dockerfile 或 Docker Compose 文件
- 有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
a2aproject/a2a-java 的其他 Issue
-
fix: handle blank push notification config id consistently across transports可能已有人在做 @prachisinghal-1 今天认领。 未关闭
难度 2/5 1-3 小时 新手友好度 75/100
a2aproject/a2a-java#1012 · 2 条评论 · 已指派 1 人 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 65/100
a2aproject/a2a-java#464 · 1 条评论 ·
维护者通常 1 天内回复
-
Create a sample that demonstrates how to use a shared contextId across multiple tasks可能重新可做 @tanish111 于 367 天前认领,目前没有进行中的 PR。 未关闭sample
难度 2/5 1-3 小时 新手友好度 70/100
a2aproject/a2a-java#375 · 5 条评论 · 已指派 1 人 ·
维护者通常 1 天内回复
-
[Bug]: Blocking message/send loses the final event when the agent emits an event before completing可能已有人在做 @jeroenbellen 于 8 天前认领。 未关闭
难度 4/5 3-5 天 新手友好度 35/100
a2aproject/a2a-java#1226 ·
维护者通常 1 天内回复
-
Task state transitions are not tracked可能已有人在做 @sumitvairagar 于 6 天前认领。 未关闭
难度 3/5 1-2 天 新手友好度 45/100
a2aproject/a2a-java#1222 · 3 条评论 ·
维护者通常 1 天内回复
查看 a2aproject/a2a-java 的全部 Issue
相似的 Issue
-
code-maintenance
难度 2/5 1-3 小时 新手友好度 77/100
维护者通常 1 天内回复
-
bug priority: medium Review effort [1-5]: 3 status: needs-triage type: bug
难度 2/5 1-3 小时 新手友好度 82/100
carlos-emr/carlos#5258 ·
维护者通常 1 天内回复
-
bug: author create/update with an unknown avatarId returns 404 instead of 400 UNKNOWN_REFERENCE可能已有人在做 @Vinu2111 今天认领。 未关闭bug
难度 2/5 1-3 小时 新手友好度 74/100
atomityhq/blog-cms#59 · 1 条评论 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 70/100
维护者通常 1 天内回复
-
dev/release: verify-release-candidate.sh runs 0 Java tests with Debian's Maven可能已有人在做 @kou 今天认领。 未关闭Type: bug
难度 2/5 1-3 小时 新手友好度 72/100
apache/arrow-adbc#4947 ·
维护者通常 1 天内回复