[FFmpeg-devel] Connection timeout to HTTP resource

[Przemysław Sobala]

W dniu 26.08.2015 o 18:27, Stephan Holljes pisze:
> Hi,
>
> On Wed, Aug 26, 2015 at 11:35 AM, Przemysław Sobala
> <przemyslaw.sobala at grupawp.pl> wrote:
>> Hi
>> I wanted to configure tcp connection timeout while connecting to eg.
>> http://google.com:81.
>>
>> ffmpeg -i http://google.com:81?timeout=1000000
>> doesn't pass "timeout" param to tcp proto, as tcp proto doesn't support
>> URL query string
>
> TCP does support some parsing for URL query strings (see
> tcp.c:tcp_open around line 83 and following), but when used with HTTP
> the query is not parsed and simply sent with the request as it
> could/should contain information only relevant to the server handling
> the request.
>
>>
>> ffmpeg -timeout 1000000 -i http://google.com:81
>> doesn't pass "timeout" param to tcp proto, as tcp proto doestn't support
>> AVDictionary and url_open2
>
> AFAIK this should happen through AVOptions and should work. When I run
> the command with -loglevel trace the parameter gets set properly. I
> added logging for s->rw_timeout in tcp.c:tcp_open, and that value also
> gets set correctly.
>
>>
>> So I tried to correct the second case, and ended up with patch:
>>
>> diff --git a/libavformat/tcp.c b/libavformat/tcp.c
>> index bee349e..2f7726f 100644
>> --- a/libavformat/tcp.c
>> +++ b/libavformat/tcp.c
>> @@ -59,7 +59,8 @@ static const AVClass tcp_class = {
>>   };
>>
>>   /* return non zero if error */
>> -static int tcp_open(URLContext *h, const char *uri, int flags)
>> +static int tcp_open(URLContext *h, const char *uri, int flags,
>> +                     AVDictionary **options)
>>   {
>>       struct addrinfo hints = { 0 }, *ai, *cur_ai;
>>       int port, fd = -1;
>> @@ -244,7 +245,7 @@ static int tcp_get_file_handle(URLContext *h)
>>
>>   URLProtocol ff_tcp_protocol = {
>>       .name                = "tcp",
>> -    .url_open            = tcp_open,
>> +    .url_open2           = tcp_open,
>>       .url_accept          = tcp_accept,
>>       .url_read            = tcp_read,
>>       .url_write           = tcp_write,
>>
>> Now it works (ffmpeg -timeout 1000000 -i http://google.com:81). What do
>> you think about that?
>
> If that's the complete patch it doesn't really change anything since
> options isn't used anywhere in tcp_open() AFAICT.
> I also couldn't reproduce the behaviour you described. When running:
>
> ffmpeg -timeout 1000000 -i http://google.com:81
>
> with the latest git master the connection properly times out.
>
You're right. I must have missed something. So I got everything I need.
One last question: why FFmpeg reports
http://google.com:81: Network is unreachable
while tcp_open returns AVERROR(ETIMEDOUT) ?
--
Regards
Przemysław Sobala


Główne Spółki Grupy Wirtualna Polska:

Wirtualna Polska Holding Spółka Akcyjna z siedzibą w Warszawie, ul. Jutrzenki 137A, 02-231 Warszawa, wpisana do Krajowego Rejestru Sądowego - Rejestru Przedsiębiorców prowadzonego przez Sąd Rejonowy dla m.st. Warszawy w Warszawie pod nr KRS: 0000407130, kapitał zakładowy: 1 245 651,90 zł (w całości wpłacony), Numer Identyfikacji Podatkowej (NIP): 521-31-11-513

Grupa Wirtualna Polska Spółka z ograniczoną odpowiedzialnością z siedzibą w Warszawie, ul. Jutrzenki 137A, 02-231 Warszawa, wpisana do Krajowego Rejestru Sądowego - Rejestru Przedsiębiorców prowadzonego przez Sąd Rejonowy dla m.st. Warszawy w Warszawie pod nr KRS: 0000373814, kapitał zakładowy: 311.005.050,00 zł, Numer Identyfikacji Podatkowej (NIP): 527-26-45-593

WP Shopping Spółka z ograniczoną odpowiedzialnością z siedzibą w Gdańsku, ul. Romualda Traugutta 115 C, 80-226 Gdańsk, wpisana do Krajowego Rejestru Sądowego - Rejestru Przedsiębiorców prowadzonego przez Sąd Rejonowy Gdańsk - Północ w Gdańsku pod nr KRS: 0000546914, kapitał zakładowy: 170.000,00 złotych (w całości wpłacony), Numer Identyfikacji Podatkowej (NIP): 957-07-51-216