google-cloud-error_reporting 0.31.1 → 0.31.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0f4cd7c625059f46ec0bcf6598e4dd6b9ac9714b23e266eb88d2efaa1ca7b02f
-  data.tar.gz: 53813d23825320611700f5e33f0aee568be3e1c38a5c18da38be43ba5f9bd322
+  metadata.gz: b4612ee47375f0a7cccc0a0a1e7a032855993f9111f8f98f835ef518d6f53390
+  data.tar.gz: d6baa8f717a81ab71e0b77940beb5873b748af7d920842c2e7645b0b195c8b98
 SHA512:
-  metadata.gz: 070f45b8c866a9077129f442beca0c0af363c3891c74274cccffa7ab96a47c06ec8868928143bed3fbe4708ac49a26dcbb1a4dbe8bfdb709dcc4f140504c2fc7
-  data.tar.gz: 6f2b345ec027cd1db0bedba29c3d45f99002bb7b8992a21f35373007e04cf047af67fcb811b80a2d48c8af3827b9ab8e1dc661134f7f7da67d15f38356773af5
+  metadata.gz: 92e08399c8762312dc2e3e7cc1aa1439c575c1ed3d7b97506b20914e6749fa10417a3dbf200960eea17618799c189370e16d5ac2a62e4fe89a04f8804aa3a45e
+  data.tar.gz: 2f7ccb7244bbd53245be67e5f9e40174ce40cce5da7f07b2efd7a32a10287bddd53e5303b4b536ea1a0fbafca183002a760584536fbf278d6d7580ad7fbe0dd1

data/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Release History
 
+### 0.31.2 / 2019-02-09
+
+* Fix conversion code for ErrorEvent and Debugee.
+  * Prepare for changes in JSON serialization coming in
+    google-protobuf 3.7.
+
 ### 0.31.1 / 2019-02-07
 
 * Update concurrent-ruby dependency
@@ -16,7 +22,7 @@
     This value was added in googleauth 0.7.0.
   * Loosen googleauth dependency
     Allow for new releases up to 0.10.
-    The googleauth devs have committed to maintanining the current API
+    The googleauth devs have committed to maintaining the current API
     and will not make backwards compatible changes before 0.10.
 
 ### 0.30.5 / 2018-09-20

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

@@ -238,74 +238,86 @@ module Google
         # @return [Devtools::Clouderrorreporting::V1beta1::ReportedErrorEvent]
         #   gRPC struct that represent an ErrorEvent.
         def to_grpc
-          error_event_hash = {
-            event_time: event_time_hash,
-            message: message,
-            service_context: service_context_hash,
-            context: error_context_hash
-          }.delete_if { |_, v| v.nil? }
-
-          grpc_class =
-            Devtools::Clouderrorreporting::V1beta1::ReportedErrorEvent
-          grpc_class.decode_json error_event_hash.to_json
+          Devtools::Clouderrorreporting::V1beta1::ReportedErrorEvent.new(
+            event_time: event_time_grpc,
+            message: message.to_s,
+            service_context: service_context_grpc,
+            context: error_context_grpc
+          )
         end
 
         private
 
         ##
-        # @private Formats the event_time as the hash representation of
-        # Google::Protobuf::Timestamp struct.
+        # @private Formats the event_time as a Google::Protobuf::Timestamp.
         #
-        def event_time_hash
+        def event_time_grpc
           return nil if event_time.nil?
-          {
+          Google::Protobuf::Timestamp.new(
             seconds: event_time.to_i,
             nanos: event_time.nsec
-          }
+          )
         end
 
         ##
-        # @private Formats the service_name and service_version as the hash
-        # representation of
-        # Google::Devtools::Clouderrorreporting::V1beta1::SourceContext struct.
+        # @private Formats the service_name and service_version as a
+        # Google::Devtools::Clouderrorreporting::V1beta1::ServiceContext.
         #
-        def service_context_hash
+        def service_context_grpc
           return nil if !service_name && !service_version
-          {
-            service: service_name,
-            version: service_version
-          }.delete_if { |_, v| v.nil? }
+          Devtools::Clouderrorreporting::V1beta1::ServiceContext.new(
+            service: service_name.to_s,
+            version: service_version.to_s
+          )
+        end
+
+        # rubocop:disable Metrics/AbcSize
+
+        ##
+        # @private Formats the http request context as a
+        # Google::Devtools::Clouderrorreporting::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(
+            method: http_method.to_s,
+            url: http_url.to_s,
+            user_agent: http_user_agent.to_s,
+            referrer: http_referrer.to_s,
+            response_status_code: http_status.to_i,
+            remote_ip: http_remote_ip.to_s
+          )
+        end
+
+        # rubocop:enable Metrics/AbcSize
+
+        ##
+        # @private Formats the source location as a
+        # Google::Devtools::Clouderrorreporting::V1beta1::SourceLocation
+        #
+        def source_location_grpc
+          return nil if !file_path && !line_number && !function_name
+          Devtools::Clouderrorreporting::V1beta1::SourceLocation.new(
+            file_path: file_path.to_s,
+            line_number: line_number.to_i,
+            function_name: function_name.to_s
+          )
         end
 
         ##
-        # @private Formats the error context info as the hash
-        # representation of
-        # Google::Devtools::Clouderrorreporting::V1beta1::ErrorContext struct.
+        # @private Formats the error context info as a
+        # Google::Devtools::Clouderrorreporting::V1beta1::ErrorContext
         #
-        def error_context_hash
-          http_request_hash = {
-            method: http_method,
-            url: http_url,
-            user_agent: http_user_agent,
-            referrer: http_referrer,
-            response_status_code: http_status,
-            remote_ip: http_remote_ip
-          }.delete_if { |_, v| v.nil? }
-
-          source_location_hash = {
-            file_path: file_path,
-            line_number: line_number,
-            function_name: function_name
-          }.delete_if { |_, v| v.nil? }
-
-          result = {
-            http_request: http_request_hash.empty? ? nil : http_request_hash,
-            user: user,
-            report_location:
-              source_location_hash.empty? ? nil : source_location_hash
-          }.delete_if { |_, v| v.nil? }
-
-          result.empty? ? nil : result
+        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(
+            http_request: http_request,
+            user: user.to_s,
+            report_location: source_location
+          )
         end
       end
     end

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

@@ -16,7 +16,7 @@
 module Google
   module Cloud
     module ErrorReporting
-      VERSION = "0.31.1".freeze
+      VERSION = "0.31.2".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.31.1
+  version: 0.31.2
 platform: ruby
 authors:
 - Google Inc
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-02-07 00:00:00.000000000 Z
+date: 2019-02-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: google-cloud-core