[vlc-devel] [PATCH] demux/adaptive: handle redirections in HTTPConnection

[Zhao Zhili]

On Fri, Jul 7, 2017 at 4:37 PM, Francois Cartegnie <fcvlcdev at free.fr> wrote:

> Le 07/07/2017 à 03:44, Zhao Zhili a écrit :
> >>
> >> @@ -95,8 +104,7 @@ namespace adaptive
> >>                  bool                chunked_eof;
> >>                  size_t              chunkLength;
> >>                  bool                queryOk;
> >> -                int                 retries;
> >> -                static const int    retryCount = 5;
> >> +                static const int    maxRedirectCount = 5;
> >>
>
> I haven't tested but I'm not sure it does not break 300 or ports redirects.
>
> And if you remove the retrycount, you break pipelining (connection
> terminated on server max requests).
>
>
Sorry I didn't find any use of  retries or retryCount.


>
> Francois
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20170707/5d994bd7/attachment.html>