[PATCH] Core: use uppercase hexadecimal digits for percent-encoding

Maxim Dounin mdounin at mdounin.ru
Tue Jul 1 01:05:51 UTC 2014


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

More information about the nginx-devel mailing list