NH-66480: do not raise in instrumentation

solarwinds · Jan 18, 2024 · 678c795 · 678c795
1 parent 9315483
commit 678c795
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/lib/solarwinds_apm/inst/graphql.rb b/lib/solarwinds_apm/inst/graphql.rb
@@ -151,8 +151,10 @@ def graphql_multiplex(data)
 
         def sanitize(query)
           return unless query
-          raise ArgumentError, "Query is too long for sanitize." if query.length > MAX_QUERY_LENGTH
-
+          if query.length > MAX_QUERY_LENGTH
+            SolarWindsAPM.logger.warn '[solarwinds_apm/graphql] query is too long for sanitizing. Return empty query string.'
+            return ''
+          end
           # remove arguments
           query.gsub(/"[^"]*"/, '"?"')                 # strings
                .gsub(/-?[0-9]*\.?[0-9]+e?[0-9]*/, '?') # ints + floats