[prev in list] [next in list] [prev in thread] [next in thread] 

List:       busybox
Subject:    Re: [PATCH] wget: increase redirections limit
From:       Denys Vlasenko <vda.linux () googlemail ! com>
Date:       2019-09-12 10:09:58
Message-ID: CAK1hOcPJV-gJepm_PzgrLQtbLOpTaU4p0ExGmxB0ezdHuN8NcQ () mail ! gmail ! com
[Download RAW message or body]

Applied, thanks

On Thu, Aug 29, 2019 at 2:04 PM David Demelier <markand@malikania.fr> wrote:
>
> Some hosting services like sourceforge perform a lot of relocations
> before actually serving the file. Example of current limitation:
>
>     busybox wget http://sourceforge.net/projects/fluxbox/files/fluxbox/1.3.7/fluxbox-1.3.7.tar.xz
>     Connecting to sourceforge.net (216.105.38.13:80)
>     Connecting to sourceforge.net (216.105.38.13:443)
>     Connecting to sourceforge.net (216.105.38.13:443)
>     Connecting to sourceforge.net (216.105.38.13:443)
>     Connecting to downloads.sourceforge.net (216.105.38.13:443)
>     wget: too many redirections
> ---
>  networking/wget.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/networking/wget.c b/networking/wget.c
> index 9e5a40b6f..f2fc9e215 100644
> --- a/networking/wget.c
> +++ b/networking/wget.c
> @@ -1109,7 +1109,7 @@ static void download_one_url(const char *url)
>                  * We are not sure it exists on remote side */
>         }
>
> -       redir_limit = 5;
> +       redir_limit = 16;
>   resolve_lsa:
>         lsa = xhost2sockaddr(server.host, server.port);
>         if (!(option_mask32 & WGET_OPT_QUIET)) {
> --
> 2.14.5
>
> _______________________________________________
> busybox mailing list
> busybox@busybox.net
> http://lists.busybox.net/mailman/listinfo/busybox
_______________________________________________
busybox mailing list
busybox@busybox.net
http://lists.busybox.net/mailman/listinfo/busybox
[prev in list] [next in list] [prev in thread] [next in thread]