<div dir="ltr"><div dir="ltr"><br></div><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, 21 Nov 2020 at 14:35, Andriy Gelman <<a href="mailto:andriy.gelman@gmail.com">andriy.gelman@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Sat, 21. Nov 20:16, Rémi Denis-Courmont wrote:<br>
> Le lauantaina 21. marraskuuta 2020, 19.59.12 EET Andriy Gelman a écrit :<br>
> > From: Andriy Gelman <<a href="mailto:andriy.gelman@gmail.com" target="_blank">andriy.gelman@gmail.com</a>><br>
> > <br>
> > The parameter rtsp-timeout is advertised to rtsp clients. Clients use<br>
> > this information to send a keep-alive message so that the connection is<br>
> > not closed.<br>
> > <br>
> > Currently, rtsp-timeout is set to 60 seconds but the underlying httpd<br>
> > timeout is hardcoded to 10 seconds in httpd_ClientInit().<br>
> <br>
> AFAIK, httpd code disables time-outs for RTSP connections (httpd.c:1575). <br>
> That's obviously wrong and it's even kinda vain, so it should be fixed.<br>
<br>
> <br>
> But to the point, there should not be a 10-second connection timeout, so I <br>
> don't really follow the rationale behind your patch. Furthermore, 10 seconds <br>
> seems way too few for the session timeout. The current default value of 60 <br>
> seconds seems much saner.<br>
</blockquote><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">No, in my testing now that path is not reached, so infinite timeout is not set.<br>
There is probably a bug somewhere in this part of the code.<br></blockquote><div><br></div><div>The commit that caused the regression is <br></div><div><span class="gmail-gI">a1116cbad0504c09b85e1db518e61fc41d4e672f</span></div><div> </div><div>--</div><div>Andriy<br></div><div><br></div></div></div>