[FFmpeg-devel] [PATCH v3] avformat/http, tls: honor http_proxy command line variable for HTTPS
Marton Balint
cus at passwd.hu
Fri Mar 19 23:46:17 EET 2021
On Sun, 14 Mar 2021, Moritz Barsnick wrote:
> On Sun, Aug 23, 2020 at 22:50:18 +0300, Martin Storsjö wrote:
>> > On Fri, Aug 21, 2020 at 12:19:06 +0300, Martin Storsjö wrote:
>> > > LGTM, with one small nit:
>> > [...]
>> > > > {"listen", "Listen for incoming connections", offsetof(pstruct, options_field . listen), AV_OPT_TYPE_INT, { .i64 = 0 }, 0, 1, .flags = TLS_OPTFL }, \
>> > > > + {"http_proxy", "set proxy to tunnel through when using HTTPS", offsetof(pstruct, options_field . http_proxy), AV_OPT_TYPE_STRING, .flags = TLS_OPTFL }, \
>> > > > {"verifyhost", "Verify against a specific hostname", offsetof(pstruct, options_field . host), AV_OPT_TYPE_STRING, .flags = TLS_OPTFL }
>> > >
>> > > I'd remove the "when using HTTPS" bit here.
>> >
>> > Done. I also chose to make the capitalization consistent, and move the
>> > option down, as "verifyhost" is related to the certificate options (and
>> > should thus perhaps even be above "listen").
>>
>> Thanks, this version LGTM.
>
> Ping. Is this okay to apply? Could someone please?
Ok, will apply.
Thanks,
Marton
More information about the ffmpeg-devel
mailing list