From: "Alexander A. Klimov" <grandmaster@al2klimov.de> To: Laurent Pinchart <laurent.pinchart@ideasonboard.com> Cc: hyun.kwon@xilinx.com, mchehab@kernel.org, robh+dt@kernel.org, michal.simek@xilinx.com, linux-media@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] Replace HTTP links with HTTPS ones: Xilinx video IP cores Date: Wed, 15 Jul 2020 08:33:35 +0200 [thread overview] Message-ID: <5fca4c17-9e83-3936-0d8f-ad73ae760033@al2klimov.de> (raw) In-Reply-To: <20200714235724.GR5854@pendragon.ideasonboard.com> Am 15.07.20 um 01:57 schrieb Laurent Pinchart: > Hi Alexander, > > Thank you for the patch. > > The subject line should be > > dt-bindings: media: xilinx: Replace HTTP links with HTTPS ones > > On Sun, Jul 05, 2020 at 09:55:24AM +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 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@al2klimov.de> > > Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > > Will you get this patch merged as part of a bigger series, or do you > expect us to take care of it ? In the latter case, if you agree with the I've just followed the patch submission instructions as for any non-treewide patch. So yes, I assume the people I write to take care of the patch. > subject line change, there's no need to resubmit, I can fix this when > applying the patch. OK. > >> --- >> Continuing my work started at 93431e0607e5. >> >> If there are any URLs to be removed completely or at least not HTTPSified: >> Just clearly say so and I'll *undo my change*. >> See also https://lkml.org/lkml/2020/6/27/64 >> >> If there are any valid, but yet not changed URLs: >> See https://lkml.org/lkml/2020/6/26/837 >> >> Documentation/devicetree/bindings/media/xilinx/video.txt | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/Documentation/devicetree/bindings/media/xilinx/video.txt b/Documentation/devicetree/bindings/media/xilinx/video.txt >> index 68ac210e688e..d0335ca0cd57 100644 >> --- a/Documentation/devicetree/bindings/media/xilinx/video.txt >> +++ b/Documentation/devicetree/bindings/media/xilinx/video.txt >> @@ -32,4 +32,4 @@ The following properties are common to all Xilinx video IP cores. >> defaults to "mono". >> >> >> -[UG934] http://www.xilinx.com/support/documentation/ip_documentation/axi_videoip/v1_0/ug934_axi_videoIP.pdf >> +[UG934] https://www.xilinx.com/support/documentation/ip_documentation/axi_videoip/v1_0/ug934_axi_videoIP.pdf >
WARNING: multiple messages have this Message-ID (diff)
From: "Alexander A. Klimov" <grandmaster@al2klimov.de> To: Laurent Pinchart <laurent.pinchart@ideasonboard.com> Cc: devicetree@vger.kernel.org, hyun.kwon@xilinx.com, michal.simek@xilinx.com, linux-kernel@vger.kernel.org, robh+dt@kernel.org, mchehab@kernel.org, linux-arm-kernel@lists.infradead.org, linux-media@vger.kernel.org Subject: Re: [PATCH] Replace HTTP links with HTTPS ones: Xilinx video IP cores Date: Wed, 15 Jul 2020 08:33:35 +0200 [thread overview] Message-ID: <5fca4c17-9e83-3936-0d8f-ad73ae760033@al2klimov.de> (raw) In-Reply-To: <20200714235724.GR5854@pendragon.ideasonboard.com> Am 15.07.20 um 01:57 schrieb Laurent Pinchart: > Hi Alexander, > > Thank you for the patch. > > The subject line should be > > dt-bindings: media: xilinx: Replace HTTP links with HTTPS ones > > On Sun, Jul 05, 2020 at 09:55:24AM +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 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@al2klimov.de> > > Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > > Will you get this patch merged as part of a bigger series, or do you > expect us to take care of it ? In the latter case, if you agree with the I've just followed the patch submission instructions as for any non-treewide patch. So yes, I assume the people I write to take care of the patch. > subject line change, there's no need to resubmit, I can fix this when > applying the patch. OK. > >> --- >> Continuing my work started at 93431e0607e5. >> >> If there are any URLs to be removed completely or at least not HTTPSified: >> Just clearly say so and I'll *undo my change*. >> See also https://lkml.org/lkml/2020/6/27/64 >> >> If there are any valid, but yet not changed URLs: >> See https://lkml.org/lkml/2020/6/26/837 >> >> Documentation/devicetree/bindings/media/xilinx/video.txt | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/Documentation/devicetree/bindings/media/xilinx/video.txt b/Documentation/devicetree/bindings/media/xilinx/video.txt >> index 68ac210e688e..d0335ca0cd57 100644 >> --- a/Documentation/devicetree/bindings/media/xilinx/video.txt >> +++ b/Documentation/devicetree/bindings/media/xilinx/video.txt >> @@ -32,4 +32,4 @@ The following properties are common to all Xilinx video IP cores. >> defaults to "mono". >> >> >> -[UG934] http://www.xilinx.com/support/documentation/ip_documentation/axi_videoip/v1_0/ug934_axi_videoIP.pdf >> +[UG934] https://www.xilinx.com/support/documentation/ip_documentation/axi_videoip/v1_0/ug934_axi_videoIP.pdf > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2020-07-15 6:33 UTC|newest] Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-07-05 7:55 [PATCH] Replace HTTP links with HTTPS ones: Xilinx video IP cores Alexander A. Klimov 2020-07-05 7:55 ` Alexander A. Klimov 2020-07-08 10:48 ` Michal Simek 2020-07-08 10:48 ` Michal Simek 2020-07-14 23:57 ` Laurent Pinchart 2020-07-14 23:57 ` Laurent Pinchart 2020-07-15 6:33 ` Alexander A. Klimov [this message] 2020-07-15 6:33 ` Alexander A. Klimov 2020-07-15 21:44 ` Rob Herring 2020-07-15 21:44 ` Rob Herring
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=5fca4c17-9e83-3936-0d8f-ad73ae760033@al2klimov.de \ --to=grandmaster@al2klimov.de \ --cc=devicetree@vger.kernel.org \ --cc=hyun.kwon@xilinx.com \ --cc=laurent.pinchart@ideasonboard.com \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-media@vger.kernel.org \ --cc=mchehab@kernel.org \ --cc=michal.simek@xilinx.com \ --cc=robh+dt@kernel.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.