lkml.org 
[lkml]   [2020]   [May]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH] Replace HTTP links with HTTPS ones: documentation
From
Date


Am 17.05.20 um 21:45 schrieb Jonathan Corbet:
> On Sat, 16 May 2020 14:27:40 +0200
> "Alexander A. Klimov" <grandmaster@al2klimov.de> wrote:
>
>> ... for security reasons.
>>
>> No breaking changes as either the HTTP vhost redirects to HTTPS
>> or both vhosts redirect to the same location
>> or both serve the same content.
>
> We're getting closer, but...
>
> - There is still too much stuff here. Remember that somebody has to look
> at and review this stuff.
OK.

>
> - A quick check shows that a fair number of these links are broken or
> redirect to somewhere else. What is the value of adding "https" to a
> broken link?
Literally none. Either some vhosts broke since I have created the first
prototype of this or (more likely) I was wrong that checking for no
breaking changes at vhost level is enough.

>
> - Various documents have maintainers who are likely to be interested in
> changes and should be copied; that is what the get_maintainer.pl script
> is for. If that generates a massive list of recipients, that's a cue
> that your patch is too large.
Huh? I *did* run that script (automated) per file and grouped files per
identical set of patch recipients. I.e. this patch should contain only
changes for the recipients I've sent it to.
If your invocation of the script doesn't agree with me, please share the
output and how to reproduce (as if I had just my own patch eMail, the
upstream Git repo and all the tools of course).
Note that I'm not arguing with you, nor not believing you, exactly the
opposite: The more pieces I shall split this stuff into, the less errors
I want to make (and have to correct for every single piece) in the future.

>
> If you really want to push this forward, please:
>
> - narrow down further. Start with, say, Documentation/maintainer and
> just do that.
OK.

>
> - Make sure every link you touch actually works. If they don't, don't
> just add "https", figure out what the link should be or, if no
> applicable link exists, delete them.
OK, I'll switch from vhost level to link level (will take some time),
but I won't touch broken links. IMAO that's a different construction area.

>
> - Justify the changes in the changelog; "for security reasons" is not, by
> itself, particularly convincing. What security threat are you
> addressing here?
OK.

>
> Then, maybe, we'll have patches that can be reviewed and applied.
>
> Thanks,
>
> jon
>

\
 
 \ /
  Last update: 2020-05-17 22:19    [W:0.080 / U:0.136 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site