fog-google 1.24.1 → 1.26.0

data/lib/fog/{storage/google_xml → google/storage/storage_xml}/requests/get_object_acl.rb

@@ -1,6 +1,6 @@
 module Fog
-  module Storage
-    class GoogleXML
+  module Google
+    class StorageXML
       class Real
         # Get access control list for an Google Storage object
         #
@@ -38,7 +38,7 @@ module Fog
                   :host       => "#{bucket_name}.#{@host}",
                   :idempotent => true,
                   :method     => "GET",
-                  :parser     => Fog::Parsers::Storage::Google::AccessControlList.new,
+                  :parser     => Fog::Google::Parsers::Storage::AccessControlList.new,
                   :path       => Fog::Google.escape(object_name),
                   :query      => query)
         end

data/lib/fog/{storage/google_xml → google/storage/storage_xml}/requests/get_object_http_url.rb

@@ -1,6 +1,6 @@
 module Fog
-  module Storage
-    class GoogleXML
+  module Google
+    class StorageXML
       module GetObjectHttpUrl
         def get_object_http_url(bucket_name, object_name, expires, options = {})
           raise ArgumentError.new("bucket_name is required") unless bucket_name

data/lib/fog/{storage/google_xml → google/storage/storage_xml}/requests/get_object_https_url.rb

@@ -1,6 +1,6 @@
 module Fog
-  module Storage
-    class GoogleXML
+  module Google
+    class StorageXML
       module GetObjectHttpsUrl
         def get_object_https_url(bucket_name, object_name, expires, options = {})
           raise ArgumentError.new("bucket_name is required") unless bucket_name

data/lib/fog/{storage/google_xml → google/storage/storage_xml}/requests/get_object_url.rb

@@ -1,6 +1,6 @@
 module Fog
-  module Storage
-    class GoogleXML
+  module Google
+    class StorageXML
       class Real
         # Get an expiring object url from Google Storage
         #
@@ -15,14 +15,14 @@ module Fog
         #   * body<~String> - url for object
 
         def get_object_url(bucket_name, object_name, expires, options = {})
-          Fog::Logger.deprecation("Fog::Storage::Google => #get_object_url is deprecated, use #get_object_https_url instead[/] [light_black](#{caller(1..1).first})")
+          Fog::Logger.deprecation("Fog::Google::Storage => #get_object_url is deprecated, use #get_object_https_url instead[/] [light_black](#{caller(1..1).first})")
           get_object_https_url(bucket_name, object_name, expires, options)
         end
       end
 
       class Mock # :nodoc:all
         def get_object_url(bucket_name, object_name, expires, options = {})
-          Fog::Logger.deprecation("Fog::Storage::Google => #get_object_url is deprecated, use #get_object_https_url instead[/] [light_black](#{caller(1..1).first})")
+          Fog::Logger.deprecation("Fog::Google::Storage => #get_object_url is deprecated, use #get_object_https_url instead[/] [light_black](#{caller(1..1).first})")
           get_object_https_url(bucket_name, object_name, expires, options)
         end
       end

data/lib/fog/{storage/google_xml → google/storage/storage_xml}/requests/get_service.rb

@@ -1,6 +1,6 @@
 module Fog
-  module Storage
-    class GoogleXML
+  module Google
+    class StorageXML
       class Real
         # List information about Google Storage buckets for authorized user
         #
@@ -19,7 +19,7 @@ module Fog
                   :host     => @host,
                   :idempotent => true,
                   :method   => "GET",
-                  :parser   => Fog::Parsers::Storage::Google::GetService.new)
+                  :parser   => Fog::Google::Parsers::Storage::GetService.new)
         end
       end
 

data/lib/fog/{storage/google_xml → google/storage/storage_xml}/requests/head_object.rb

@@ -1,6 +1,6 @@
 module Fog
-  module Storage
-    class GoogleXML
+  module Google
+    class StorageXML
       class Real
         # Get headers for an object from Google Storage
         #

data/lib/fog/{storage/google_xml → google/storage/storage_xml}/requests/put_bucket.rb

@@ -1,6 +1,6 @@
 module Fog
-  module Storage
-    class GoogleXML
+  module Google
+    class StorageXML
       class Real
         # Create an Google Storage bucket
         #

data/lib/fog/{storage/google_xml → google/storage/storage_xml}/requests/put_bucket_acl.rb

@@ -1,6 +1,6 @@
 module Fog
