fix: handle blank push notification config id consistently across transports
维护者通常 1 天内回复
还没有人认领这个 Issue。
评估
调研方向
检查 GetTaskPushNotificationConfigParams 类及其构造函数校验。查看 GrpcTransport 和 RestTransport 的实现，了解它们如何处理 id 参数。为 blank-id 情况编写测试，以确保行为一致。修复应使两个 transports 以相同方式处理空白的 id，要么视为缺失，要么视为空字符串，并相应更新 Javadoc。
由索引模型根据 Issue 内容生成。
描述
GetTaskPushNotificationConfigParams rejects a null id but permits an
empty string, and the two transports diverge on that input:
GrpcTransportsends.setId("").RestTransporttreats 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.
- 主要语言
- Java
- 星标
- 504
- 派生
- 184
- 平均合并
- 1 天 13 小时
- 30 天内合并 PR
- 31
环境准备
- 没有 Dockerfile 或 Docker Compose 文件
- 有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
a2aproject/a2a-java 的其他 Issue
-
难度 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 于 353 天前认领，目前没有进行中的 PR。 未关闭sample
难度 2/5 1-3 小时 新手友好度 70/100
a2aproject/a2a-java#375 · 5 条评论 · 已指派 1 人 ·
维护者通常 1 天内回复
-
难度 3/5 1-2 天 新手友好度 55/100
a2aproject/a2a-java#1207 ·
维护者通常 1 天内回复
-
Enable checkstyle failOnViolation=true可能已有人在做 @ZYZ666-RGB 于 1 天前认领。 未关闭
难度 3/5 1-2 天 新手友好度 58/100
a2aproject/a2a-java#1206 · 2 条评论 · 已指派 1 人 ·
维护者通常 1 天内回复
-
[Bug]: REST transport never signals normal stream completion to the client (SSE hangs until timeout)可能已有人在做 关联的 PR 仍在进行中或已合并。 未关闭
难度 4/5 3-5 天 新手友好度 48/100
a2aproject/a2a-java#1194 ·
维护者通常 1 天内回复
查看 a2aproject/a2a-java 的全部 Issue
相似的 Issue
-
[BUG] S3 CORS responses omit Access-Control-Allow-Credentials for matched origins可能已有人在做 关联的 PR 仍在进行中或已合并。 未关闭
难度 2/5 1-3 小时 新手友好度 72/100
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 72/100
sqlcipher/sqlcipher-android#97 · 1 条评论 ·
-
area-integrations
难度 2/5 1-3 小时 新手友好度 68/100
维护者通常 1 天内回复
-
bug IIIF interoperability
难度 2/5 1-3 小时 新手友好度 72/100
-
难度 2/5 1-3 小时 新手友好度 72/100