Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: modules/trac/files/site.conf

Issue 5734493784637440: Issue 245 - Unify SSL configuration for all servers (Closed)
Patch Set: Created April 2, 2014, 2:05 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: modules/trac/files/site.conf
===================================================================
rename from modules/trac/templates/site.erb
rename to modules/trac/files/site.conf
--- a/modules/trac/templates/site.erb
+++ b/modules/trac/files/site.conf
@@ -1,45 +1,12 @@
-server {
- server_name <%= @domain %>;
-
- <% if is_default %>
- listen 80 default_server;
- listen [::]:80 default_server;
- <% else %>
- listen 80;
- listen [::]:80;
- <% end %>
-
- location / {
- rewrite (.*) https://$host$1 permanent;
- }
+location /
+{
Felix Dahlke 2014/04/09 18:13:25 We're not really consistent with opening braces in
+ fastcgi_pass unix:/tmp/trac-fastcgi.sock;
+ fastcgi_split_path_info ^()(.*)$;
+ include /etc/nginx/fastcgi_params;
+ fastcgi_param PATH_INFO $fastcgi_path_info;
}
-server {
- server_name <%= @domain %>;
-
- <% if is_default %>
- listen 443 ssl default_server;
- listen [::]:443 ssl default_server;
- <% else %>
- listen 443 ssl;
- listen [::]:443 ssl;
- <% end %>
-
- ssl_certificate adblockplus.org_sslcert.pem;
- ssl_certificate_key adblockplus.org_sslcert.key;
-
- add_header Strict-Transport-Security max-age=31536000;
-
- location /
- {
- fastcgi_pass unix:/tmp/trac-fastcgi.sock;
- fastcgi_split_path_info ^()(.*)$;
- include /etc/nginx/fastcgi_params;
- fastcgi_param PATH_INFO $fastcgi_path_info;
- }
-
- location /chrome/
- {
- alias /home/trac/htdocs/htdocs/;
- }
+location /chrome/
+{
+ alias /home/trac/htdocs/htdocs/;
}

Powered by Google App Engine
This is Rietveld