ufo 6.1.0 → 6.1.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4394550f855c4e125e7bbcd6e62d4eee7fe5720c48037201cc66fcba19013d36
-  data.tar.gz: 1b3d1f7ef2b040bc3bbe2ec7815b7da0a5d5e366b8906bf8d997d57cd8193ad5
+  metadata.gz: 881d674019c976e4c8cae9030f69bd2125e59c7af970a1fa2005dcbfb77a4887
+  data.tar.gz: 5c3e8f73e200390396c99aa292f949f5c0a7b0e457bab47e2b88dbc583adc0fa
 SHA512:
-  metadata.gz: 5ca6d18a47212fbfd9beba2dcdc130d1341f5c3f50bd9a27396150828217ed61e44d885e9fd83904b5029822a89585e089288567fb931e1497765c9a74b443cc
-  data.tar.gz: 85afc2fb866da1a8de2aab7d10f942248706088adc10fda04cd23b537dc5d2fe21779ee11337b4f324cc4c57b41a8d6ce6996dc070d3a9508028f7525c4bf1e4
+  metadata.gz: 7eaca7eaa3999ce52dd4087dd2bd7862156f34c59443fcf06140f50356beb50237c4dea35d2dc68a1171576e17529c18144e5c97bc6aa4da4ea1edfa3b9411ef
+  data.tar.gz: 820645713455edf1d3e7fd242c68c21f32267f6eb031c67dd9236578d93c56e0a616750170f202ab27f32c86a0093a673fa5e7321c800b3d82121f38f156b29f

data/CHANGELOG.md

