feat: path_rewrite redirect action#44920
Open
rudrakhp wants to merge 2 commits into
Open
Conversation
|
CC @envoyproxy/api-shepherds: Your approval is needed for changes made to |
rudrakhp
force-pushed
the
redirect_path_format
branch
from
b44c613 to
91da421
Compare
rudrakhp
force-pushed
the
redirect_path_format
branch
from
91da421 to
52888d7
Compare
Signed-off-by: Rudrakh Panigrahi <rudrakh97@gmail.com>
rudrakhp
force-pushed
the
redirect_path_format
branch
from
52888d7 to
73ae45b
Compare
wbpcode
reviewed
May 21, 2026
| // Would redirect to ``/new/v2`` given a request header ``x-version: v2``. | ||
| // If the substitution produces an empty string the path redirect is ignored | ||
| // and the original path is preserved. | ||
| string path_redirect_format = 10; |
Member
There was a problem hiding this comment.
Just call it path_rewrite because we have supported prefix_rewrite, regex_rewrite here, and it should be good to call new field as path_rewrite like our RouteAction.path_rewrite
Signed-off-by: Rudrakh Panigrahi <rudrakh97@gmail.com>
rudrakhp
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Commit Message: feat:
path_rewriteredirect actionAdditional Description: Implement support for a new
path_rewriteredirect action that supports substitution and CEL formatting to set the new URI for redirect.Risk Level: Low (new feature)
Testing: Unit testing
Docs Changes: N/A
Release Notes: Yes
Fixes: #44844