opendev/puppet-refstack
Code Issues Proposed changes

Merge "Fix scope of vhost template variables"

Browse Source
This commit is contained in:
Zuul 2018-06-12 22:33:33 +00:00 committed by Gerrit Code Review
commit 85c3a19c6a
2 changed files with 22 additions and 22 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
templates/refstack_http.vhost.erb
View File

@ -1,12 +1,12 @@
<VirtualHost <%= @hostname %>:80>
<% if !!@server_admin %>
ServerAdmin <%= @server_admin %>
<VirtualHost <%= scope.lookupvar('::refstack::apache::http::hostname') %>:80>
<% if !!scope.lookupvar('::refstack::apache::http::server_admin') %>
ServerAdmin <%= scope.lookupvar('::refstack::apache::http::server_admin') %>
<% end %>
ServerName <%= @hostname %>
ServerName <%= scope.lookupvar('::refstack::apache::http::hostname') %>
DocumentRoot <%= @install_www_root %>
DocumentRoot <%= scope.lookupvar('::refstack::apache::http::install_www_root') %>
WSGIDaemonProcess refstack user=<%= @user %> group=<%= @group %> threads=5
WSGIDaemonProcess refstack user=<%= scope.lookupvar('::refstack::apache::http::user') %> group=<%= scope.lookupvar('::refstack::apache::http::group') %> threads=5
WSGIScriptAlias /api /etc/refstack/app.wsgi
WSGIPassAuthorization On
LogLevel warn

32
templates/refstack_https.vhost.erb
View File

@ -1,12 +1,12 @@
<VirtualHost <%= @hostname %>:80>
<% if !!@server_admin %>
ServerAdmin <%= @server_admin %>
<VirtualHost <%= scope.lookupvar('::refstack::apache::https::hostname') %>:80>
<% if !!scope.lookupvar('::refstack::apache::https::server_admin') %>
ServerAdmin <%= scope.lookupvar('::refstack::apache::https::server_admin') %>
<% end %>
ServerName <%= @hostname %>
ServerName <%= scope.lookupvar('::refstack::apache::https::hostname') %>
DocumentRoot <%= @install_www_root %>
DocumentRoot <%= scope.lookupvar('::refstack::apache::https::install_www_root') %>
Redirect / https://<%= @hostname %>/
Redirect / https://<%= scope.lookupvar('::refstack::apache::https::hostname') %>/
LogLevel warn
ErrorLog ${APACHE_LOG_DIR}/refstack-error.log
@ -14,11 +14,11 @@
</VirtualHost>
<IfModule mod_ssl.c>
<VirtualHost <%= @hostname %>:443>
<% if !!@server_admin %>
ServerAdmin <%= @server_admin %>
<VirtualHost <%= scope.lookupvar('::refstack::apache::https::hostname') %>:443>
<% if !!scope.lookupvar('::refstack::apache::https::server_admin') %>
ServerAdmin <%= scope.lookupvar('::refstack::apache::https::server_admin') %>
<% end %>
ServerName <%= @hostname %>
ServerName <%= scope.lookupvar('::refstack::apache::https::hostname') %>
LogLevel warn
ErrorLog ${APACHE_LOG_DIR}/refstack-ssl-error.log
@ -27,10 +27,10 @@
SSLEngine on
SSLProtocol All -SSLv2 -SSLv3
SSLCertificateFile <%= @ssl_cert %>
SSLCertificateKeyFile <%= @ssl_key %>
<% if !!@resolved_ssl_ca %>
SSLCertificateChainFile <%= @resolved_ssl_ca %>
SSLCertificateFile <%= scope.lookupvar('::refstack::apache::https::ssl_cert') %>
SSLCertificateKeyFile <%= scope.lookupvar('::refstack::apache::https::ssl_key') %>
<% if !!scope.lookupvar('::refstack::apache::https::resolved_ssl_ca') %>
SSLCertificateChainFile <%= scope.lookupvar('::refstack::apache::https::resolved_ssl_ca') %>
<% end %>
<FilesMatch "\.(cgi|shtml|phtml|php)$">
@ -46,9 +46,9 @@
# MSIE 7 and newer should be able to use keepalive
BrowserMatch "MSIE [17-9]" ssl-unclean-shutdown
DocumentRoot <%= @install_www_root %>
DocumentRoot <%= scope.lookupvar('::refstack::apache::https::install_www_root') %>
WSGIDaemonProcess refstack user=<%= @user %> group=<%= @group %> threads=5
WSGIDaemonProcess refstack user=<%= scope.lookupvar('::refstack::apache::https::user') %> group=<%= scope.lookupvar('::refstack::apache::https::group') %> threads=5
WSGIScriptAlias /api /etc/refstack/app.wsgi
WSGIPassAuthorization On