@@ -3,6 +3,21 @@
 All notable changes to this project will be documented in this file.
 This project *tries* to adhere to [Semantic Versioning](http://semver.org/), even before v1.0.
 
+## [6.1.3] - 2022-03-14
+- [#149](https://github.com/tongueroo/ufo/pull/149) stack_output helper improve message when stack not found
+
+## [6.1.2] - 2022-03-14
+- [#148](https://github.com/tongueroo/ufo/pull/148) Acm cert and user messaging improvements with config
+- acm_cert helper improvements: warn when not found and only create ssl listener if cert is found
+- improve squeezer: prevent infinite loop for edge case when data is [nil]
+- messaging improvements: provide exact line of context of the config with the issue
+- update dsl_evaluator dependency to at least 0.3.0
+
+## [6.1.1] - 2022-03-14
+- [#146](https://github.com/tongueroo/ufo/pull/146) dont set default vpc explicitly, instead use implicit setting
+- [#147](https://github.com/tongueroo/ufo/pull/147) explicitly set default vpc when not set cloudformation resources expect it
+- clean up debugging puts in cli help
+
 ## [6.1.0] - 2022-03-11
 - [#136](https://github.com/tongueroo/ufo/pull/136) ufo central: dont load config
 - [#137](https://github.com/tongueroo/ufo/pull/137) ufo new boot_hook generator

data/lib/ufo/cfn/stack/builder/resources/listener_ssl.rb

@@ -27,11 +27,11 @@ class Ufo::Cfn::Stack::Builder::Resources
     # nil on purpose
     def certificates
       ssl = Ufo.config.elb.ssl
-      normalize(ssl.certificates)
+      normalize(ssl.certificates) if ssl.certificates
     end
 
     def normalize(*certs)
-      certs = certs.flatten
+      certs = certs.flatten.compact
       certs.map do |cert|
         if cert.is_a?(String)
           {CertificateArn: cert}

data/lib/ufo/cfn/stack/vars.rb

@@ -59,7 +59,8 @@ class Ufo::Cfn::Stack
 
     # if the configuration is set to anything then enable it
     def create_listener_ssl?
-      Ufo.config.elb.ssl.enabled
+      elb = Ufo.config.elb
+      elb.ssl.enabled && elb.ssl.certificates
     end
 
     def create_elb?

data/lib/ufo/cli/help.rb

@@ -18,13 +18,10 @@ class Ufo::CLI
         list.sort! { |a, b| a[0] <=> b[0] }
         filter = Proc.new do |command, desc|
           command = command.sub(/ --.*/,'')
-          puts "command #{command.inspect}".color(:yellow)
           detected = main_commands.detect do |c|
             expr = "^ufo #{c}$"
-            puts "expr #{expr}"
             command =~ Regexp.new(expr)
           end
-          puts "detected #{detected.inspect}"
           detected
         end
         main = list.select(&filter)

data/lib/ufo/config.rb

@@ -187,14 +187,12 @@ module Ufo
       add_ext!(paths)
     end
 
-      def add_ext!(paths)
-      ext = "rb"
+    def add_ext!(paths)
       paths.map! do |path|
         path = path.sub(/\/$/,'') if path.ends_with?('/')
         "#{path}.rb"
       end
       paths
     end
-
-end
+  end
 end

data/lib/ufo/core.rb

@@ -16,7 +16,6 @@ module Ufo
       if @@config_loaded
         config.app
       else
-        call_line = caller.find {|l| l.include?('.ufo') }
         puts "ERROR: Using Ufo.app or :APP expansion very early in the UFO boot process".color(:red)
         puts <<~EOL.color(:red)
           The Ufo.app or :APP expansions are not yet available at this point
@@ -25,11 +24,9 @@ module Ufo
           1. Use the UFO_APP env var to set it, which allows it to be used.
           2. Hard code your actual app name.
 
-          Called from:
-
-              #{call_line}
-
         EOL
+        call_line = caller.find {|l| l.include?('.ufo') }
+        DslEvaluator.print_code(call_line)
         exit 1
       end
     end

data/lib/ufo/task_definition/helpers/acm.rb

@@ -1,12 +1,23 @@
 module Ufo::TaskDefinition::Helpers
   module Acm
+    include Ufo::Utils::CallLine
+    include Ufo::Utils::Pretty
+
     # returns cert arn
     def acm_cert(domain)
       certs = acm_certs
       cert = certs.find do |c|
         c.domain_name == domain
       end
-      cert.certificate_arn if cert
+      if cert
+        cert.certificate_arn
+      else
+        # Logger causes infinite loop when waf helper used in .ufo/
+        logger.warn "WARN: ACM cert not found: #{domain}".color(:yellow)
+        call_line = ufo_config_call_line
+        DslEvaluator.print_code(call_line)
+        nil
+      end
     end
 
     # TODO: handle when there are lots of certs by paging

data/lib/ufo/task_definition/helpers/ecr.rb

@@ -11,14 +11,9 @@ module Ufo::TaskDefinition::Helpers
       resp = ecr.describe_repositories(repository_names: [name])
       resp.repositories.first
     rescue Aws::ECR::Errors::RepositoryNotFoundException => e
-      call_line = ufo_config_call_line
       logger.warn "WARN: #{e.class} #{e.message}".color(:yellow)
-      logger.warn <<~EOL
-        Called from
-
-            #{call_line}
-
-      EOL
+      call_line = ufo_config_call_line
+      DslEvaluator.print_code(call_line)
       nil
     end
   end

data/lib/ufo/task_definition/helpers/stack_output.rb

@@ -6,16 +6,19 @@ module Ufo::TaskDefinition::Helpers
     def stack_output(name)
       stack_name, output_key = name.split(".")
       stack_name = names.expansion(stack_name)
-      resp = cloudformation.describe_stacks(stack_name: stack_name)
-      stack = resp.stacks.first
-      if stack
-        o = stack.outputs.detect { |h| h.output_key == output_key }
+      stack = find_stack(stack_name)
+      unless stack
+        logger.error "ERROR: Stack not found: #{stack_name}".color(:red)
+        call_line = ufo_config_call_line
+        DslEvaluator.print_code(call_line)
+        return
       end
 
+      o = stack.outputs.detect { |h| h.output_key == output_key }
       if o
         o.output_value
       else
-        logger.info "WARN: NOT FOUND: output #{key} for stack #{stack_name}"
+        logger.warn "WARN: NOT FOUND: output #{output_key} for stack #{stack_name}".color(:yellow)
         nil
       end
     end

data/lib/ufo/task_definition/helpers/waf.rb

@@ -19,15 +19,9 @@ module Ufo::TaskDefinition::Helpers
         web_acl.arn
       else
         # Logger causes infinite loop when waf helper used in .ufo/
-        call_line = ufo_config_call_line
         logger.warn "WARN: Web ACL not found: #{name}".color(:yellow)
-        logger.info <<~EOL
-          Called from:
-
-              #{call_line}
-
-          Are you sure it's a regional WAF ACL?
-        EOL
+        call_line = ufo_config_call_line
+        DslEvaluator.print_code(call_line)
       end
     end
   end

data/lib/ufo/utils/call_line.rb

@@ -3,8 +3,7 @@ module Ufo::Utils
     include Pretty
 
     def ufo_config_call_line
-      call_line = caller.find { |l| l.include?('.ufo/') }
-      pretty_path(call_line)
+      caller.find { |l| l.include?('.ufo/') }
     end
   end
 end

data/lib/ufo/utils/squeezer.rb

@@ -9,7 +9,9 @@ module Ufo::Utils
 
       case data
       when Array
-        data.map! { |v| squeeze(v) }
+        # .compact prevents infinite loop when data = [nil] on accident
+        # IE: config.elb.ssl.certificates = acm_cert("domain.not.found")
+        data.compact.map! { |v| squeeze(v) }
       when Hash
         data.each_with_object({}) do |(k,v), squeezed|
           # only remove nil and empty Array values within Hash structures

data/lib/ufo/version.rb

@@ -1,3 +1,3 @@
 module Ufo
-  VERSION = "6.1.0"
+  VERSION = "6.1.3"
 end

data/spec/ufo/utils/squeezer_spec.rb

@@ -0,0 +1,21 @@
+describe Ufo::Utils::Squeezer do
+  subject { Ufo::Utils::Squeezer.new(data) }
+
+  context("Array with nil") do
+    let(:data) { [nil] }
+    # Prevents infinite loop
+    it "remove nil" do
+      squeezed = subject.squeeze
+      expect(squeezed).to eq []
+    end
+  end
+
+  context("Hash with nil value") do
+    let(:data) { {a: 1, b: nil } }
+    # Prevents infinite loop
+    it "remove nil" do
+      squeezed = subject.squeeze
+      expect(squeezed).to eq(a: 1)
+    end
+  end
+end

data/ufo.gemspec

@@ -34,7 +34,7 @@ Gem::Specification.new do |spec|
   spec.add_dependency "cfn-status"
   spec.add_dependency "cli-format"
   spec.add_dependency "deep_merge"
-  spec.add_dependency "dsl_evaluator", ">= 0.2.5" # for DslEvaluator.print_code
+  spec.add_dependency "dsl_evaluator", ">= 0.3.0" # for DslEvaluator.print_code
   spec.add_dependency "memoist"
   spec.add_dependency "plissken"
   spec.add_dependency "rainbow"

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: ufo
 version: !ruby/object:Gem::Version
-  version: 6.1.0
+  version: 6.1.3
 platform: ruby
 authors:
 - Tung Nguyen
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2022-03-11 00:00:00.000000000 Z
+date: 2022-03-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: aws-logs
@@ -240,14 +240,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.2.5
+        version: 0.3.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.2.5
+        version: 0.3.0
 - !ruby/object:Gem::Dependency
   name: memoist
   requirement: !ruby/object:Gem::Requirement
@@ -653,6 +653,7 @@ files:
 - spec/ufo/iam_role/builder_spec.rb
 - spec/ufo/iam_role/dsl_spec.rb
 - spec/ufo/logs_spec.rb
+- spec/ufo/utils/squeezer_spec.rb
 - ufo.gemspec
 homepage: http://ufoships.com
 licenses:
@@ -699,3 +700,4 @@ test_files:
 - spec/ufo/iam_role/builder_spec.rb
 - spec/ufo/iam_role/dsl_spec.rb
 - spec/ufo/logs_spec.rb
+- spec/ufo/utils/squeezer_spec.rb