opendev/system-config
Code Issues Proposed changes

Update comment in logs.vhost

Browse Source 
Update the comment regarding htmlify to say what really happens.
Matching all .html.gz files isn't done since it messes with testr
results.

Also fix the line wrap to 80cols

Change-Id: I7aaa50610f1ca5418d3d4a3bbe21de58ad8036b4
This commit is contained in:
Joshua Hesketh 2014-10-21 21:12:16 +11:00
parent aace25f971
commit 21a11326f4
1 changed files with 8 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
modules/openstack_project/templates/logs.vhost.erb
View File

@ -58,11 +58,14 @@ NameVirtualHost <%= vhost_name %>:<%= port %>
</Directory>
RewriteEngine On
# rewrite all txt.gz & html.gz files to map to our internal htmlify wsgi app
# PT, Pass-through: to come back around and get picked up by the WSGIScriptAlias
# NS, No-subrequest: on coming back through, mod-autoindex may have added index.html
# which would match the !-f condition. We therefore ensure the
# rewrite doesn't trigger by disallowing subrequests.
# rewrite txt.gz & console.html[.gz] files to map to our internal htmlify
# wsgi app
# PT, Pass-through: to come back around and get picked up by the
# WSGIScriptAlias
# NS, No-subrequest: on coming back through, mod-autoindex may have added
# index.html which would match the !-f condition. We
# therefore ensure the rewrite doesn't trigger by
# disallowing subrequests.
RewriteRule ^/(.*\.txt\.gz)$ /htmlify/$1 [QSA,L,PT,NS]
RewriteRule ^/(.*console\.html(\.gz)?)$ /htmlify/$1 [QSA,L,PT,NS]