client.rb in google-cloud-compute-v1-2.15.0

- old
+ new

@@ -179,21 +179,41 @@
 
                 @snapshots_stub = ::Google::Cloud::Compute::V1::Snapshots::Rest::ServiceStub.new(
                   endpoint: @config.endpoint,
                   endpoint_template: DEFAULT_ENDPOINT_TEMPLATE,
                   universe_domain: @config.universe_domain,
-                  credentials: credentials
+                  credentials: credentials,
+                  logger: @config.logger
                 )
+
+                @snapshots_stub.logger(stub: true)&.info do |entry|
+                  entry.set_system_name
+                  entry.set_service
+                  entry.message = "Created client for #{entry.service}"
+                  entry.set_credentials_fields credentials
+                  entry.set "customEndpoint", @config.endpoint if @config.endpoint
+                  entry.set "defaultTimeout", @config.timeout if @config.timeout
+                  entry.set "quotaProject", @quota_project_id if @quota_project_id
+                end
               end
 
               ##
               # Get the associated client for long-running operations via GlobalOperations.
               #
               # @return [::Google::Cloud::Compute::V1::GlobalOperations::Rest::Client]
               #
               attr_reader :global_operations
 
+              ##
+              # The logger used for request/response debug logging.
+              #
+              # @return [Logger]
+              #
+              def logger
+                @snapshots_stub.logger
+              end
+
               # Service calls
 
               ##
               # Deletes the specified Snapshot resource. Keep in mind that deleting a single snapshot might not necessarily delete all the data on that snapshot. If any data on the snapshot that is marked for deletion is needed for subsequent snapshots, the data will be moved to the next corresponding snapshot. For more information, see Deleting snapshots.
               #
@@ -277,11 +297,11 @@
                       "project" => request.project
                     },
                     options: options
                   )
                   yield result, response if block_given?
-                  return result
+                  throw :response, result
                 end
               rescue ::Gapic::Rest::Error => e
                 raise ::Google::Cloud::Error.from_error(e)
               end
 
@@ -358,11 +378,10 @@
                                        metadata:     @config.metadata,
                                        retry_policy: @config.retry_policy
 
                 @snapshots_stub.get request, options do |result, operation|
                   yield result, operation if block_given?
-                  return result
                 end
               rescue ::Gapic::Rest::Error => e
                 raise ::Google::Cloud::Error.from_error(e)
               end
 
@@ -441,11 +460,10 @@
                                        metadata:     @config.metadata,
                                        retry_policy: @config.retry_policy
 
                 @snapshots_stub.get_iam_policy request, options do |result, operation|
                   yield result, operation if block_given?
-                  return result
                 end
               rescue ::Gapic::Rest::Error => e
                 raise ::Google::Cloud::Error.from_error(e)
               end
 
@@ -532,11 +550,11 @@
                       "project" => request.project
                     },
                     options: options
                   )
                   yield result, response if block_given?
-                  return result
+                  throw :response, result
                 end
               rescue ::Gapic::Rest::Error => e
                 raise ::Google::Cloud::Error.from_error(e)
               end
 
@@ -622,11 +640,11 @@
                                        retry_policy: @config.retry_policy
 
                 @snapshots_stub.list request, options do |result, operation|
                   result = ::Gapic::Rest::PagedEnumerable.new @snapshots_stub, :list, "items", request, result, options
                   yield result, operation if block_given?
-                  return result
+                  throw :response, result
                 end
               rescue ::Gapic::Rest::Error => e
                 raise ::Google::Cloud::Error.from_error(e)
               end
 
@@ -705,11 +723,10 @@
                                        metadata:     @config.metadata,
                                        retry_policy: @config.retry_policy
 
                 @snapshots_stub.set_iam_policy request, options do |result, operation|
                   yield result, operation if block_given?
-                  return result
                 end
               rescue ::Gapic::Rest::Error => e
                 raise ::Google::Cloud::Error.from_error(e)
               end
 
@@ -796,11 +813,11 @@
                       "project" => request.project
                     },
                     options: options
                   )
                   yield result, response if block_given?
-                  return result
+                  throw :response, result
                 end
               rescue ::Gapic::Rest::Error => e
                 raise ::Google::Cloud::Error.from_error(e)
               end
 
@@ -879,11 +896,10 @@
                                        metadata:     @config.metadata,
                                        retry_policy: @config.retry_policy
 
                 @snapshots_stub.test_iam_permissions request, options do |result, operation|
                   yield result, operation if block_given?
-                  return result
                 end
               rescue ::Gapic::Rest::Error => e
                 raise ::Google::Cloud::Error.from_error(e)
               end
 
@@ -959,10 +975,15 @@
               # @!attribute [rw] universe_domain
               #   The universe domain within which to make requests. This determines the
               #   default endpoint URL. The default value of nil uses the environment
               #   universe (usually the default "googleapis.com" universe).
               #   @return [::String,nil]
+              # @!attribute [rw] logger
+              #   A custom logger to use for request/response debug logging, or the value
+              #   `:default` (the default) to construct a default logger, or `nil` to
+              #   explicitly disable logging.
+              #   @return [::Logger,:default,nil]
               #
               class Configuration
                 extend ::Gapic::Config
 
                 # @private
@@ -980,9 +1001,10 @@
                 config_attr :timeout,       nil, ::Numeric, nil
                 config_attr :metadata,      nil, ::Hash, nil
                 config_attr :retry_policy,  nil, ::Hash, ::Proc, nil
                 config_attr :quota_project, nil, ::String, nil
                 config_attr :universe_domain, nil, ::String, nil
+                config_attr :logger, :default, ::Logger, nil, :default
 
                 # @private
                 def initialize parent_config = nil
                   @parent_config = parent_config unless parent_config.nil?
 