fix: use location instead of rewrite directive to avoid rewrite loop (#15)

Author: lemeurherve

docker/default.conf

@@ -4,14 +4,32 @@ server {
 
   root /htdocs;
 
+  # Serve index.html
+  location = / {
+    try_files /index.html =404;
+  }
+
   # Exact generated redirects
   include includes/jira-to-github-redirects.conf;
 
   # Generic fallback rewrites
-  rewrite ^/issue/([0-9]+)$ https://issues.jenkins.io/browse/JENKINS-$1 permanent;
-  rewrite ^/issue/([A-Z]+-[0-9]+)$ https://issues.jenkins.io/browse/$1 permanent;
-  rewrite ^/([A-Z]+-[0-9]+)$ https://issues.jenkins.io/browse/$1 permanent;
-  rewrite ^/browse/([A-Z]+-[0-9]+)$ https://issues.jenkins.io/browse/$1 permanent;
+  # /issue/12345
+  location ~ ^/issue/([0-9]+)$ {
+    return 301 https://issues.jenkins.io/browse/JENKINS-$1;
+  }
+
+  # /issue/JENKINS-12345 or /issue/INFRA-123
+  location ~ ^/issue/([A-Z]+-[0-9]+)$ {
+    return 301 https://issues.jenkins.io/browse/$1;
+  }
+
+  # /JENKINS-12345 or /INFRA-123
+  location ~ ^/([A-Z]+-[0-9]+)$ {
+    return 301 https://issues.jenkins.io/browse/$1;
+  }
 
-  return 404;
+  # /browse/JENKINS-12345 or /browse/INFRA-123
+  location ~ ^/browse/([A-Z]+-[0-9]+)$ {
+    return 301 https://issues.jenkins.io/browse/$1;
+  }
 }