From 38e2d51aa09e34020c8d1c80b18808abe620df80 Mon Sep 17 00:00:00 2001
From: Luotio <juha@luotio.net>
Date: Wed, 1 Jan 2025 12:38:36 +0200
Subject: [PATCH 1/2] Update reverse-proxy.md

Fixed the IIS reverse proxy configuration, not considering special characters, and added HTTP_X_FORWARDED headers.

Signed-off-by: Luotio <juha@luotio.net>
---
 reverse-proxy.md | 12 ++++++++++--
 1 file changed, 10 insertions(+), 2 deletions(-)

diff --git a/reverse-proxy.md b/reverse-proxy.md
index 43cf0277..c4def69e 100644
--- a/reverse-proxy.md
+++ b/reverse-proxy.md
@@ -719,7 +719,8 @@ Add the following `web.config` file to the root of the site you created as the r
   </system.web>
   <system.webServer>
     <rewrite>
-      <rules>
+      <!-- useOriginalURLEncoding needs to be set to false, otherwise IIS will double encode urls causing all files with spaces or special characters to be inaccessible -->
+      <rules useOriginalURLEncoding="false">
         <!-- Force https -->
         <rule name="Https" stopProcessing="true">
           <match url="(.*)" />
@@ -734,7 +735,14 @@ Add the following `web.config` file to the root of the site you created as the r
           <conditions>
             <add input="{HTTPS}" pattern="^ON$" />
           </conditions>
-          <action type="Rewrite" url="http://nc-server-farm:11000/{UNENCODED_URL}" appendQueryString="false" />
+          <!-- Note that {UNENCODED_URL} already contains starting slash, so we must add it directly after the port number without additional slash -->
+          <action type="Rewrite" url="http://nc-server-farm:11000{UNENCODED_URL}" appendQueryString="false" />
+          <!-- Setting forwarded headers is not strictly needed but it is a good practice -->
+	  <serverVariables>
+            <set name="HTTP_X_FORWARDED_HOST" value="{HTTP_HOST}" />
+	    <set name="HTTP_X_FORWARDED_SCHEMA" value="https" />
+	    <set name="HTTP_X_FORWARDED_PROTO" value="https" />
+	  </serverVariables>
         </rule>
       </rules>
     </rewrite>

From dc761e259a3922eefd6e354103a871fbe8025295 Mon Sep 17 00:00:00 2001
From: Luotio <juha@luotio.net>
Date: Wed, 1 Jan 2025 12:45:50 +0200
Subject: [PATCH 2/2] Update reverse-proxy.md

Removed X_FORWARDED headers as they did not work as expected

Signed-off-by: Luotio <juha@luotio.net>
---
 reverse-proxy.md | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/reverse-proxy.md b/reverse-proxy.md
index c4def69e..173b812f 100644
--- a/reverse-proxy.md
+++ b/reverse-proxy.md
@@ -737,12 +737,6 @@ Add the following `web.config` file to the root of the site you created as the r
           </conditions>
           <!-- Note that {UNENCODED_URL} already contains starting slash, so we must add it directly after the port number without additional slash -->
           <action type="Rewrite" url="http://nc-server-farm:11000{UNENCODED_URL}" appendQueryString="false" />
-          <!-- Setting forwarded headers is not strictly needed but it is a good practice -->
-	  <serverVariables>
-            <set name="HTTP_X_FORWARDED_HOST" value="{HTTP_HOST}" />
-	    <set name="HTTP_X_FORWARDED_SCHEMA" value="https" />
-	    <set name="HTTP_X_FORWARDED_PROTO" value="https" />
-	  </serverVariables>
         </rule>
       </rules>
     </rewrite>