[FFmpeg-devel] [PATCH v8 1/1] avformat: Add IPFS protocol support.

[Mark Gaiser]

On Wed, Mar 9, 2022 at 12:45 AM Michael Niedermayer <michael at niedermayer.cc>
wrote:

> 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 at 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 at niedermayer.cc>
> > > > wrote:
> > > >
> > > > > On Mon, Feb 28, 2022 at 02:09:15PM +0100, Tomas Härdin wrote:
> > > > > > sön 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 seen
> > > outside the context of FFmpeg. So from this biased single sample that 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=https://dweb.link ./ffplay ipfs://...
> works
> so such a fallback is all thats needed for it to just work
>

Yes, the beauty of gateways.

Are you suggesting that I update the patch to add this default?
I would prefer not to add that even though it would give a feeling of "just
works".
I'm mostly concerned about the bandwidth usage it could cause on that site.
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.

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 would
be allowed.


> thx
>
> [...]
> --
> Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
>
> If the United States is serious about tackling the national security
> threats
> related to an insecure 5G network, it needs to rethink the extent to which
> it
> values corporate profits and government espionage over security.-Bruce
> Schneier
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".
>