diff --git a/modules/openstack_project/templates/legacy.vhost.erb b/modules/openstack_project/templates/legacy.vhost.erb index 352a795501..579e935f83 100644 --- a/modules/openstack_project/templates/legacy.vhost.erb +++ b/modules/openstack_project/templates/legacy.vhost.erb @@ -7,7 +7,7 @@ NameVirtualHost <%= @vhost_name %>:<%= @port %> ServerName <%= @srvname %> <% if @serveraliases.is_a? Array -%> <% @serveraliases.each do |name| -%><%= " ServerAlias #{name}\n" %><% end -%> -<% elsif @serveraliases != '' -%> +<% elsif ! ['', nil].include?(@serveraliases) -%> <%= " ServerAlias #{@serveraliases}" %> <% end -%> diff --git a/modules/openstack_project/templates/static-governance.vhost.erb b/modules/openstack_project/templates/static-governance.vhost.erb index dd3637291b..3b6a57c6a2 100644 --- a/modules/openstack_project/templates/static-governance.vhost.erb +++ b/modules/openstack_project/templates/static-governance.vhost.erb @@ -6,7 +6,7 @@ ServerName <%= @vhost_name %> <% if @serveraliases.is_a? Array -%> <% @serveraliases.each do |name| -%><%= " ServerAlias #{name}\n" %><% end -%> -<% elsif @serveraliases != '' -%> +<% elsif ! ['', nil].include?(@serveraliases) -%> <%= " ServerAlias #{@serveraliases}" %> <% end -%> RewriteEngine On diff --git a/modules/openstack_project/templates/static-http-and-https.vhost.erb b/modules/openstack_project/templates/static-http-and-https.vhost.erb index a82bd24b9f..e29fbabe0c 100644 --- a/modules/openstack_project/templates/static-http-and-https.vhost.erb +++ b/modules/openstack_project/templates/static-http-and-https.vhost.erb @@ -7,7 +7,7 @@ <% if @serveraliases.is_a? Array -%> # Permanently redirect these ServerAlias entries to ServerName <% @serveraliases.each do |name| -%><%= " ServerAlias #{name}\n" %><% end -%> -<% elsif @serveraliases != '' -%> +<% elsif ! ['', nil].include?(@serveraliases) -%> <%= " ServerAlias #{@serveraliases}" %> RewriteEngine On RewriteCond %{HTTP_HOST} !<%= @vhost_name %>$ [NC] diff --git a/modules/openstack_project/templates/static-https-redirect.vhost.erb b/modules/openstack_project/templates/static-https-redirect.vhost.erb index f77b59464a..3ae6dd1147 100644 --- a/modules/openstack_project/templates/static-https-redirect.vhost.erb +++ b/modules/openstack_project/templates/static-https-redirect.vhost.erb @@ -6,7 +6,7 @@ ServerName <%= @vhost_name %> <% if @serveraliases.is_a? Array -%> <% @serveraliases.each do |name| -%><%= " ServerAlias #{name}\n" %><% end -%> -<% elsif @serveraliases != '' -%> +<% elsif ! ['', nil].include?(@serveraliases) -%> <%= " ServerAlias #{@serveraliases}" %> <% end -%> RewriteEngine On diff --git a/modules/openstack_project/templates/status.vhost.erb b/modules/openstack_project/templates/status.vhost.erb index e668353c43..481db90071 100644 --- a/modules/openstack_project/templates/status.vhost.erb +++ b/modules/openstack_project/templates/status.vhost.erb @@ -7,7 +7,7 @@ NameVirtualHost <%= @vhost_name %>:<%= @port %> ServerName <%= @srvname %> <% if @serveraliases.is_a? Array -%> <% @serveraliases.each do |name| -%><%= " ServerAlias #{name}\n" %><% end -%> -<% elsif @serveraliases != '' -%> +<% elsif ! ['', nil].include?(@serveraliases) -%> <%= " ServerAlias #{@serveraliases}" %> <% end -%> DocumentRoot <%= @docroot %> diff --git a/modules/openstack_project/templates/summit.vhost.erb b/modules/openstack_project/templates/summit.vhost.erb index d5b4f5f923..29963ac5e3 100644 --- a/modules/openstack_project/templates/summit.vhost.erb +++ b/modules/openstack_project/templates/summit.vhost.erb @@ -7,7 +7,7 @@ NameVirtualHost <%= @vhost_name %>:<%= @port %> ServerName <%= @srvname %> <% if @serveraliases.is_a? Array -%> <% @serveraliases.each do |name| -%><%= " ServerAlias #{name}\n" %><% end -%> -<% elsif @serveraliases != '' -%> +<% elsif ! ['', nil].include?(@serveraliases) -%> <%= " ServerAlias #{@serveraliases}" %> <% end -%>