Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add config (path and method) for setHeaderRoute #3879

Open
danielsantosdfs13 opened this issue Oct 8, 2024 · 0 comments
Open

Add config (path and method) for setHeaderRoute #3879

danielsantosdfs13 opened this issue Oct 8, 2024 · 0 comments
Labels
canary Canary related issue enhancement New feature or request istio

Comments

@danielsantosdfs13
Copy link

danielsantosdfs13 commented Oct 8, 2024

Summary

Currently, setHeaderRoute adds a new route in the virtualService but only allows configuring a match with headerName and headerValue. It would be interesting to also allow configuring the method and path settings similar to setMirrorRoute.

Use Cases

Situations where the environment uses a common FQDN for more than one workload api.foo.local and routing is done using path /v1/bar, /v1/foo. With the current scenario, activating the setHeaderRoute will cause overlap and requests will be directed to the incorrect target.


Message from the maintainers:

Impacted by this bug? Give it a 👍. We prioritize the issues with the most 👍.

@danielsantosdfs13 danielsantosdfs13 added the enhancement New feature or request label Oct 8, 2024
@kostis-codefresh kostis-codefresh added istio canary Canary related issue labels Oct 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
canary Canary related issue enhancement New feature or request istio
Projects
None yet
Development

No branches or pull requests

2 participants