[Bug]: REST client drops task cutoff and push-config pagination parameters
Maintainer thường phản hồi trong vòng 1 ngày
Đánh giá
- Độ khó
- 2/5
- Thời gian dự kiến
- 1-3 giờ
- Mức phù hợp với người mới
- 78/100
Hướng nghiên cứu
Bắt đầu với các đường dẫn xây dựng truy vấn REST cho listTasks và listTaskPushNotificationConfigurations, sau đó đọc các trường hợp hiện có trong RestTransportTest ghi lại yêu cầu. Chạy các bài kiểm thử đó và kiểm tra rằng statusTimestampAfter, pageSize và pageToken dạng opaque được đưa vào URL yêu cầu, còn các tham số chưa được đặt thì bị lược bỏ. Hoàn thành khi các trường hợp kiểm thử đều đạt.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
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.
- Ngôn ngữ chính
- Java
- Star
- 505
- Fork
- 187
- Merge trung bình
- 1 ngày 17 giờ
- Pull request đã merge (30 ngày)
- 24
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Có mẫu pull request
- Đọc hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của a2aproject/a2a-java
-
fix: handle blank push notification config id consistently across transportsCó thể đã có người làm @prachisinghal-1 đã nhận 2 ngày trước. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
a2aproject/a2a-java#1012 · 2 bình luận · 1 người được giao ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
a2aproject/a2a-java#464 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Create a sample that demonstrates how to use a shared contextId across multiple tasksCó thể làm lại được @tanish111 đã nhận 369 ngày trước và không có pull request nào đang mở. Đang mởsample
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
a2aproject/a2a-java#375 · 5 bình luận · 1 người được giao ·
Maintainer thường phản hồi trong vòng 1 ngày
-
[Bug]: Blocking message/send loses the final event when the agent emits an event before completingCó thể đã có người làm @jeroenbellen đã nhận 10 ngày trước. Đang mở
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 35/100
a2aproject/a2a-java#1226 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Task state transitions are not trackedCó thể làm lại được @sumitvairagar đã nhận 9 ngày trước và không có pull request nào đang mở. Đang mở
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 45/100
a2aproject/a2a-java#1222 · 3 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của a2aproject/a2a-java
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 67/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
linagora/twake-calendar-side-service#1218 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
[Bug] [Go] Ack returns nil when the server answers with a non-OK statusCó thể đã có người làm @leizhiyuan đã nhận hôm nay. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 84/100
apache/rocketmq-clients#1465 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
Maintainer thường phản hồi trong vòng 1 ngày
-
pdfium-drift
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 88/100
Stirling-Tools/JPDFium#188 ·
Maintainer thường phản hồi trong vòng 1 ngày