googleapis / googleapis/google-cloud-java

[sdk-platform-java] Support `:**` for wildcard matching in routing parameters

未关闭
#12,500 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看
priority: p2 type: feature request
主要语言
Java
星标
2.1k
派生
1.2k
平均合并
1 天 23 小时
30 天内合并 PR
157

描述

Storage has a need to support `:**` in the end of a routing parameter, see [proto config](https://github.com/googleapis/googleapis/blob/533398389c34b567e3ca11775fa127a111258ea1/google/storage/v2/storage.proto#L255). This was first found in [Ruby](https://b.corp.google.com/issues/277650689), but Java does not support it either after manual testing.
The original design of routing proto is not clear if this format should be supported or not. See [AIP-4222](https://google.aip.dev/client-libraries/4222) and [routing.proto](https://github.com/googleapis/googleapis/blob/533398389c34b567e3ca11775fa127a111258ea1/google/api/routing.proto) for details. There is an open [PR](https://github.com/aip-dev/google.aip.dev/pull/1067/files) to update the AIP to make it clear.
The current implementation in [PathTemplate](https://github.com/googleapis/gapic-generator-java/blob/main/api-common-java/src/main/java/com/google/api/pathtemplate/PathTemplate.java) only support `**` after `\`, we need to treat `:` as a proper delimeter as well.

贡献指南

打开贡献指南

调研方向

从 api-common-java/src/main/java/com/google/api/pathtemplate/PathTemplate.java 开始,检查当前实现如何处理 `\\` 后的 `**`。将该行为与 storage proto 示例和 AIP-4222 进行比较;当 Java 按 routing format 的要求支持 routing parameter 末尾的 `:**` 时,即表示完成。

由索引模型根据 Issue 内容生成。

评估

技术栈
java
领域
api
Issue 类型
缺陷
难度
3/5
预计耗时
1-2 天
活跃度
停滞
描述清晰度
基本清楚
新手友好度
45/100

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。