-
Notifications
You must be signed in to change notification settings - Fork 5k
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
Fix capitalization of PathPrefixStrip in kubernetes doc #1695
Conversation
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 a lot, LGTM.
Note to reviewers: this was implicitly changed through #1630 where we removed the custom rules list from the k8s provider. While the change of behavior represents a bit of a breaking change, I think there's too little value in holding on to the lower case variants. |
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
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 👼
be9b804
to
9ab8cd8
Compare
9ab8cd8
to
d958a27
Compare
Description
This changes the doc and example from pathprefixstrip to PathPrefixStrip.
With v1.3 pathprefixstrip does not work, with the following error: