[PATCH] Core: use uppercase hexadecimal digits for percent-encoding
Maxim Dounin
mdounin at mdounin.ru
Tue Jul 1 01:05:51 UTC 2014
Hello!
On Mon, Jun 30, 2014 at 02:01:30PM -0700, Piotr Sikora wrote:
> Hey Maxim,
>
> > I can't say I like this change. I've considered this a while ago,
> > and decided to keep it as is. This preserve compatibility with
> > what nginx used to do for years. And it also looks like Apache
> > does the same.
> >
> > Any other opinions?
>
> Compatibility for whom? This change is transparent for decoders
> (upper- and lowercase hex digits are equivalent, per RFC) and, as far
> as I can tell, it only affects people who try to encode URLs to match
> what nginx produces and/or do case-sensitive matching (like
> nginx-tests/autoindex.t).
So, you've already found at least one place which will be broken
by this change. And I would expect more.
--
Maxim Dounin
http://nginx.org/
More information about the nginx-devel
mailing list