google-cloud-error_reporting 0.35.2 → 0.40.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1310d85b13af53a81f9169eb31f7d9b1629964aadac7d2cd1b0cf12c87160354
-  data.tar.gz: c90692577531e1d69b83d31d72929724233cee7e8f2fdb8761094cbbfbd75552
+  metadata.gz: 452ba5aa7f919e40361ff114937df9378a02d14a420edf3966b9b2c9d283a7c2
+  data.tar.gz: fca091beb1e8497cf3cdfdfda14bab9de60baf7df48ff9d6eb7b59fe6d521d40
 SHA512:
-  metadata.gz: b47ead95a38b2d5a16ba8755cbe3a8d5fba2550e24fefc83f99c4bf357eaedeffe21d10a45baa36764c2395ea7001496f3a42e129436207ae8e81ce3982e6f6d
-  data.tar.gz: 41ab14e7ef0be79a9dab3cbf4b4db462c4ebc3234a9c5c8b362741b93e342df75bea22ce4dbf090f822177cf9d2a0b8db02610e9817338a136fdde2b5530b4cb
+  metadata.gz: 64057bf9d228aa8632d6f69bf69293c5df32301a1b7efa959eb30f4aa6b7761fc5eeb3ae200141d3a48adea61b2a89282a47f2548b72d5303cb9b6cc8414edeb
+  data.tar.gz: 52a6dbc6f51c0f93ed6e1928b461ee8440cc39f0b26705b32304397399a467b063fe9fc05df9c32b1b6b0b66ea26ab957ea64ef8110732e7cb8a19369781a611

data/AUTHENTICATION.md

@@ -76,7 +76,8 @@ The environment variables that google-cloud-error_reporting checks for project I
 1. `ERROR_REPORTING_PROJECT`
 2. `GOOGLE_CLOUD_PROJECT`
 
-The environment variables that google-cloud-error_reporting checks for credentials are configured on {Google::Cloud::ErrorReporting::V1beta1::Credentials}:
+The environment variables that google-cloud-error_reporting checks for credentials
+are configured on {Google::Cloud::ErrorReporting::Credentials}:
 
 1. `ERROR_REPORTING_CREDENTIALS` - Path to JSON file, or JSON contents
 2. `ERROR_REPORTING_KEYFILE` - Path to JSON file, or JSON contents

data/CHANGELOG.md

@@ -1,5 +1,28 @@
 # Release History
 
+### 0.40.0 / 2020-07-21
+
+This is a major update that removes the "low-level" client interface code, and
+instead adds the new `google-cloud-error_reporting-v1beta1` gem as a dependency.
+This is a rewritten low-level client produced by a next-generation client code
+generator, with improved performance and stability.
+
+This change should have no effect on the high-level interface that most users
+will use. The one exception is that the (mostly undocumented) `client_config`
+argument, for adjusting low-level parameters such as RPC retry settings on
+client objects, has been removed. If you need to adjust these parameters, use
+the configuration interface in `google-cloud-error_reporting-v1beta1`.
+
+Substantial changes have been made in the low-level interfaces, however. If you
+are using the low-level classes under the `Google::Cloud::ErrorReporting::V1beta1`
+module, please review the docs for the new `google-cloud-error_reporting-v1beta1`
+gem. In particular:
+
+* Some classes have been renamed, notably the client class itself.
+* The client constructor takes a configuration block instead of configuration
+  keyword arguments.
+* All RPC method arguments are now keyword arguments.
+
 ### 0.35.2 / 2020-06-08
 
 #### Documentation

data/lib/google-cloud-error_reporting.rb

@@ -41,8 +41,6 @@ module Google
     #   * `https://www.googleapis.com/auth/cloud-platform`
     #
     # @param [Integer] timeout Default timeout to use in requests. Optional.
-    # @param [Hash] client_config A hash of values to override the default
-    #   behavior of the API client. Optional.
     #
     # @return [Google::Cloud::ErrorReporting::Project]
     #
@@ -59,11 +57,10 @@ module Google
     #                                             service_version: "v8"
     #   error_reporting.report error_event
     #
-    def error_reporting scope: nil, timeout: nil, client_config: nil
+    def error_reporting scope: nil, timeout: nil
       Google::Cloud.error_reporting @project, @keyfile,
                                     scope: scope,
