Merge pull request 'reverse proxy: also add tag-based headers for redirections' (#17 ) from Armael/tricot:headers_for_redirects into main

Reviewed-on: Deuxfleurs/tricot#17
reverse proxy: also add tag-based headers for redirections
2024-12-01 15:31:13 +00:00 · 2024-12-01 15:40:25 +01:00
1 changed files with 2 additions and 2 deletions
--- a/src/https.rs
+++ b/src/https.rs
@ -371,8 +371,8 @@ async fn do_proxy(
 		reverse_proxy::call(remote_addr.ip(), &to_addr, req).await?
 	};

-	if response.status().is_success() {
-		// (TODO: maybe we want to add these headers even if it's not a success?)
+	if response.status().is_success() || response.status().is_redirection() {
+		// (TODO: maybe we want to add these headers even if it's not a success or redirection?)
 		for (header, value) in proxy_to.add_headers.iter() {
 			response.headers_mut().insert(
 				HeaderName::from_bytes(header.as_bytes())?,
Author	SHA1	Message	Date
Armael	2a679f97b6	Merge pull request 'reverse proxy: also add tag-based headers for redirections' (#17 ) from Armael/tricot:headers_for_redirects into main Reviewed-on: Deuxfleurs/tricot#17	2024-12-01 15:31:13 +00:00
Armaël Guéneau	e89f3b67bd	reverse proxy: also add tag-based headers for redirections	2024-12-01 15:40:25 +01:00