All of lore.kernel.org
 help / color / mirror / Atom feed
* github version complaints about the gitlab CI requirements.txt
@ 2023-11-12 20:33 Linus Torvalds
  2023-11-13 12:01 ` Helen Koike
  0 siblings, 1 reply; 2+ messages in thread
From: Linus Torvalds @ 2023-11-12 20:33 UTC (permalink / raw)
  To: Helen Koike, Maarten Lankhorst, Maxime Ripard, Thomas Zimmermann,
	Daniel Vetter, David Heidelberg, Vignesh Raman
  Cc: dri-devel

[-- Attachment #1: Type: text/plain, Size: 1217 bytes --]

So every time I push to my github mirror, github now ends up having a
'dependabot' thing that warns about some of the CI version
requirements for the gitlab automated testing file.

It wants to update the pip requirements from 23.2.1 to 23.3

 - When installing a package from a Mercurial VCS URL, e.g. pip install
   hg+..., with pip prior to v23.3, the specified Mercurial revision
   could be used to inject arbitrary configuration options to the hg
   clone call (e.g. --config). Controlling the Mercurial configuration
   can modify how and which repository is installed. This vulnerability
   does not affect users who aren't installing from Mercurial.

and upgrade the urllib3 requirements from 2.0.4 to 2.0.7:

 - urllib3's request body not stripped after redirect from 303 status
   changes request method to GET

 - `Cookie` HTTP header isn't stripped on cross-origin redirects

And it's not like any of this looks like a big deal, but I'd like to
shut up the messages I get.

I can either just close those issues, or I can apply a patch something
like the attached (which also adds a missing newline at the end).

I thought I should ask the people who actually set this up. Comments?

               Linus

[-- Attachment #2: patch.diff --]
[-- Type: text/x-patch, Size: 717 bytes --]

 drivers/gpu/drm/ci/xfails/requirements.txt | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/gpu/drm/ci/xfails/requirements.txt b/drivers/gpu/drm/ci/xfails/requirements.txt
index d8856d1581fd..e9994c9db799 100644
--- a/drivers/gpu/drm/ci/xfails/requirements.txt
+++ b/drivers/gpu/drm/ci/xfails/requirements.txt
@@ -5,7 +5,7 @@ termcolor==2.3.0
 certifi==2023.7.22
 charset-normalizer==3.2.0
 idna==3.4
-pip==23.2.1
+pip==23.3
 python-gitlab==3.15.0
 requests==2.31.0
 requests-toolbelt==1.0.0
@@ -13,5 +13,5 @@ ruamel.yaml==0.17.32
 ruamel.yaml.clib==0.2.7
 setuptools==68.0.0
 tenacity==8.2.3
-urllib3==2.0.4
-wheel==0.41.1
\ No newline at end of file
+urllib3==2.0.7
+wheel==0.41.1

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: github version complaints about the gitlab CI requirements.txt
  2023-11-12 20:33 github version complaints about the gitlab CI requirements.txt Linus Torvalds
@ 2023-11-13 12:01 ` Helen Koike
  0 siblings, 0 replies; 2+ messages in thread
From: Helen Koike @ 2023-11-13 12:01 UTC (permalink / raw)
  To: Linus Torvalds, Maarten Lankhorst, Maxime Ripard,
	Thomas Zimmermann, Daniel Vetter, David Heidelberg,
	Vignesh Raman
  Cc: dri-devel

Hi Linus,

On 12/11/2023 17:33, Linus Torvalds wrote:
> So every time I push to my github mirror, github now ends up having a
> 'dependabot' thing that warns about some of the CI version
> requirements for the gitlab automated testing file.
> 
> It wants to update the pip requirements from 23.2.1 to 23.3
> 
>   - When installing a package from a Mercurial VCS URL, e.g. pip install
>     hg+..., with pip prior to v23.3, the specified Mercurial revision
>     could be used to inject arbitrary configuration options to the hg
>     clone call (e.g. --config). Controlling the Mercurial configuration
>     can modify how and which repository is installed. This vulnerability
>     does not affect users who aren't installing from Mercurial.
> 
> and upgrade the urllib3 requirements from 2.0.4 to 2.0.7:
> 
>   - urllib3's request body not stripped after redirect from 303 status
>     changes request method to GET
> 
>   - `Cookie` HTTP header isn't stripped on cross-origin redirects
> 
> And it's not like any of this looks like a big deal, but I'd like to
> shut up the messages I get.
> 
> I can either just close those issues, or I can apply a patch something
> like the attached (which also adds a missing newline at the end).
> 
> I thought I should ask the people who actually set this up. Comments?

I just tested your attached patch and looks fine, the scripts with those
requirements worked as expected, so please go ahead with your patch with
Tested-by: Helen Koike <helen.koike@collabora.com>

Now I'm thinking how to prevent those warnings in the future.

Thank you,
Helen

> 
>                 Linus

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2023-11-13 12:01 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-11-12 20:33 github version complaints about the gitlab CI requirements.txt Linus Torvalds
2023-11-13 12:01 ` Helen Koike

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.