Hacktoberfest 2026: die Issues, die Maintainer für den Oktober markiert haben – offen und einsteigerfreundlich. Hacktoberfest-Issues durchsuchen

[Bug]: REST client drops task cutoff and push-config pagination parameters

Offen Anfängerfreundlich
  • Stimmt etwas nicht?
#1,233 2 Kommentare 0 Reaktionen 1 zugewiesene Person Auf GitHub ansehen

Maintainer antworten meist innerhalb von 1 Tag

@sumitvairagar arbeitet bereits daran.

Seit 20.10.2026.

  • #1239 von @sumitvairagar — offen

Bewertung

Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Anfängerfreundlichkeit
78/100
Issue-Typ
Bug
Klarheit
Klar beschrieben
Aktivitätsstatus
Aktiv
Tech-Stack
java
Bereich
api, backend

Rechercherichtung

Beginne mit den Pfaden zum Erstellen von REST-Abfragen für listTasks und listTaskPushNotificationConfigurations und lies anschließend die bestehenden Fälle in RestTransportTest, die Anfragen aufzeichnen. Führe diese Tests aus und prüfe, dass statusTimestampAfter, pageSize und pageToken als opaker Wert in den Anforderungs-URLs ankommen, während nicht gesetzte Parameter ausgelassen werden. Fertig ist die Arbeit, wenn die Fälle erfolgreich sind.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

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.

Vorherrschende Sprache
Java
Sterne
505
Forks
187
Ø Merge
1 T. 17 Std.
Gemergte PRs (30 T.)
24

Entwicklungsumgebung

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus a2aproject/a2a-java

Alle Issues in a2aproject/a2a-java

Ähnliche Issues

Weitere Issues zu Java

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.