firespring_dev_commands 2.1.32.pre.alpha.0 → 2.1.32.pre.alpha.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 288c13e632766bcc695d3acc7991a25fa566d943d1297be1af0744daad47f827
-  data.tar.gz: aec2fcbe07d4e10ef2de90a7937a7b6c48d741e321c7f4b89926a4799400a4a9
+  metadata.gz: 41b024ecb06f97d872f8a5b83ca9afb7af3fc92cae13e91674ff72812d920c68
+  data.tar.gz: fbbc7b27d086c5734a3041c6a43033e7b654e9486aac0280655fd70da25e75c8
 SHA512:
-  metadata.gz: 67b9ed3827875c7607de46ad05e75d6b3d83036c413bb7545caff7cccd48f2a5e5a2204ae368f019be08dd9a01bc0a1b6829b15a12ab8926887fcf61671d12e7
-  data.tar.gz: 0dbeb26c9a792c84b74ef074cf12498b478089ed3933781193f77f479140810b167ef6450cdc252073de0dfe9877204e6c092796b320fb0add7db3b5dad088d7
+  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,55 +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 DOMAIN='foo.com' to specify the hosted zone to activate." \
+                     "\n\toptionally specify HOSTED_ZONE_GROUP='arn:aws:logs:REGION:ACCOUNT_ID:' to specify the ARN of the target log group." \
                      "\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.0'.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.0
+  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