Merge branch 'main' into fix-streaming-error-handling

looker-open-source · Dec 18, 2024 · 212d190 · 212d190
2 parents e3658bb + 42c5cd3
commit 212d190
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/Gemfile.lock b/Gemfile.lock
@@ -40,7 +40,7 @@ GEM
       rack (>= 1.3)
     rake (13.2.1)
     redcarpet (3.6.0)
-    rexml (3.3.6)
+    rexml (3.3.9)
       strscan
     ruby2_keywords (0.0.5)
     sawyer (0.9.2)

diff --git a/lib/looker-sdk/error.rb b/lib/looker-sdk/error.rb
@@ -1,7 +1,7 @@
 ############################################################################################
 # The MIT License (MIT)
 #
-# Copyright (c) 2022 Looker Data Sciences, Inc.
+# Copyright (c) 2024 Google, LLC
 #
 # Permission is hereby granted, free of charge, to any person obtaining a copy
 # of this software and associated documentation files (the "Software"), to deal
@@ -76,6 +76,12 @@ def message
       response_message
     end
 
+    # Looker SDK error objects (e.g. LookerSDK::BadRequest) raise a
+    # WebMock::NetConnectNotAllowedError if they are marshal dumped.
+    def marshal_dump
+      raise TypeError.new("Refusing to marshal")
+    end
+
     # Error Doc URL
     #
     # @return [String]