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

- old
+ new

@@ -201,21 +201,41 @@
 
                 @backend_services_stub = ::Google::Cloud::Compute::V1::BackendServices::Rest::ServiceStub.new(
                   endpoint: @config.endpoint,
                   endpoint_template: DEFAULT_ENDPOINT_TEMPLATE,
                   universe_domain: @config.universe_domain,
-                  credentials: credentials
+                  credentials: credentials,
+                  logger: @config.logger
                 )
+
+                @backend_services_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
+                @backend_services_stub.logger
+              end
+
               # Service calls
 
               ##
               # Adds a key for validating requests with signed URLs for this backend service.
               #
@@ -301,11 +321,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
 
@@ -395,11 +415,11 @@
                                        retry_policy: @config.retry_policy
 
                 @backend_services_stub.aggregated_list request, options do |result, operation|
                   result = ::Gapic::Rest::PagedEnumerable.new @backend_services_stub, :aggregated_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
 
@@ -486,11 +506,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
 
@@ -579,11 +599,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
 
@@ -660,11 +680,10 @@
                                        metadata:     @config.metadata,
                                        retry_policy: @config.retry_policy
 
                 @backend_services_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
 
@@ -742,11 +761,10 @@
                                        metadata:     @config.metadata,
                                        retry_policy: @config.retry_policy
 
                 @backend_services_stub.get_health 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
 
@@ -825,11 +843,10 @@
                                        metadata:     @config.metadata,
                                        retry_policy: @config.retry_policy
 
                 @backend_services_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
 
@@ -916,11 +933,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
 
@@ -1006,11 +1023,11 @@
                                        retry_policy: @config.retry_policy
 
                 @backend_services_stub.list request, options do |result, operation|
                   result = ::Gapic::Rest::PagedEnumerable.new @backend_services_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
 
@@ -1096,11 +1113,11 @@
                                        retry_policy: @config.retry_policy
 
                 @backend_services_stub.list_usable request, options do |result, operation|
                   result = ::Gapic::Rest::PagedEnumerable.new @backend_services_stub, :list_usable, "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
 
@@ -1189,11 +1206,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
 
@@ -1282,11 +1299,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
 
@@ -1365,11 +1382,10 @@
                                        metadata:     @config.metadata,
                                        retry_policy: @config.retry_policy
 
                 @backend_services_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
 
@@ -1458,11 +1474,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
 
@@ -1541,11 +1557,10 @@
                                        metadata:     @config.metadata,
                                        retry_policy: @config.retry_policy
 
                 @backend_services_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
 
@@ -1634,11 +1649,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
 
@@ -1714,10 +1729,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
@@ -1735,9 +1755,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?
 