From 0c185c9e21f7f0bd9c9f7a1a1cda8f33e12c2a2a Mon Sep 17 00:00:00 2001 From: Sean Bright Date: Sat, 21 Nov 2020 12:51:48 -0500 Subject: [PATCH] res_http_media_cache.c: Set reasonable number of redirects By default libcurl does not follow redirects, so we explicitly enable it by setting CURLOPT_FOLLOWLOCATION. Once that is enabled, libcurl will follow up to CURLOPT_MAXREDIRS redirects, which by default is configured to be unlimited. This patch sets CURLOPT_MAXREDIRS to a more reasonable default (8). If we determine at some point that this needs to be increased on configurable it is a trivial change. ASTERISK-29173 #close Change-Id: I4925ebbcf0c7d728bb9252b3795b3479ae225b30 --- res/res_http_media_cache.c | 1 + 1 file changed, 1 insertion(+) diff --git a/res/res_http_media_cache.c b/res/res_http_media_cache.c index bca576372c..d761442b41 100644 --- a/res/res_http_media_cache.c +++ b/res/res_http_media_cache.c @@ -215,6 +215,7 @@ static CURL *get_curl_instance(struct curl_bucket_file_data *cb_data) curl_easy_setopt(curl, CURLOPT_HEADERFUNCTION, curl_header_callback); curl_easy_setopt(curl, CURLOPT_USERAGENT, GLOBAL_USERAGENT); curl_easy_setopt(curl, CURLOPT_FOLLOWLOCATION, 1); + curl_easy_setopt(curl, CURLOPT_MAXREDIRS, 8); curl_easy_setopt(curl, CURLOPT_URL, ast_sorcery_object_get_id(cb_data->bucket_file)); curl_easy_setopt(curl, CURLOPT_HEADERDATA, cb_data); -- 2.47.2