Re: [PATCH] Replace HTTP links with HTTPS ones: Documentation/process

2020-06-26 Thread Jonathan Corbet
On Sun, 21 Jun 2020 15:36:30 +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

Re: [PATCH] Replace HTTP links with HTTPS ones: Documentation/process

2020-06-22 Thread Alexander A. Klimov
Am 22.06.20 um 22:06 schrieb Miguel Ojeda: On Mon, Jun 22, 2020 at 7:29 PM Joe Perches wrote: scripts/get_maintainer.pl --self-test=links has a reachability test using wget. Perhaps a script like that could be used for http:// vs https:// +1 Not sure about `--no-check-certificate` if th

Re: [PATCH] Replace HTTP links with HTTPS ones: Documentation/process

2020-06-22 Thread Miguel Ojeda
On Mon, Jun 22, 2020 at 7:29 PM Joe Perches wrote: > > scripts/get_maintainer.pl --self-test=links has a reachability test > using wget. > > Perhaps a script like that could be used for http:// vs https:// +1 Not sure about `--no-check-certificate` if the goal is to move to "proper HTTPS". Perha

Re: [PATCH] Replace HTTP links with HTTPS ones: Documentation/process

2020-06-22 Thread Joe Perches
On Mon, 2020-06-22 at 19:03 +0200, Miguel Ojeda wrote: > On Mon, Jun 22, 2020 at 3:06 PM Jonathan Corbet wrote: > > As has been noted elsewhere, checkpatch.pl seems like the appropriate > > place to make this check. As for "the entire tree"...if this job gets > > completed, "git grep" should be a

Re: [PATCH] Replace HTTP links with HTTPS ones: Documentation/process

2020-06-22 Thread Miguel Ojeda
On Mon, Jun 22, 2020 at 3:06 PM Jonathan Corbet wrote: > > As has been noted elsewhere, checkpatch.pl seems like the appropriate > place to make this check. As for "the entire tree"...if this job gets > completed, "git grep" should be a fine way to do that. `checkpatch` is not really enforced in

Re: [PATCH] Replace HTTP links with HTTPS ones: Documentation/process

2020-06-22 Thread Jonathan Corbet
On Mon, 22 Jun 2020 01:43:12 +0200 Miguel Ojeda wrote: > > * The script should not be neccessary once all of my changes[1] arrive > > in torvalds/master. Instead reviewers should say like C'mon dude, what's > > this new plain-HTTP link doing in your patch? We have 2020! Look at e.g. > > 93431e060

Re: [PATCH] Replace HTTP links with HTTPS ones: Documentation/process

2020-06-21 Thread Miguel Ojeda
Hi Alexander, On Sun, Jun 21, 2020 at 4:30 PM Alexander A. Klimov wrote: > > Which discussion? 93431e0607e5 ? IMAO the patches don't depend on each > other. The one we had the other day. It does not matter that the patches depend on each other. It is information for whoever sees this commit. >

Re: [PATCH] Replace HTTP links with HTTPS ones: Documentation/process

2020-06-21 Thread Alexander A. Klimov
Am 21.06.20 um 19:42 schrieb Joe Perches: On Sun, 2020-06-21 at 16:30 +0200, Alexander A. Klimov wrote: Am 21.06.20 um 15:46 schrieb Miguel Ojeda: On Sun, Jun 21, 2020 at 3:37 PM Alexander A. Klimov wrote: [] Replace HTTP with HTTPS. [] W/o a such central "rule on how to H

Re: [PATCH] Replace HTTP links with HTTPS ones: Documentation/process

2020-06-21 Thread Joe Perches
On Sun, 2020-06-21 at 16:30 +0200, Alexander A. Klimov wrote: > Am 21.06.20 um 15:46 schrieb Miguel Ojeda: > > On Sun, Jun 21, 2020 at 3:37 PM Alexander A. Klimov > > wrote: [] > > > Replace HTTP with HTTPS. [] > W/o a > such central "rule on how to HTTPSify links" Maybe: --- scrip

Re: [PATCH] Replace HTTP links with HTTPS ones: Documentation/process

2020-06-21 Thread Alexander A. Klimov
Am 21.06.20 um 15:46 schrieb Miguel Ojeda: Hi Alexander, On Sun, Jun 21, 2020 at 3:37 PM 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

Re: [PATCH] Replace HTTP links with HTTPS ones: Documentation/process

2020-06-21 Thread Miguel Ojeda
Hi Alexander, On Sun, Jun 21, 2020 at 3:37 PM 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: > I

[PATCH] Replace HTTP links with HTTPS ones: Documentation/process

2020-06-21 Thread Alexander A. Klimov
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|/)`: