Skipper: Skipper should allow URLs to be treated as case-insensitive using MatchingOption

[addityasingh]

The current default for skipper is to allow URLs to be case-sensitive. This creates a requirement for all the route definition to have all combinations of upper and lowercase for a given route.

Skipper should allow IgnoreCaseSensitive as a MatchingOptions so that we don't need to define multiple routes.

E.g. The wishlist route works for www.zalando.de/wishlist but not for www.zalando.de/WISHLIST

[aryszka]

i don't think this should be fixed. Depending on the particular requirements, it can be more reasonable to create a custom redirect filter extension that converts the path into lowercases.

lowercase: PathRegexp(/[^A-Z]*/) -> "https://www.example.org";
notLowerCase: * -> lowerCaseRedirect() -> <shunt>;

Other opinions?

[hjacobs]

I agree that a redirect to some canonical URL (being lowercase) is the best solution.

[lmineiro]

Another vote here for the redirect approach. This guarantees that route matching keeps its performance, which is fundamental, and also makes it transparent (bots?) what the canonical URL should be with a permanent redirect.

[addityasingh]

Makes sense. @aryszka I will check if its already there else create a PR for the lowercase filter