Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
 help / color / mirror / Atom feed
From: Michael Niedermayer <michael@niedermayer.cc>
To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org>
Subject: Re: [FFmpeg-devel] [PATCH] ipfsgateway: Remove default gateway
Date: Mon, 15 Aug 2022 23:47:53 +0200
Message-ID: <20220815214753.GU2088045@pb2> (raw)
In-Reply-To: <7e0c471e-992d-9bed-8f47-7ce3401f9863@gmail.com>


[-- Attachment #1.1: Type: text/plain, Size: 2812 bytes --]

On Mon, Aug 15, 2022 at 08:35:18PM +0100, Derek Buitenhuis wrote:
> On 8/10/2022 11:27 PM, Derek Buitenhuis wrote:
> > A gateway can see everything, and we should not be shipping a hardcoded
> > default from a third party company; it's a security risk.
> > 
> > Signed-off-by: Derek Buitenhuis <derek.buitenhuis@gmail.com>
> > ---
> >  libavformat/ipfsgateway.c | 11 ++++-------
> >  1 file changed, 4 insertions(+), 7 deletions(-)
> 
> I've been asked by almost all the active developers on FFmpeg at this
> point to push this while we debate the error message, gateway list / solution,
> nuking, etc.
> 
> I quintuple checked on IRC it was OK for me to push in the meantime. Logs are
> there.
> 
> Many distros/packagers have already backported this patch to 5.1 themselves,
> as they really (shockingly) do not want a default gateway.
> 
> So: Pushed.
> 
> Let us continue the discusion on the other aspects of this topic.
> 
> (Please do not send me hate mail or harassment. I really did check.)

I just like to note that many of these statments are not untrue but also
not exactly true the way they are written. That begins with the commit message
"it's a security risk.", sure it is one for some definiton of risk but 
after this patch there is more risk in practice for the average end user

I would be carefull with the distros which backported this. Honestly
i would question these distros security more than the security of the
gateway. Because I dont think they reviewed this 
It says this now:
                   "IPFS does not appear to be running.\n\n"
                   "Installing IPFS locally is recommended to "
                   "improve performance and reliability, "
That removes the gateway (because it cant be trusted and replaced it
by a litteral recommandition to install their software)
replace loging by code exec ... either you trust them or not

No, iam not angry or anything like that at all. Iam also not asking for any
revert or anything and i fully aknowledge that there seems to be a clear majority 
for the removal of the default.
I just have to point this out because i think something went a bit wrong here.

Also some of the argumentation today on IRC about crypto & NFTs felt a
bit like the "distant dark past" where random projects and random people where
attacked.
If one was concerned that using a default gateway could be seen
as endorsment by us. Be concerned please about how FFmpeg looks
when its developers attack other projects on its official development IRC channels.

thx


[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

If a bugfix only changes things apparently unrelated to the bug with no
further explanation, that is a good sign that the bugfix is wrong.

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 195 bytes --]

[-- Attachment #2: Type: text/plain, Size: 251 bytes --]

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

  parent reply	other threads:[~2022-08-15 21:48 UTC|newest]

Thread overview: 57+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-08-10 22:27 Derek Buitenhuis
2022-08-11 12:08 ` Timo Rothenpieler
2022-08-11 16:26   ` Mark Gaiser
2022-08-11 16:49     ` Timo Rothenpieler
2022-08-11 17:21       ` Mark Gaiser
2022-08-11 17:35         ` Timo Rothenpieler
2022-08-11 17:56           ` Mark Gaiser
2022-08-11 19:18             ` Derek Buitenhuis
2022-08-11 20:18             ` Michael Niedermayer
2022-08-11 22:03               ` Timo Rothenpieler
2022-08-11 22:51                 ` Derek Buitenhuis
2022-08-12 13:43                   ` Mark Gaiser
2022-08-12 14:22                   ` Vittorio Giovara
2022-08-12 14:30                     ` Kieran Kunhya
2022-08-12 14:34                       ` Mark Gaiser
2022-08-12 14:45                         ` Kieran Kunhya
2022-08-12 14:48                         ` Derek Buitenhuis
2022-08-12 14:50                           ` Kieran Kunhya
2022-08-12 14:55                   ` Nicolas George
2022-08-12 15:05                 ` Michael Niedermayer
2022-08-12 17:01                   ` Nicolas George
2022-08-12 17:18                     ` Michael Niedermayer
2022-08-12 17:21                       ` Timo Rothenpieler
2022-08-13 16:29                         ` Michael Niedermayer
2022-08-13 19:06                           ` Timo Rothenpieler
2022-08-14 18:00                             ` Michael Niedermayer
2022-08-15 14:09                           ` Nicolas George
2022-08-15 14:27                             ` Jean-Baptiste Kempf
2022-08-17 15:03           ` Tomas Härdin
2022-08-18 14:31             ` Michael Niedermayer
2022-08-19  9:15               ` Tomas Härdin
2022-08-19 12:52                 ` Mark Gaiser
2022-08-22  9:12                   ` Tomas Härdin
2022-08-22 12:52                     ` Nicolas George
2022-08-23 12:53                       ` Ronald S. Bultje
2022-08-23 12:55                         ` Nicolas George
2022-08-24 16:35                       ` Tomas Härdin
2022-08-24 20:54                         ` Michael Niedermayer
2022-08-27  7:05                           ` Tomas Härdin
2022-08-28 14:14                             ` Michael Niedermayer
2022-08-24 21:03                         ` Michael Niedermayer
2022-08-24 21:18                           ` Kieran Kunhya
2022-08-25 13:57                             ` Michael Niedermayer
2022-08-25 14:41                               ` Kieran Kunhya
2022-08-27  7:29                           ` Tomas Härdin
2022-08-27  7:53                             ` Paul B Mahol
2022-08-27 11:30                               ` Tomas Härdin
2022-08-27 17:34                                 ` Baptiste Coudurier
2022-08-28 11:49                                   ` Tomas Härdin
2022-08-15 17:53 ` Michael Niedermayer
2022-08-15 19:35 ` Derek Buitenhuis
2022-08-15 19:37   ` James Almer
2022-08-15 21:47   ` Michael Niedermayer [this message]
2022-08-15 21:57     ` Nicolas George
2022-08-15 23:53       ` Mark Gaiser
2022-08-16 14:46     ` Michael Niedermayer
2022-08-14 13:24 thelostone123

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=20220815214753.GU2088045@pb2 \
    --to=michael@niedermayer.cc \
    --cc=ffmpeg-devel@ffmpeg.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: link

Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \
		ffmpegdev@gitmailbox.com
	public-inbox-index ffmpegdev

Example config snippet for mirrors.


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git