firespring_dev_commands 2.1.32.pre.alpha.1 → 2.1.32.pre.alpha.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 69a50b275c7679e4e84cb4a89fc37b1ab5c99c33ba7c53bef46fa507804e9389
-  data.tar.gz: be434b186af195f448e2f4e4d6506151c489d223aaac71dead335a96f8debfe0
+  metadata.gz: 41b024ecb06f97d872f8a5b83ca9afb7af3fc92cae13e91674ff72812d920c68
+  data.tar.gz: fbbc7b27d086c5734a3041c6a43033e7b654e9486aac0280655fd70da25e75c8
 SHA512:
-  metadata.gz: ec623e8b07e6f6266c8c5d163d0a95070ad8acf1f3239d7f14ded6c4dba38d205aaf1266e1d3959fa8eb5dae054e904abfae3365a4aeda3d17ee6389d48da3c2
-  data.tar.gz: e8572a4144e5fdd0be44fbd01b914c6e78e1b9bd9b8d708dca73fd5bb479d8d068f785c2d6be6526181bd0635d608adf8217378e978854181b09b497081bb277
+  metadata.gz: 80e32906a5e5e5aa84fd2c985636327a7068fe7b75c8e94fbbe5a85bb78d0c0d239acef5f0dfa2b95299e201940898e2517b3337f966d059db15949eecaf61bc
+  data.tar.gz: 5284e5c970113eff3d78d8c86e3ae8adb6f0d54c5fe0d82a1fff6225f9cdd0a50093af41af315a1b4234d950f0ec47c1d1910562afbca7e5d790246c4e12105d

data/lib/firespring_dev_commands/aws/route53.rb

@@ -8,20 +8,23 @@ module Dev
 
       def initialize
         @client = ::Aws::Route53::Client.new
-        @zones = []
       end
 
-      def get_hosted_zones(domains)
+      def hosted_zones(domains)
+        @zones = []
         if domains.empty?
-          response = client.list_hosted_zones
-          response.hosted_zones.each do |hosted_zone|
-            @zones << hosted_zone.id
+          @zones = [].tap do |ary|
+            Dev::Aws.each_page(client, :list_hosted_zones, {max_items: 2}) do |response|
+              response.hosted_zones.each do |hosted_zone|
+                ary << hosted_zone.id unless hosted_zone.config.private_zone
+              end
+            end
           end
         else
           domains.each do |domain_name|
             zone = client.list_hosted_zones_by_name({dns_name: domain_name, max_items: 1})
-            target_name = zone.hosted_zones[0].name.chomp!('.') if zone.hosted_zones[0].name.end_with?('.')
-            @zones << zone.hosted_zones[0].id unless target_name != domain_name
+            target_name = zone.hosted_zones.first.name.chomp!('.')
+            @zones << zone.hosted_zones.first.id unless target_name != domain_name
           end
         end
         raise 'Hosted zone(s) not found.' if @zones.empty?
@@ -32,7 +35,7 @@ module Dev
           hosted_zone_id: zone_id,
           max_results: '1'
         )
-        config.query_logging_configs[0].id unless config.query_logging_configs.empty? || config.query_logging_configs[0].hosted_zone_id == zone_id
+        config.query_logging_configs.first.id unless config.query_logging_configs.empty? || config.query_logging_configs.first.hosted_zone_id == zone_id
       end
 
       def activate_query_logging(log_group)

data/lib/firespring_dev_commands/templates/aws.rb

@@ -121,56 +121,60 @@ module Dev
         end
       end
 
-      # rubocop:disable Metrics/MethodLength
       # Create the rake task for the hosted zone method
-      def create_hosted_zone_task!
+      def create_dns_logging_activate_task!
         # Have to set a local variable to be accessible inside of the instance_eval block
         exclude = @exclude
 
         DEV_COMMANDS_TOP_LEVEL.instance_eval do
           namespace :aws do
-            return if exclude.include?(:hosted_zone)
+            return if exclude.include?(:dns_logging)
 
             namespace :hosted_zone do
               namespace :dns_logging do
-                task :init do
-                  @route53 = Dev::Aws::Route53.new
-
-                  domains = ENV['DOMAINS'].to_s.strip.split(',')
-                  domain = ENV['DOMAIN'].to_s.strip
-                  domains << domain unless domain.empty?
-
-                  # Set global zone id array
-                  @route53.get_hosted_zones(domains)
-                end
-
                 desc 'Activates query logging for all hosted zones by default.' \
                      'This command should be run from the account the hosted zone(s) reside.' \
                      "\n\toptionally specify HOSTED_ZONE_GROUP='arn:aws:logs:REGION:ACCOUNT_ID:' to specify the ARN of the target log group." \
-                     "\n\toptionally specify DOMAIN='foo.com' to specify the hosted zone to activate." \
                      "\n\toptionally specify DOMAINS='foo.com,foobar.com' to specify the hosted zones to activate." \
                      "\n\t\tComma delimited list."
-                task activate: %w(init) do
+                task :activate do
+                  route53 = Dev::Aws::Route53.new
+                  route53.hosted_zones(ENV['DOMAINS'].to_s.strip.split(','))
                   # Use user defined log group. Otherwise, go get the default.
                   log_group = (ENV['HOSTED_ZONE_GROUP'] || Dev::Aws::Parameter.new.get_value('/Firespring/Internal/Route53/hosted-zone/log-group-arn'))
-
-                  @route53.activate_query_logging(log_group)
+                  route53.activate_query_logging(log_group)
                 end
+              end
+            end
+          end
+        end
+      end
 
+      # Create the rake task for the hosted zone method
+      def create_dns_logging_deactivate_task!
+        # Have to set a local variable to be accessible inside of the instance_eval block
+        exclude = @exclude
+
+        DEV_COMMANDS_TOP_LEVEL.instance_eval do
+          namespace :aws do
+            return if exclude.include?(:dns_logging_de)
+
+            namespace :hosted_zone do
+              namespace :dns_logging do
                 desc 'Deactivates query logging for all hosted zones by default. ' \
                      'This command should be run from the account the hosted zone(s) reside.' \
-                     "\n\toptionally specify DOMAIN='foo.com' to specify the hosted zone to activate." \
                      "\n\toptionally specify DOMAINS='foo.com,foobar.com' to specify the hosted zones to activate." \
                      "\n\t\tComma delimited list."
-                task deactivate: %w(init) do
-                  @route53.deactivate_query_logging
+                task :deactivate do
+                  route53 = Dev::Aws::Route53.new
+                  route53.hosted_zones(ENV['DOMAINS'].to_s.strip.split(','))
+                  route53.deactivate_query_logging
                 end
               end
             end
           end
         end
       end
-      # rubocop:enable Metrics/MethodLength
     end
   end
 end

data/lib/firespring_dev_commands/version.rb

@@ -6,6 +6,6 @@ module Dev
     # Use 'v.v.v.pre.alpha.v' for pre-release vesions
     # Use 'v.v.v.beta.v for beta versions
     # Use semantic versioning for any releases (https://semver.org/)
-    VERSION = '2.1.32.pre.alpha.1'.freeze
+    VERSION = '2.1.32.pre.alpha.2'.freeze
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: firespring_dev_commands
 version: !ruby/object:Gem::Version
-  version: 2.1.32.pre.alpha.1
+  version: 2.1.32.pre.alpha.2
 platform: ruby
 authors:
 - Firespring
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-04-17 00:00:00.000000000 Z
+date: 2024-04-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport