smart_proxy_container_gateway 3.4.0 → 3.4.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 470113167c3182629d62ee7babddf775b3e99e4c21ce0df5a9716894867dad86
-  data.tar.gz: 7adbe20e3c0dda9a9a3704501144c5f4134c7d9c266f36b19f24e83df09ad997
+  metadata.gz: 3d6e568aa93e31c6a60234698cbbc0a86fb61c71179e01692dd7f22193e3e096
+  data.tar.gz: 2085424bbca11bd822b190eadf4ccb14519397e2b8f1d8fff6182bacbd73c5ca
 SHA512:
-  metadata.gz: b017c8a19c575d9ddcae254500917089eef21739914b5dc73447ba39af9840679028a4401bf15fd1216ae7299b7da7f55c6200a87830d7553fd8b380bc6c821b
-  data.tar.gz: e37b37684d5eaace152ab018067f4fa9292a5f5663543e030ed07eba894e59d6486c6b43a0c465c11dd7de65af14599b2e54a22882ff541dd5cb45e1ae381776
+  metadata.gz: e5ebdf2d44f61b518be4115b73e5b6fae57364753c84cd61fc361e3275883b9c07f9c60e28b0e674f968dd14abacd35e1e62c1288d48c7f8dff0d60134fe6c24
+  data.tar.gz: f4a21694af41c33ba437e2c5e53e6bbed64a0e26d99a9c4ffd56c27b421d16ee8e28d261016d5a36b53d55e3405d376eabe5d1d58344590f7deccf3b041e717f

data/lib/smart_proxy_container_gateway/container_gateway_api.rb

@@ -262,7 +262,8 @@ module Proxy
         database.connection.transaction(isolation: :serializable, retry_on: [Sequel::SerializationFailure]) do
           hosts_table = database.connection[:hosts]
           hosts_table.delete
-          hosts_table.import(%i[uuid], hosts.map { |host| [host['uuid']] })
+          valid_hosts = hosts.filter_map { |host| [host['uuid']] if host['uuid'].present? }
+          hosts_table.import(%i[uuid], valid_hosts) unless valid_hosts.empty?
         end
         {}
       end
@@ -277,6 +278,8 @@ module Proxy
         do_authorize_any
         params['hosts'].flat_map do |host_map|
           host_map.filter_map do |host_uuid, repos|
+            next if host_uuid.nil? || host_uuid.to_s.strip.empty?
+
             if repos.nil? || repos.empty?
               repo_names = []
             else

data/lib/smart_proxy_container_gateway/container_gateway_main.rb

@@ -195,33 +195,50 @@ module Proxy
         # Insert all in a single transaction
         database.connection.transaction(isolation: :serializable, retry_on: [Sequel::SerializationFailure]) do
           hosts_repositories.delete
-          hosts_repositories.import(%i[repository_id host_id], entries)
+          hosts_repositories.import(%i[repository_id host_id], entries) unless entries.nil? || entries.empty?
         end
       end
 
       def build_host_repository_mapping(host_repo_maps)
+        return [] if host_repo_maps['hosts'].nil?
+
         hosts = database.connection[:hosts]
         repositories = database.connection[:repositories]
+
         entries = host_repo_maps['hosts'].flat_map do |host_map|
           host_map.filter_map do |host_uuid, repos|
-            host = hosts[{ uuid: host_uuid }]
-            next unless host
-
-            repo_names = repos
-                         .select { |repo| repo['auth_required'].to_s.downcase == "true" }
-                         .map { |repo| repo['repository'] }
-
-            repositories
-              .where(name: repo_names, auth_required: true)
-              .select(:id)
-              .map { |repo| [repo[:id], host[:id]] }
+            build_host_entries(hosts, repositories, host_uuid, repos)
           end
         end
-        entries.flatten!(1)
+        entries&.flatten(1)&.compact
+      end
+
+      def build_host_entries(hosts, repositories, host_uuid, repos)
+        return if host_uuid.nil? || host_uuid.to_s.strip.empty?
+
+        host = hosts[{ uuid: host_uuid }]
+        return unless host
+        return if repos.nil? || repos.empty?
+
+        repo_names = extract_auth_required_repo_names(repos)
+        repositories
+          .where(name: repo_names, auth_required: true)
+          .select(:id)
+          .map { |repo| [repo[:id], host[:id]] }
+      end
+
+      def extract_auth_required_repo_names(repos)
+        repos
+          .select { |repo| repo['auth_required'].to_s.downcase == "true" }
+          .map { |repo| repo['repository'] }
       end
 
       def update_host_repositories(uuid, repositories)
+        return if uuid.nil? || uuid.to_s.strip.empty?
+
         host = find_or_create_host(uuid)
+        return unless host
+
         hosts_repositories = database.connection[:hosts_repositories]
         database.connection.transaction(isolation: :serializable,
                                         retry_on: [Sequel::SerializationFailure],

data/lib/smart_proxy_container_gateway/version.rb

@@ -1,5 +1,5 @@
 module Proxy
   module ContainerGateway
-    VERSION = '3.4.0'.freeze
+    VERSION = '3.4.2'.freeze
   end
 end

metadata

@@ -1,13 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: smart_proxy_container_gateway
 version: !ruby/object:Gem::Version
-  version: 3.4.0
+  version: 3.4.2
 platform: ruby
 authors:
 - Ian Ballou
+autorequire:
 bindir: bin
 cert_chain: []
-date: 1980-01-02 00:00:00.000000000 Z
+date: 2025-12-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -76,8 +77,8 @@ email: ianballou67@gmail.com
 executables: []
 extensions: []
 extra_rdoc_files:
-- LICENSE
 - README.md
+- LICENSE
 files:
 - LICENSE
 - README.md
@@ -102,6 +103,7 @@ homepage: https://github.com/Katello/smart_proxy_container_gateway
 licenses:
 - GPL-3.0-only
 metadata: {}
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -109,14 +111,15 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '2.7'
+      version: '3.0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.6.7
+rubygems_version: 3.2.33
+signing_key:
 specification_version: 4
 summary: Pulp 3 container registry support for Foreman/Katello Smart-Proxy
 test_files: []