Gentoo Archives: gentoo-dev

From: "Michał Górny" <mgorny@g.o>
To: gentoo-dev@l.g.o
Subject: Re: [gentoo-dev] Re: [PATCH 2/2] git-r3.eclass: Explicitly warn about unsecure protocols
Date: Sat, 19 Aug 2017 22:40:21
Message-Id: 1503182399.6857.0.camel@gentoo.org
In Reply to: [gentoo-dev] Re: [PATCH 2/2] git-r3.eclass: Explicitly warn about unsecure protocols by Duncan <1i5t5.duncan@cox.net>
1 W dniu sob, 19.08.2017 o godzinie 22∶01 +0000, użytkownik Duncan
2 napisał:
3 > Michał Górny posted on Sat, 19 Aug 2017 10:25:02 +0200 as excerpted:
4 >
5 > > Explicitly warn about any URI that uses an unsecure protocol (git, http)
6 > > even if it's a fallback URI. This is necessary because an attacker may
7 > > block HTTPS connections, effectively forcing the fallback to
8 > > the unsecure protocol.
9 >
10 > Thanks for this pair of patches. One minor correction, below.
11 >
12 > > eclass/git-r3.eclass | 11 ++++++++++-
13 > > 1 file changed, 10 insertions(+), 1 deletion(-)
14 > >
15 > > diff --git a/eclass/git-r3.eclass b/eclass/git-r3.eclass
16 > > index 42b586811368..1eb0baedc67f 100644
17 > > --- a/eclass/git-r3.eclass
18 > > +++ b/eclass/git-r3.eclass
19 > > @@ -570,6 +570,15 @@ git-r3_fetch() {
20 > >
21 > > [[ ${repos[@]} ]] || die "No URI provided and EGIT_REPO_URI unset"
22 > >
23 > > + local r
24 > > + for r in "${repos[@]}"; do
25 > > + if [[ ${r} == git:* || ${r} == http:* ]]; then
26 > > + ewarn "git-r3: ${r%%:*} protocol in unsafe and may be subject to MITM attacks"
27 >
28 > s/in unsafe/is unsafe/
29 >
30
31 Thanks, fixed locally.
32
33 --
34 Best regards,
35 Michał Górny