-                                    timeout: (timeout || @timeout),
-                                    client_config: client_config
+                                    timeout: (timeout || @timeout)
     end
 
     ##
@@ -89,8 +86,6 @@ module Google
     #   * `https://www.googleapis.com/auth/cloud-platform`
     #
     # @param [Integer] timeout Default timeout to use in requests. Optional.
-    # @param [Hash] client_config A hash of values to override the default
-    #   behavior of the API client. Optional.
     #
     # @return [Google::Cloud::ErrorReporting::Project]
     #
@@ -107,13 +102,11 @@ module Google
     #                                             service_version: "v8"
     #   error_reporting.report error_event
     #
-    def self.error_reporting project_id = nil, credentials = nil, scope: nil,
-                             timeout: nil, client_config: nil
+    def self.error_reporting project_id = nil, credentials = nil, scope: nil, timeout: nil
       require "google/cloud/error_reporting"
       Google::Cloud::ErrorReporting.new project_id: project_id,
                                         credentials: credentials,
-                                        scope: scope, timeout: timeout,
-                                        client_config: client_config
+                                        scope: scope, timeout: timeout
     end
   end
 end
@@ -148,6 +141,7 @@ Google::Cloud.configure.add_config! :error_reporting do |config|
   default_version = Google::Cloud::Config.deferred do
     ENV["ERROR_REPORTING_VERSION"]
   end
+  default_scopes = ["https://www.googleapis.com/auth/cloud-platform"]
 
   config.add_field! :project_id, default_project, match: String, allow_nil: true
   config.add_alias! :project, :project_id
@@ -155,10 +149,9 @@ Google::Cloud.configure.add_config! :error_reporting do |config|
                     match: [String, Hash, Google::Auth::Credentials],
                     allow_nil: true
   config.add_alias! :keyfile, :credentials
-  config.add_field! :scope, nil, match: [String, Array]
+  config.add_field! :scope, default_scopes, match: [String, Array]
   config.add_field! :timeout, nil, match: Integer
-  config.add_field! :client_config, nil, match: Hash
-  config.add_field! :endpoint, nil, match: String
+  config.add_field! :endpoint, "clouderrorreporting.googleapis.com", match: String
   config.add_field! :service_name, default_service,
                     match: String, allow_nil: true
   config.add_field! :service_version, default_version,

data/lib/google/cloud/error_reporting.rb

@@ -62,8 +62,6 @@ module Google
       #   * `https://www.googleapis.com/auth/cloud-platform`
       #
       # @param [Integer] timeout Default timeout to use in requests. Optional.
-      # @param [Hash] client_config A hash of values to override the default
-      #   behavior of the API client. Optional.
       # @param [String] endpoint Override of the endpoint host name. Optional.
       #   If the param is nil, uses the default endpoint.
       # @param [String] project Alias for the `project_id` argument. Deprecated.
@@ -78,25 +76,25 @@ module Google
       #   error_reporting = Google::Cloud::ErrorReporting.new
       #   # ...
       #
-      def self.new project_id: nil, credentials: nil, scope: nil, timeout: nil,
-                   client_config: nil, endpoint: nil, project: nil, keyfile: nil
+      def self.new project_id: nil,
+                   credentials: nil,
+                   scope: nil,
+                   timeout: nil,
+                   endpoint: nil,
+                   project: nil,
+                   keyfile: nil
         project_id    ||= project
         project_id    ||= ErrorReporting::Project.default_project_id
         scope         ||= configure.scope
         timeout       ||= configure.timeout
-        client_config ||= configure.client_config
         endpoint      ||= configure.endpoint
         credentials   ||= (keyfile || default_credentials(scope: scope))
 
         credentials = resolve_credentials credentials, scope
         project_id = resolve_project_id project_id, credentials
 
-        ErrorReporting::Project.new(
-          ErrorReporting::Service.new(
-            project_id, credentials,
-            host: endpoint, timeout: timeout, client_config: client_config
-          )
-        )
+        service = ErrorReporting::Service.new project_id, credentials, host: endpoint, timeout: timeout
+        ErrorReporting::Project.new service
       end
 
       ##
@@ -118,8 +116,6 @@ module Google
       # * `scope` - (String, Array<String>) The OAuth 2.0 scopes controlling
       #   the set of resources and operations that the connection can access.
       # * `timeout` - (Integer) Default timeout to use in requests.
