AlexStocks commented on code in PR #158: URL: https://github.com/apache/dubbo-go-pixiu-samples/pull/158#discussion_r3793191852
########## dubbogo/simple/traffic/test/pixiu_header_test.go: ########## @@ -0,0 +1,75 @@ +//go:build manual Review Comment: [P1] 这组回归测试既未进入 CI，也会接受错误的 header 路由行为。该文件被 `manual` tag 排除，而当前集成流程只用 `pixiu/conf.yaml` 运行 `TestCanary*`；exact-Head CI 日志中没有任何 `TestHeader*`。同时，`header-conf.yaml` 的 v1 路由要求 `X-A` 和 `X-B`，但 `TestHeaderGET1` 只发送 `X-A`。我用该 workflow 对应且路由代码未变化的 Pixiu develop 实际启动此配置后，无 header 和仅 `X-A` 的公开 HTTP 请求都错误返回 v1，随后这里三条 manual 测试仍全部通过。这说明测试会放过“prefix 路由忽略 headers”的实现。请把 `header-conf.yaml` 场景接入 CI；正例同时发送 `X-A`/`X-B`，并增加缺少 `X-B` 时不得命中 v1 的负例。 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
