From: Daniel Stenberg <daniel@haxx.se>
Date: Wed, 2 Aug 2023 16:03:59 +0000 (+0200)
Subject: url: change default value for CURLOPT_MAXREDIRS to 30
X-Git-Tag: curl-8_3_0~234
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=d135d040df5b276df5736688eba88d150b0d8a57;p=thirdparty%2Fcurl.git

url: change default value for CURLOPT_MAXREDIRS to 30

It was previously unlimited by default, but that's not a sensible
default. While changing this has a remote risk of breaking an existing
use case, I figure it is more likely to actually save users from loops.

Closes #11581
---

diff --git a/docs/libcurl/opts/CURLOPT_MAXREDIRS.3 b/docs/libcurl/opts/CURLOPT_MAXREDIRS.3
index 3aeb0746c8..bd1108931c 100644
--- a/docs/libcurl/opts/CURLOPT_MAXREDIRS.3
+++ b/docs/libcurl/opts/CURLOPT_MAXREDIRS.3
@@ -41,7 +41,7 @@ Setting the limit to 0 will make libcurl refuse any redirect.
 
 Set it to -1 for an infinite number of redirects.
 .SH DEFAULT
--1, unlimited
+30 (since 8.3.0), it was previously unlimited.
 .SH PROTOCOLS
 HTTP(S)
 .SH EXAMPLE
diff --git a/lib/url.c b/lib/url.c
index 8eecfc4214..1d6f2fad42 100644
--- a/lib/url.c
+++ b/lib/url.c
@@ -490,7 +490,7 @@ CURLcode Curl_init_userdefined(struct Curl_easy *data)
 
   set->filesize = -1;        /* we don't know the size */
   set->postfieldsize = -1;   /* unknown size */
-  set->maxredirs = -1;       /* allow any amount by default */
+  set->maxredirs = 30;       /* sensible default */
 
   set->method = HTTPREQ_GET; /* Default HTTP request */
 #ifndef CURL_DISABLE_RTSP