-      # * `client_config` - (Hash) A hash of values to override the default
-      #   behavior of the API client.
       # * `endpoint` - (String) Override of the endpoint host name, or `nil`
       #   to use the default endpoint.
       # * `service_name` - (String) Name for the application.
@@ -292,4 +288,11 @@ module Google
       private_class_method :default_credentials
     end
   end
+
+  # Aliases for compatibility with older spellings.
+  # @private
+  module Devtools
+    # @private
+    Clouderrorreporting = ::Google::Cloud::ErrorReporting unless const_defined? :Clouderrorreporting
+  end
 end

data/lib/google/cloud/error_reporting/credentials.rb

@@ -13,7 +13,7 @@
 # limitations under the License.
 
 
-require "google/cloud/error_reporting/v1beta1/credentials"
+require "google/cloud/error_reporting/v1beta1/report_errors_service/credentials"
 
 module Google
   module Cloud
@@ -37,7 +37,7 @@ module Google
       #
       #   error_reporting.project_id #=> "my-project"
       #
-      class Credentials < Google::Cloud::ErrorReporting::V1beta1::Credentials
+      class Credentials < Google::Cloud::ErrorReporting::V1beta1::ReportErrorsService::Credentials
       end
     end
   end

data/lib/google/cloud/error_reporting/error_event.rb

@@ -23,8 +23,7 @@ module Google
       # service.
       #
       # Google::Cloud::ErrorReporting::ErrorEvent is able to be transformed
-      # into
-      # {Google::Devtools::Clouderrorreporting::V1beta1::ReportedErrorEvent}
+      # into the `Google::Cloud::ErrorReporting::V1beta1::ReportedErrorEvent`
       # gRPC structure. Once an error event is reported, the GCP
       # Stackdriver ErrorReporting service is able to parse the message and
       # backtrace, then group the error events by content.
@@ -119,13 +118,10 @@ module Google
 
         ##
         # Build a new ErrorEvent from a
-        # Google::Devtools::Clouderrorreporting::V1beta1::ReportedErrorEvent
-        # object.
+        # `Google::Cloud::ErrorReporting::V1beta1::ReportedErrorEvent` object.
         #
-        # @param [
-        #   Google::Devtools::Clouderrorreporting::V1beta1::ReportedErrorEvent]
-        #   grpc A
-        #   Google::Devtools::Clouderrorreporting::V1beta1::ReportedErrorEvent
+        # @param [Google::Cloud::ErrorReporting::V1beta1::ReportedErrorEvent]
+        #   grpc A `Google::Cloud::ErrorReporting::V1beta1::ReportedErrorEvent`
         #   object
         #
         # @return [ErrorEvent] A new ErrorEvent instance derived from given grpc
@@ -233,10 +229,10 @@ module Google
         ##
         # Convert ErrorEvent object to gRPC struct.
         #
-        # @return [Devtools::Clouderrorreporting::V1beta1::ReportedErrorEvent]
+        # @return [Google::Cloud::ErrorReporting::V1beta1::ReportedErrorEvent]
         #   gRPC struct that represent an ErrorEvent.
         def to_grpc
