Hi, On Wed, Aug 26, 2015 at 11:35 AM, Przemysław Sobala <przemyslaw.sob...@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. > > -- > 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 > _______________________________________________ > ffmpeg-devel mailing list > ffmpeg-devel@ffmpeg.org > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel Regards, Stephan _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
