From 281acd62ab137d139912a7335b923282baf571eb Mon Sep 17 00:00:00 2001 From: Colleen Murphy Date: Fri, 27 Apr 2018 22:49:35 +0200 Subject: [PATCH] Fix scope of vhost template variables Dynamic scoping for variables in ERB templates was removed in puppet 4[1] which means that the variables defined in the manifest cannot be found when it is referenced in the httpd::vhost defined type and will be evaluated as nil when puppet runs. Use scope.lookupvar instead to be explicit about the variable's source. [1] https://puppet.com/docs/puppet/4.10/lang_updating_manifests.html#dynamic-scoping-in-erb Change-Id: I8d2b351537ad8fc0a06c58e1dd23d8423f38a328 --- templates/ethercalc.vhost.erb | 15 ++++++++------- 1 file changed, 8 insertions(+), 7 deletions(-) diff --git a/templates/ethercalc.vhost.erb b/templates/ethercalc.vhost.erb index 5c587fd..2b048b3 100644 --- a/templates/ethercalc.vhost.erb +++ b/templates/ethercalc.vhost.erb @@ -38,20 +38,21 @@ # MSIE 7 and newer should be able to use keepalive BrowserMatch "MSIE [17-9]" ssl-unclean-shutdown - <% if @auth_openid != nil %> + <% auth_openid = scope["ethercalc::apache::auth_openid"] %> + <% if ! [nil, :undef].include?(auth_openid) %> AuthType OpenID - AuthName "<%= @auth_openid['banner'] %>" + AuthName "<%= auth_openid['banner'] %>" AuthOpenIDSecureCookie On AuthOpenIDCookieLifespan 3600 AuthOpenIDTrustRoot https://<%= scope.lookupvar("ethercalc::apache::vhost_name") %> AuthOpenIDServerName https://<%= scope.lookupvar("ethercalc::apache::vhost_name") %> - AuthOpenIDSingleIdP <%= @auth_openid['singleIdp'] %> - AuthOpenIDTrusted <%= @auth_openid['trusted'] %> - <% if @auth_openid['any_valid_user'] %> + AuthOpenIDSingleIdP <%= auth_openid['singleIdp'] %> + AuthOpenIDTrusted <%= auth_openid['trusted'] %> + <% if auth_openid['any_valid_user'] %> Require valid-user - <% elsif !@auth_openid['users'].empty? %> - <% @auth_openid['users'].each do |user| -%> + <% elsif !auth_openid['users'].empty? %> + <% auth_openid['users'].each do |user| -%> Require user <%= user %> <% end -%> <% end %>