Re: [PATCH] Replace HTTP links with HTTPS ones: USB MASS STORAGE DRIVER

From: Greg KH
Date: Wed Jul 08 2020 - 06:39:34 EST


On Wed, Jul 08, 2020 at 11:55:00AM +0200, Alexander A. Klimov 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:
> If not .svg:
> For each line:
> If doesn't contain `\bxmlns\b`:
> For each link, `\bhttp://[^# \t\r\n]*(?:\w|/)`:
> If neither `\bgnu\.org/license`, nor `\bmozilla\.org/MPL\b`:
> 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>

Your subject lines are very odd compared to all patches for this
subsystem, as well as all other kernel subsystems. Any reason you are
doing it this way and not the normal and standard method of:
USB: storage: replace http links with https

That would look more uniform as well as not shout at anyone.

thanks,

greg k-h