-          Devtools::Clouderrorreporting::V1beta1::ReportedErrorEvent.new(
+          Google::Cloud::ErrorReporting::V1beta1::ReportedErrorEvent.new(
             event_time: event_time_grpc,
             message: message.to_s,
             service_context: service_context_grpc,
@@ -259,11 +255,11 @@ module Google
 
         ##
         # @private Formats the service_name and service_version as a
-        # Google::Devtools::Clouderrorreporting::V1beta1::ServiceContext.
+        # Google::Cloud::ErrorReporting::V1beta1::ServiceContext.
         #
         def service_context_grpc
           return nil if !service_name && !service_version
-          Devtools::Clouderrorreporting::V1beta1::ServiceContext.new(
+          Google::Cloud::ErrorReporting::V1beta1::ServiceContext.new(
             service: service_name.to_s,
             version: service_version.to_s
           )
@@ -271,12 +267,12 @@ module Google
 
         ##
         # @private Formats the http request context as a
-        # Google::Devtools::Clouderrorreporting::V1beta1::HttpRequestContext
+        # Google::Cloud::ErrorReporting::V1beta1::HttpRequestContext
         #
         def http_request_grpc
           return nil if !http_method && !http_url && !http_user_agent &&
                         !http_referrer && !http_status && !http_remote_ip
-          Devtools::Clouderrorreporting::V1beta1::HttpRequestContext.new(
+          Google::Cloud::ErrorReporting::V1beta1::HttpRequestContext.new(
             method: http_method.to_s,
             url: http_url.to_s,
             user_agent: http_user_agent.to_s,
@@ -288,11 +284,11 @@ module Google
 
         ##
         # @private Formats the source location as a
-        # Google::Devtools::Clouderrorreporting::V1beta1::SourceLocation
+        # Google::Cloud::ErrorReporting::V1beta1::SourceLocation
         #
         def source_location_grpc
           return nil if !file_path && !line_number && !function_name
-          Devtools::Clouderrorreporting::V1beta1::SourceLocation.new(
+          Google::Cloud::ErrorReporting::V1beta1::SourceLocation.new(
             file_path: file_path.to_s,
             line_number: line_number.to_i,
             function_name: function_name.to_s
@@ -301,13 +297,13 @@ module Google
 
         ##
         # @private Formats the error context info as a
-        # Google::Devtools::Clouderrorreporting::V1beta1::ErrorContext
+        # Google::Cloud::ErrorReporting::V1beta1::ErrorContext
         #
         def error_context_grpc
           http_request = http_request_grpc
           source_location = source_location_grpc
           return nil if !http_request && !source_location && !user
-          Devtools::Clouderrorreporting::V1beta1::ErrorContext.new(
+          Google::Cloud::ErrorReporting::V1beta1::ErrorContext.new(
             http_request: http_request,
             user: user.to_s,
             report_location: source_location

data/lib/google/cloud/error_reporting/service.rb

@@ -16,7 +16,6 @@
 require "google/cloud/errors"
 require "google/cloud/error_reporting/version"
 require "google/cloud/error_reporting/v1beta1"
-require "google/gax/errors"
 require "uri"
 
 module Google
@@ -26,31 +25,27 @@ module Google
       # @private Represents the gRPC Error Reporting service, including all the
       #   API methods.
       class Service
-        attr_accessor :project, :credentials, :timeout, :client_config, :host
+        attr_accessor :project, :credentials, :timeout, :host
 
         ##
         # Creates a new Service instance.
-        def initialize project, credentials, timeout: nil, client_config: nil,
-                       host: nil
+        def initialize project, credentials, timeout: nil, host: nil
           @project = project
           @credentials = credentials
           @timeout = timeout
-          @client_config = client_config || {}
-          @host = host || V1beta1::ReportErrorsServiceClient::SERVICE_ADDRESS
+          @host = host
         end
 
         def error_reporting
           return mocked_error_reporting if mocked_error_reporting
           @error_reporting ||= \
-            V1beta1::ReportErrorsServiceClient.new(
-              credentials: credentials,
-              timeout: timeout,
-              client_config: client_config,
-              service_address: service_address,
-              service_port: service_port,
-              lib_name: "gccl",
-              lib_version: Google::Cloud::ErrorReporting::VERSION
-            )
+            V1beta1::ReportErrorsService::Client.new do |config|
+              config.credentials = credentials if credentials
+              config.timeout = timeout if timeout
+              config.endpoint = host if host
+              config.lib_name = "gccl"
+              config.lib_version = Google::Cloud::ErrorReporting::VERSION
+            end
         end
         attr_accessor :mocked_error_reporting
 
@@ -81,32 +76,13 @@ module Google
 
           error_event_grpc = error_event.to_grpc
 
-          execute do
-            error_reporting.report_error_event project_path, error_event_grpc
-          end
+          error_reporting.report_error_event project_name: project_path, event: error_event_grpc
         end
 
         protected
 
-        def service_address
-          return nil if host.nil?
-          URI.parse("//#{host}").host
-        end
-
-        def service_port
-          return nil if host.nil?
-          URI.parse("//#{host}").port
-        end
-
         def project_path
-          V1beta1::ReportErrorsServiceClient.project_path project
-        end
-
-        def execute
-          yield
-        rescue Google::Gax::GaxError => e
-          # GaxError wraps BadStatus, but exposes it as #cause
-          raise Google::Cloud::Error.from_error(e.cause)
+          V1beta1::ReportErrorsService::Paths.project_path project: project
         end
       end
     end

data/lib/google/cloud/error_reporting/version.rb

@@ -16,7 +16,7 @@
 module Google
   module Cloud
     module ErrorReporting
-      VERSION = "0.35.2".freeze
+      VERSION = "0.40.0".freeze
     end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: google-cloud-error_reporting
 version: !ruby/object:Gem::Version
-  version: 0.35.2
+  version: 0.40.0
 platform: ruby
 authors:
 - Google Inc
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-06-08 00:00:00.000000000 Z
+date: 2020-07-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: google-cloud-core
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.2'
+        version: '1.5'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.2'
+        version: '1.5'
 - !ruby/object:Gem::Dependency
   name: stackdriver-core
   requirement: !ruby/object:Gem::Requirement
@@ -39,59 +39,19 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '1.3'
 - !ruby/object:Gem::Dependency
-  name: google-gax
+  name: google-cloud-error_reporting-v1beta1
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.8'
+        version: '0.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.8'
-- !ruby/object:Gem::Dependency
-  name: googleapis-common-protos
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 1.3.9
-    - - "<"
-      - !ruby/object:Gem::Version
-        version: '2.0'
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 1.3.9
-    - - "<"
-      - !ruby/object:Gem::Version
-        version: '2.0'
-- !ruby/object:Gem::Dependency
-  name: googleapis-common-protos-types
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 1.0.4
-    - - "<"
-      - !ruby/object:Gem::Version
-        version: '2.0'
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: 1.0.4
-    - - "<"
-      - !ruby/object:Gem::Version
-        version: '2.0'
+        version: '0.0'
 - !ruby/object:Gem::Dependency
   name: concurrent-ruby
   requirement: !ruby/object:Gem::Requirement
@@ -287,28 +247,7 @@ files:
 - lib/google/cloud/error_reporting/project.rb
 - lib/google/cloud/error_reporting/rails.rb
 - lib/google/cloud/error_reporting/service.rb
-- lib/google/cloud/error_reporting/v1beta1.rb
-- lib/google/cloud/error_reporting/v1beta1/credentials.rb
-- lib/google/cloud/error_reporting/v1beta1/doc/google/devtools/clouderrorreporting/v1beta1/common.rb
-- lib/google/cloud/error_reporting/v1beta1/doc/google/devtools/clouderrorreporting/v1beta1/error_group_service.rb
-- lib/google/cloud/error_reporting/v1beta1/doc/google/devtools/clouderrorreporting/v1beta1/error_stats_service.rb
-- lib/google/cloud/error_reporting/v1beta1/doc/google/devtools/clouderrorreporting/v1beta1/report_errors_service.rb
-- lib/google/cloud/error_reporting/v1beta1/doc/google/protobuf/duration.rb
-- lib/google/cloud/error_reporting/v1beta1/doc/google/protobuf/timestamp.rb
-- lib/google/cloud/error_reporting/v1beta1/error_group_service_client.rb
-- lib/google/cloud/error_reporting/v1beta1/error_group_service_client_config.json
-- lib/google/cloud/error_reporting/v1beta1/error_stats_service_client.rb
-- lib/google/cloud/error_reporting/v1beta1/error_stats_service_client_config.json
-- lib/google/cloud/error_reporting/v1beta1/report_errors_service_client.rb
-- lib/google/cloud/error_reporting/v1beta1/report_errors_service_client_config.json
 - lib/google/cloud/error_reporting/version.rb
-- lib/google/devtools/clouderrorreporting/v1beta1/common_pb.rb
-- lib/google/devtools/clouderrorreporting/v1beta1/error_group_service_pb.rb
-- lib/google/devtools/clouderrorreporting/v1beta1/error_group_service_services_pb.rb
-- lib/google/devtools/clouderrorreporting/v1beta1/error_stats_service_pb.rb
-- lib/google/devtools/clouderrorreporting/v1beta1/error_stats_service_services_pb.rb
-- lib/google/devtools/clouderrorreporting/v1beta1/report_errors_service_pb.rb
-- lib/google/devtools/clouderrorreporting/v1beta1/report_errors_service_services_pb.rb
 homepage: https://github.com/googleapis/google-cloud-ruby/tree/master/google-cloud-error_reporting
 licenses:
 - Apache-2.0