avoid recursive valid value check.

Author: 0robustus1

sdk/lib/opentelemetry/sdk/internal.rb

@@ -28,21 +28,29 @@ def valid_simple_value?(value)
         value.instance_of?(String) || boolean?(value) || numeric?(value) || value.nil?
       end
 
-      def valid_array_value?(value)
-        return false unless value.is_a?(Array)
-
-        value.all? { |v| valid_value?(v) }
-      end
-
-      def valid_mapping_value?(value)
-        return false unless value.is_a?(Hash)
-        return false unless value.keys.all? { |k| valid_key?(k) }
+      def valid_value?(value)
+        to_check = [value]
+        seen = Set.new
+        until to_check.empty?
+          current = to_check.pop
+          next if valid_simple_value?(current)
 
-        value.values.all? { |v| valid_value?(v) }
-      end
+          return false if seen.include?(current.object_id)
+          seen << current.object_id
 
-      def valid_value?(value)
-        valid_simple_value?(value) || valid_array_value?(value) || valid_mapping_value?(value)
+          case current
+          when Array
+            current.each { |v| to_check << v }
+          when Hash
+            current.each do |k, v|
+              return false unless valid_key?(k)
+              to_check << v
+            end
+          else
+            return false
+          end
+        end
+        true
       end
 
       def valid_attributes?(owner, kind, attrs)