a2aproject / a2aproject/a2a-java

fix: handle blank push notification config id consistently across transports

Đang mở
#1,012 0 bình luận 0 reaction 0 người được giao Xem trên GitHub
Ngôn ngữ chính
Java
Star
490
Fork
172
Merge trung bình
1 ngày 6 giờ
Pull request đã merge (30 ngày)
55

Mô tả

`GetTaskPushNotificationConfigParams` rejects a null `id` but permits an
empty string, and the two transports diverge on that input:

- `GrpcTransport` sends `.setId("")`.
- `RestTransport` treats it as absent and requests the trailing-slash
"default config" URL.

Identical caller code against the same `ClientTransport` method therefore
produces two different requests. This aligns both on
and adds coverage for the blank-id path, which was previously untested.

Also corrects the Javadoc on `GetTaskPushNotificationConfigParams`, which
described `id` as optional and documented `@throws` for `taskId or tenant`
when the constructor actually validates `taskId` and `id`.

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Đánh giá

Issue này chưa được đánh giá.

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.