Merge pull request #11 from Guslington/custom-dns-to-loadbalancer

only supply the sslcertid mapping if using a listener with protocol https
theonestack · Oct 10, 2018 · 06c7916 · 06c7916
2 parents 6cc6807 + 258a87a
commit 06c7916
Showing 1 changed file with 9 additions and 4 deletions.
diff --git a/loadbalancer.cfhighlander.rb b/loadbalancer.cfhighlander.rb
@@ -9,9 +9,14 @@
       map 'AccountId'
       attribute 'DnsDomain'
     end
-    MappingParam('SslCertId') do
-      map 'AccountId'
-      attribute 'SslCertId'
+
+    if defined?(listeners)
+      unless listeners.select { |listener,properties| properties['protocol'] == 'https' }.empty?
+        MappingParam('SslCertId') do
+          map 'AccountId'
+          attribute 'SslCertId'
+        end
+      end
     end
 
     maximum_availability_zones.times do |x|
@@ -25,4 +30,4 @@
 
     ComponentParam 'VPCId', type: 'AWS::EC2::VPC::Id'
   end
-end
+end