Welcome! Log In Create A New Profile

Advanced

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

Maxim Dounin
June 30, 2014 09:06PM
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/

_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-devel
Subject Author Views Posted

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

Piotr Sikora 761 June 27, 2014 02:42AM

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

Maxim Dounin 300 June 27, 2014 01:58PM

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

Brian Moran 360 June 27, 2014 02:06PM

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

Valentin V. Bartenev 311 June 30, 2014 08:42AM

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

Maxim Dounin 322 June 30, 2014 09:16PM

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

Piotr Sikora 361 June 30, 2014 05:02PM

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

Maxim Dounin 385 June 30, 2014 09:06PM

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

Yichun Zhang (agentzh) 278 June 30, 2014 07:26PM

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

Maxim Dounin 356 June 30, 2014 09:14PM



Sorry, you do not have permission to post/reply in this forum.

Online Users

Guests: 157
Record Number of Users: 8 on April 13, 2023
Record Number of Guests: 421 on December 02, 2018
Powered by nginx      Powered by FreeBSD      PHP Powered      Powered by MariaDB      ipv6 ready