Welcome! Log In Create A New Profile

Advanced

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

Piotr Sikora
June 30, 2014 05:02PM
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).

The code in both: nginx and Apache predates RFC, which explains why it
wasn't uppercase from the beginning, but since RFC is out, there is no
good reason for keeping it this way.

Best regards,
Piotr Sikora

_______________________________________________
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 321 June 30, 2014 09:16PM

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

Piotr Sikora 360 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: 159
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