Switch to temporary redirect for URL rewrites

  * Allows us to revert to ASCII URLs if/when we migrate to a platform
    that supports them

Author: maquchizi

nginx.conf.sigil

@@ -34,8 +34,8 @@
 
 # Determine if the URL needs conversion by checking if it contains non-ASCII characters.
 map $uri $needs_conversion {
-    ~[^\x00-\x7F] "yes";
-    default        "no";
+  ~[^\x00-\x7F] "yes";
+  default        "no";
 }
 
 server {
@@ -98,8 +98,9 @@ server {
         -- First check for Medium ID and remove it before processing
         local has_medium_id = string.match(ngx.var.uri, "/(.*)-([%x][%x][%x][%x][%x][%x][%x][%x][%x][%x][%x]?[%x]?)$")
         if has_medium_id then
-            -- Remove Medium ID and redirect to the cleaned URL
+            -- Remove Medium ID
             local clean_path = string.gsub(ngx.var.uri, "/(.*)-([%x][%x][%x][%x][%x][%x][%x][%x][%x][%x][%x]?[%x]?)$", "/%1")
+            -- Permanent redirect since the Medium ID doesn't mean much on any other platform
             ngx.redirect(clean_path, 301)
             return
         end
@@ -150,7 +151,8 @@ server {
               local converted_path = convert_string(original_path)
 
               -- Redirect to the converted path
-              ngx.redirect("/" .. converted_path, 301)
+              -- Only do a temporary redirect allowing us to revert to ASCII URLs in future
+              ngx.redirect("/" .. converted_path, 307)
               return
         end
     }
@@ -177,7 +179,7 @@ server {
 }
 
 upstream {{ .APP }} {
-{{ range .DOKKU_APP_WEB_LISTENERS | split " " }}
-  server {{ . }};
-{{ end }}
+  {{ range .DOKKU_APP_WEB_LISTENERS | split " " }}
+    server {{ . }};
+  {{ end }}
 }