-  module Storage
-    class GoogleXML
+  module Google
+    class StorageXML
       class Mock
         def put_bucket_acl(_bucket_name, _acl)
           # :no-coverage:

data/lib/fog/{storage/google_xml → google/storage/storage_xml}/requests/put_object.rb

@@ -1,6 +1,6 @@
 module Fog
-  module Storage
-    class GoogleXML
+  module Google
+    class StorageXML
       class Real
         # Create an object in an Google Storage bucket
         #

data/lib/fog/{storage/google_xml → google/storage/storage_xml}/requests/put_object_acl.rb

@@ -1,6 +1,6 @@
 module Fog
-  module Storage
-    class GoogleXML
+  module Google
+    class StorageXML
       class Real
         # TODO: move this methods to helper to use them with put_bucket_acl request
         def tag(name, value)

data/lib/fog/{storage/google_xml → google/storage/storage_xml}/requests/put_object_url.rb

@@ -1,6 +1,6 @@
 module Fog
-  module Storage
-    class GoogleXML
+  module Google
+    class StorageXML
       class Real
         # Get an expiring object url from Google Storage for putting an object
         #

data/lib/fog/{storage/google_xml → google/storage/storage_xml}/utils.rb

@@ -1,28 +1,30 @@
+# frozen_string_literal: true
+
 module Fog
-  module Storage
-    class GoogleXML
+  module Google
+    class StorageXML
       module Utils
         # https://cloud.google.com/storage/docs/access-control#predefined-acl
-        VALID_ACLS = [
-          "authenticated-read",
-          "bucket-owner-full-control",
-          "bucket-owner-read",
-          "private",
-          "project-private",
-          "public-read",
-          "public-read-write"
-        ].freeze
+        VALID_ACLS = %w(
+          authenticated-read
+          bucket-owner-full-control
+          bucket-owner-read
+          private
+          project-private
+          public-read
+          public-read-write
+        ).freeze
 
         def http_url(params, expires)
-          "http://" << host_path_query(params, expires)
+          "http://#{host_path_query(params, expires)}"
         end
 
         def https_url(params, expires)
-          "https://" << host_path_query(params, expires)
+          "https://#{host_path_query(params, expires)}"
         end
 
         def url(params, expires)
-          Fog::Logger.deprecation("Fog::Storage::Google => #url is deprecated, use #https_url instead [light_black](#{caller.first})[/]")
+          Fog::Logger.deprecation("Fog::Google::Storage => #url is deprecated, use #https_url instead [light_black](#{caller.first})[/]")
           https_url(params, expires)
         end
 
@@ -46,7 +48,7 @@ module Fog
 
         def request_params(params)
           subdomain = params[:host].split(".#{@host}").first
-          if @path_style || subdomain !~ /^(?!goog)(?:[a-z]|\d(?!\d{0,2}(?:\.\d{1,3}){3}$))(?:[a-z0-9]|\.(?![\.\-])|\-(?![\.])){1,61}[a-z0-9]$/
+          if @path_style || subdomain !~ /^(?!goog)(?:[a-z]|\d(?!\d{0,2}(?:\.\d{1,3}){3}$))(?:[a-z0-9]|\.(?![.-])|-(?!\.)){1,61}[a-z0-9]$/
             if subdomain =~ /_/
               # https://github.com/fog/fog/pull/1258#issuecomment-10248620.
               Fog::Logger.warning("fog: the specified google storage bucket name (#{subdomain}) is not DNS compliant (only characters a through z, digits 0 through 9, and the hyphen).")
@@ -61,12 +63,13 @@ module Fog
             end
 
             params[:host] = params[:host].split("#{subdomain}.")[-1]
-            params[:path] =
-              if params[:path]
+            if params[:path]
+              params[:path] =
                 "#{subdomain}/#{params[:path]}"
-              else
+            else
+              params[:path] =
                 subdomain.to_s
-              end
+            end
 
             subdomain = nil
           end

data/lib/fog/{storage/google_xml.rb → google/storage/storage_xml.rb}

@@ -1,20 +1,20 @@
 module Fog
-  module Storage
-    class GoogleXML < Fog::Service
-      autoload :Mock, File.expand_path("../google_xml/mock", __FILE__)
-      autoload :Real, File.expand_path("../google_xml/real", __FILE__)
-      autoload :Utils, File.expand_path("../google_xml/utils", __FILE__)
+  module Google
+    class StorageXML < Fog::Service
+      autoload :Mock, "fog/google/storage/storage_xml/mock"
+      autoload :Real, "fog/google/storage/storage_xml/real"
+      autoload :Utils, "fog/google/storage/storage_xml/utils"
 
       requires :google_storage_access_key_id, :google_storage_secret_access_key
       recognizes :host, :port, :scheme, :persistent, :path_style
 
