8000 Add exact matches and regex support to location paths in VS/VSR by Rulox · Pull Request #766 · nginx/kubernetes-ingress · GitHub
[go: up one dir, main page]

Skip to content

Conversation

Rulox
Copy link
Contributor
@Rulox Rulox commented Nov 18, 2019

Proposed changes

Add the ability to use exact matches (=) and case sensitive/insensitive regular expressions (~, ~*) in VS/VSR route paths.

@tellet Please have a look at the Test methods and let me know if more edge cases can be added to improve validation, thank you.

@Rulox Rulox added the enhancement Pull requests for new features/feature enhancements label Nov 18, 2019
Copy link
Contributor
@pleshakov pleshakov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Rulox thanks! please see my comments and suggestions

@Rulox Rulox requested review from pleshakov and tellet November 20, 2019 17:06
Copy link
Contributor
@pleshakov pleshakov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

a small doc bug and a suggestion. otherwise, lgtm! thanks!

Copy link
Contributor
@tellet tellet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@Rulox Rulox merged commit c5196a8 into master Nov 22, 2019
@Rulox Rulox deleted the location-regex branch November 22, 2019 12:34
@pleshakov pleshakov changed the title Add regex support to location paths Add exact matches and regex support to location paths Dec 17, 2019
@pleshakov pleshakov changed the title Add exact matches and regex support to location paths Add exact matches and regex support to location paths in VS/VSR Dec 17, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Pull requests for new features/feature enhancements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants
0