Re: [PATCH] Replace HTTP links with HTTPS ones: Documentation/sphinx/parse-headers.pl

From: Jonathan Corbet
Date: Fri Jun 26 2020 - 12:38:04 EST


On Sat, 20 Jun 2020 09:54:02 +0200
"Alexander A. Klimov" <grandmaster@xxxxxxxxxxxx> wrote:

> Rationale:
> Reduces attack surface on kernel devs opening the links for MITM
> as HTTPS traffic is much harder to manipulate.
>
> Deterministic algorithm:
> For each file:
> For each line:
> If doesn't contain `\bxmlns\b`:
> For each link, `\bhttp://[^# \t\r\n]*(?:\w|/)`:
> If both the HTTP and HTTPS versions
> return 200 OK and serve the same content:
> Replace HTTP with HTTPS.
>
> Signed-off-by: Alexander A. Klimov <grandmaster@xxxxxxxxxxxx>
> ---
> Documentation/sphinx/parse-headers.pl | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Documentation/sphinx/parse-headers.pl b/Documentation/sphinx/parse-headers.pl
> index c518050ffc3f..00a69aceff44 100755
> --- a/Documentation/sphinx/parse-headers.pl
> +++ b/Documentation/sphinx/parse-headers.pl
> @@ -393,7 +393,7 @@ Report bugs to Mauro Carvalho Chehab <mchehab@xxxxxxxxxx>
>
> Copyright (c) 2016 by Mauro Carvalho Chehab <mchehab+samsung@xxxxxxxxxx>.
>
> -License GPLv2: GNU GPL version 2 <http://gnu.org/licenses/gpl.html>.
> +License GPLv2: GNU GPL version 2 <https://gnu.org/licenses/gpl.html>.
>
> This is free software: you are free to change and redistribute it.
> There is NO WARRANTY, to the extent permitted by law.

I've applied this, but I think the correct fix is to replace that notice
with an SPDX header.

jon