-      model_path "fog/storage/google_xml/models"
+      model_path "fog/google/storage/storage_xml/models"
       collection :directories
       model :directory
       collection :files
       model :file
 
-      request_path "fog/storage/google_xml/requests"
+      request_path "fog/google/storage/storage_xml/requests"
       request :copy_object
       request :delete_bucket
       request :delete_object

data/lib/fog/{storage/google.rb → google/storage.rb}

@@ -1,18 +1,18 @@
 module Fog
-  module Storage
-    class Google < Fog::Service
+  module Google
+    class Storage < Fog::Service
       def self.new(options = {})
         begin
           fog_creds = Fog.credentials
-        rescue
+        rescue StandardError
           fog_creds = nil
         end
 
         if options.keys.include?(:google_storage_access_key_id) ||
            (!fog_creds.nil? && fog_creds.keys.include?(:google_storage_access_key_id))
-          Fog::Storage::GoogleXML.new(options)
+          Fog::Google::StorageXML.new(options)
         else
-          Fog::Storage::GoogleJSON.new(options)
+          Fog::Google::StorageJSON.new(options)
         end
       end
     end

data/lib/fog/google/version.rb

@@ -1,5 +1,5 @@
 module Fog
   module Google
-    VERSION = "1.24.1".freeze
+    VERSION = "1.26.0".freeze
   end
 end

data/lib/fog/google.rb

@@ -4,20 +4,17 @@ require "fog/xml"
 require "fog/google/version"
 
 module Fog
-  module Compute
-    autoload :Google, File.expand_path("../compute/google", __FILE__)
-  end
-
-  module DNS
-    autoload :Google, File.expand_path("../dns/google", __FILE__)
-  end
-
   module Google
+    autoload :Compute, File.expand_path("../google/compute", __FILE__)
+    autoload :DNS, File.expand_path("../google/dns", __FILE__)
     autoload :Mock, File.expand_path("../google/mock", __FILE__)
     autoload :Monitoring, File.expand_path("../google/monitoring", __FILE__)
     autoload :Pubsub, File.expand_path("../google/pubsub", __FILE__)
     autoload :Shared, File.expand_path("../google/shared", __FILE__)
     autoload :SQL, File.expand_path("../google/sql", __FILE__)
+    autoload :Storage, File.expand_path("../google/storage", __FILE__)
+    autoload :StorageJSON, 'fog/google/storage/storage_json'
+    autoload :StorageXML, 'fog/google/storage/storage_xml'
 
     extend Fog::Provider
 
@@ -37,17 +34,34 @@ module Fog
         "%" + Regexp.last_match(1).unpack("H2" * Regexp.last_match(1).bytesize).join("%").upcase
       end
     end
-  end
 
-  module Parsers
-    module Storage
-      autoload :Google, File.expand_path("../parsers/storage/google", __FILE__)
+    module Parsers
+      autoload :Storage, 'fog/google/parsers/storage'
     end
   end
+end
 
-  module Storage
-    autoload :Google, File.expand_path("../storage/google", __FILE__)
-    autoload :GoogleJSON, File.expand_path("../storage/google_json", __FILE__)
-    autoload :GoogleXML, File.expand_path("../storage/google_xml", __FILE__)
+# Add shims for backward compatibility
+# This allows old style references like Fog::Compute::Google to work
+# by redirecting them to the new namespace Fog::Google::Compute
+
+module Fog
+  # List of services from the original module
+  GOOGLE_SERVICES = %w[Compute DNS Monitoring Pubsub Storage SQL]
+
+  # Dynamically create shim modules for each service
+  GOOGLE_SERVICES.each do |service|
+    # Create the module namespace
+    const_set(service, Module.new) unless const_defined?(service)
+
+    # Get reference to the module
+    service_module = const_get(service)
+
+    # Define the Google submodule with the shim
+    service_module.const_set(:Google, Module.new)
+    service_module::Google.define_singleton_method(:new) do |*args|
+      warn "[DEPRECATION] `Fog::#{service}::Google.new` is deprecated. Please use `Fog::Google::#{service}.new` instead."
+      Fog::Google.const_get(service).new(*args)
+    end
   end
 end