-
Notifications
You must be signed in to change notification settings - Fork 591
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
feat: implement rewrite path prefix filter for HTTPRoute in traditional router mode #5895
Conversation
074427f
to
7e55454
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #5895 +/- ##
======================================
Coverage ? 74.2%
======================================
Files ? 179
Lines ? 18377
Branches ? 0
======================================
Hits ? 13654
Misses ? 3725
Partials ? 998 ☔ View full report in Codecov by Sentry. |
7a6f49d
to
4e50224
Compare
8cdbbea
to
497da9c
Compare
497da9c
to
5e090f0
Compare
internal/dataplane/translator/testdata/golden/httproute-url-rewrite-path-prefix/in.yaml
Outdated
Show resolved
Hide resolved
Co-authored-by: Jakub Warczarek <jakub.warczarek@konghq.com>
…write-path-prefix/in.yaml Co-authored-by: Jakub Warczarek <jakub.warczarek@konghq.com>
What this PR does / why we need it:
Implements
HTTPRoute
'sURLRewrite
filter withpath
typeReplacePrefixMatch
for the traditional router. EnablesSupportHTTPRoutePathRewrite
feature in Gateway API conformance tests.Support for
expressions
router will be implemented in a separate PR.Which issue this PR fixes:
Part of #3686.
PR Readiness Checklist:
Complete these before marking the PR as
ready to review
:CHANGELOG.md
release notes have been updated to reflect any significant (and particularly user-facing) changes introduced by this PR