Similarityoung opened a new pull request, #158:
URL: https://github.com/apache/dubbo-go-pixiu-samples/pull/158

   ## Summary
   - Fixed YAML indentation in `header-conf.yaml`: Route 3's `route:` was 
indented 2 spaces too deep, making it a child of `match:` instead of a sibling. 
YAML parser silently ignored it, causing the cluster field to be empty and 
resulting in 404.
   - Fixed `TestHeaderGET2`: Added missing `X-B` header — `matchHeaders` uses 
AND logic (all headers must match), but the test only sent `X-C`.
   - Changed test URLs from `/` to `/user` for HeaderOnly routes, since the 
backend servers only handle `/user` path.
   
   ## Test plan
   - [x] TestHeaderGET1 (prefix + header): `GET /user` + `X-A:t1` → user-v1
   - [x] TestHeaderGET2 (header only, AND logic): `GET /user` + `X-B:t4` + 
`X-C:t1` → user-v2
   - [x] TestHeaderGET3 (header regex): `GET /user` + `REG:tt` → user-v3
   
   🤖 Generated with [Claude Code](https://claude.com/claude-code)


-- 
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]

Reply via email to