-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Support RegularExpression for path matching #10717
Support RegularExpression for path matching #10717
Conversation
Hello @dmavrommatis and thanks for your contribution, As this is an enhancement, please could you rebase this pull request on master? |
c41e48a
to
8245755
Compare
Changed the target branch to master and rebased. Previously I chose v3.0 branch because of the PR guide advising to use the v3.0 branch for enchancements.
|
8245755
to
749930d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
What does this PR do?
Pull request introduced support for path matching of type
RegularExpression
from Gateway API.Motivation
Moving from traefik/IngressRoute to gateway/HttpRoute is not possible since we are using a lot of
PathRegexp
which is not yet supported.More
Additional Notes