From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org [79.124.17.100]) by master.gitmailbox.com (Postfix) with ESMTP id 69BFC40C43 for ; Wed, 9 Mar 2022 09:35:53 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 335A168B081; Wed, 9 Mar 2022 11:35:51 +0200 (EET) Received: from relay12.mail.gandi.net (relay12.mail.gandi.net [217.70.178.232]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 02C6268AC6D for ; Wed, 9 Mar 2022 11:35:44 +0200 (EET) Received: from localhost (213-47-68-29.cable.dynamic.surfer.at [213.47.68.29]) (Authenticated sender: michael@niedermayer.cc) by mail.gandi.net (Postfix) with ESMTPSA id 17969200009 for ; Wed, 9 Mar 2022 09:35:43 +0000 (UTC) Date: Wed, 9 Mar 2022 10:35:43 +0100 From: Michael Niedermayer To: FFmpeg development discussions and patches Message-ID: <20220309093543.GT2829255@pb2> References: <9d46b2e33342cef9793b1c71fb5a60aac9108ea1.camel@acc.umu.se> <20220301220134.GX2829255@pb2> <20220304180916.GD2829255@pb2> <20220308234522.GS2829255@pb2> MIME-Version: 1.0 In-Reply-To: Subject: Re: [FFmpeg-devel] [PATCH v8 1/1] avformat: Add IPFS protocol support. X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: FFmpeg development discussions and patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: FFmpeg development discussions and patches Content-Type: multipart/mixed; boundary="===============4233884193051553071==" Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Archived-At: List-Archive: List-Post: --===============4233884193051553071== Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="KIpW+QI0GzdoiyXk" Content-Disposition: inline --KIpW+QI0GzdoiyXk Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Wed, Mar 09, 2022 at 01:30:30AM +0100, Mark Gaiser wrote: > On Wed, Mar 9, 2022 at 12:45 AM Michael Niedermayer > wrote: >=20 > > On Tue, Mar 08, 2022 at 01:49:22PM +0100, Mark Gaiser wrote: > > > On Fri, Mar 4, 2022 at 7:09 PM Michael Niedermayer < > > michael@niedermayer.cc> > > > wrote: > > > > > > > On Thu, Mar 03, 2022 at 03:58:53PM +0100, Mark Gaiser wrote: > > > > > On Tue, Mar 1, 2022 at 11:01 PM Michael Niedermayer < > > > > michael@niedermayer.cc> > > > > > wrote: > > > > > > > > > > > On Mon, Feb 28, 2022 at 02:09:15PM +0100, Tomas H=E4rdin wrote: > > > > > > > s=F6n 2022-02-27 klockan 15:29 +0100 skrev Mark Gaiser: > > > > > > > > Ping 2.... > > > > > > > > > > > > > > > > I'd really like to get this merged! > > > > > > > > This kinda blocks me right now from proceeding with IPFS > > > > integration > > > > > > > > in > > > > > > > > Kodi, MPV and VLC. Implementations in those (who rely on > > ffmpeg) > > > > are > > > > > > > > significantly easier once this patch is finally landed in > > ffmpeg. > > > > > > > > > > > > > > I'd like to hear at least one other dev chime in on this one > > > > > > > > > > > > what exactly are you not sure about ? > > > > > > what exactly needs a 2nd look ? > > > > > > > > > > > > > > > > My assumption. > > > > > In general just a second look by someone other than Tomas. > > > > > And, as he was skeptical about this patch at first, likely another > > > > opinion > > > > > if this makes sense to add in ffmpeg. > > > > > To me it does very much but i'm biased :) > > > > > > > > ipfs support makes sense to be added to ffmpeg. ive seen ipfs urls = and > > ive > > > > already been annoyed that some tools dont "just" work with them. > > > > While if i compare this to many other formats which i have never se= en > > > > outside the context of FFmpeg. So from this biased single sample th= at i > > > > am, ipfs seems more widespread and thats why iam in favor of its > > support > > > > > > > > thx > > > > > > > > Great to have your support :) > > > Reading that is quite motivating to work on it, no joke! > > > > > > Just to be clear here. Having this in ffmpeg won't make it "just work" > > yet. > > > For a minimal feeling of "hey, it works out of the box" you'd need: > > > - The next or version after the next IPFS. > > > - MPV support which relies on this patch to even be supported in mpv > > > - Have a node running locally > > > > if theres no local node it should fallback to a public node > > ATM > > IPFS_GATEWAY=3Dhttps://dweb.link ./ffplay ipfs://... > > works > > so such a fallback is all thats needed for it to just work > > >=20 > Yes, the beauty of gateways. >=20 > Are you suggesting that I update the patch to add this default? Iam not sure > I would prefer not to add that even though it would give a feeling of "ju= st > works". > I'm mostly concerned about the bandwidth usage it could cause on that sit= e. you could add more than one you could point people to https://ipfs.io/ipns/ipnso.com/ to let them select their own or maybe theres a better way > But also about potential hacks. If this is a default and well used then it > becomes quite appealing for hackers to take control of dweb.link and send > back data that wasn't requested. Thats a valid concern but not adding a default is not really solving this Because what do most people do then ? They google for a gateway and pick the first that works.=20 Thats plausibly not going to give them the fastest nor the most secure nor= =20 even not the same as the last million people searching. To setup a ipfs node on ones own machiene, first it would be needed that th= is is VERY simple and clean if i run "apt search ipfs", theres none, so that already fails here but for the fun, i tried to search for ipfs node on the app store on my iph= one no, also nothing. so i dont think "install an ipfs node" is really a viable solution for the average joe. we have a wide range of platforms, linux, windows, android, ios just to name the major ones. This protocol should work on all of them. I think a default gateway is the easy way to make that happen, asking the user to set a gateway will already leave android and iphone users probably wondering how to do that. Is there a way ? So really iam not saying "add a default", iam really saying "make it work for everyone", its ok if the user has to choose one or set some default but it really has to work on all platforms and with all user apps using libavformat. It should not be specific to mpv or windows/linux you can also print a big nasty warning that a default is used and the user should really setup their own node and why that is better. thx >=20 > If you insist this would be really better to add then I'll need to go find > the ones managing that site (paying for it) to ask permission if this wou= ld > be allowed. [...] --=20 Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Complexity theory is the science of finding the exact solution to an approximation. Benchmarking OTOH is finding an approximation of the exact --KIpW+QI0GzdoiyXk Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iF0EABEIAB0WIQSf8hKLFH72cwut8TNhHseHBAsPqwUCYih06wAKCRBhHseHBAsP q7g0AJ93b3cb7YW50XP4Su5w3Z0yfcGeFQCfXf3+stdRikpvfDe2Sgluywpkb1s= =sPXz -----END PGP SIGNATURE----- --KIpW+QI0GzdoiyXk-- --===============4233884193051553071== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ 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". --===============4233884193051553071==--