Refactoring de la gestion des options headers/datas de recuperer_url() pour...

Refactoring de la gestion des options headers/datas de recuperer_url() pour mieux gerer les sequences POST => 301 + Location => GET sur la nouvelle URL
voire POST => 307 sur url2 => POST sur url2 => 302 sur url3 => GET sur url3 le tout sans perdre les headers d'origine
en consequence on refabrique les donnes formattes a chaque redirection si besoin a partir des options headers+datas dont la derniere est eventuellement videe en cours de route

ecrire/inc/distant.php

@@ -453,30 +453,28 @@ function recuperer_url($url, $options = []) {
 
 
 	// Ajout des en-têtes spécifiques si besoin
-	$head_add = '';
+	$formatted_data = '';
 	if (!empty($options['headers'])) {
 		foreach ($options['headers'] as $champ => $valeur) {
-			$head_add .= $champ . ': ' . $valeur . "\r\n";
+			$formatted_data .= $champ . ': ' . $valeur . "\r\n";
 		}
-		// ne pas le repasser a recuperer_url si on follow un location, car ils seront dans datas
-		unset($options['headers']);
 	}
 
 	if (!empty($options['datas'])) {
 		[$head, $postdata] = prepare_donnees_post($options['datas'], $options['boundary']);
-		$head .= $head_add;
+		$head .= $formatted_data;
 		if (stripos($head, 'Content-Length:') === false) {
 			$head .= 'Content-Length: ' . strlen($postdata) . "\r\n";
 		}
-		$options['datas'] = $head . "\r\n" . $postdata;
+		$formatted_data = $head . "\r\n" . $postdata;
 		if (
 			strlen($postdata)
 			and !$methode_demandee
 		) {
 			$options['methode'] = 'POST';
 		}
-	} elseif ($head_add) {
-		$options['datas'] = $head_add . "\r\n";
+	} elseif ($formatted_data) {
+		$formatted_data .= "\r\n";
 	}
 
 	// Accepter les URLs au format feed:// ou qui ont oublie le http:// ou les urls relatives au protocole
@@ -508,7 +506,7 @@ function recuperer_url($url, $options = []) {
 		$url,
 		$refuser_gz,
 		$options['uri_referer'],
-		$options['datas'],
+		$formatted_data,
 		$options['version_http'],
 		$options['if_modified_since']
 	);
@@ -551,7 +549,15 @@ function recuperer_url($url, $options = []) {
 			fclose($handle);
 			include_spip('inc/filtres');
 			$url = suivre_lien($url, $res['location']);
-			spip_log("recuperer_url recommence sur $url", 'distant');
+
+			// une redirection doit se faire en GET, sauf status explicite 307 ou 308 qui indique de garder la meme methode
+			if ($options['methode'] !== 'GET') {
+				if (empty($res['status']) or !in_array($res['status'], [307, 308])) {
+					$options['methode'] = 'GET';
+					$options['datas'] = '';
+				}
+			}
+			spip_log("recuperer_url recommence " . $options['methode']. " sur $url", 'distant');
 
 			return recuperer_url($url, $options);
 		} elseif ($res['status'] !== 200) {