Case insensitive query params #4

Closed
wants to merge 15 commits into
from

2 participants

@grahamrhay

Apologies for the size of the pull request, the commits you might be interested in are the last 3.

@serialseb
openrasta member

URIs and their query parameters are case-sensitive by definition, which is important to ensure cache hits are processed correctly. I won't be applying this patch, as it'd make an invalid scenario the default.

I'd suggest normalizing your input URIs if you so wish using a UriDecorator.

@serialseb
openrasta member

Happy to cherrypick 8d9 and e833, won't apply 951. Will close this pull request when i've done this.

@serialseb
openrasta member

Shoud've been closed, it is now.

@serialseb serialseb closed this Oct 5, 2011
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment