Make sure to dup the default value when setting it

lib/datadog/core/configuration/option.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require_relative '../utils/safe_dup'
+
 module Datadog
   module Core
     module Configuration
@@ -137,7 +139,7 @@ def default_value
           if definition.default.instance_of?(Proc)
             context_eval(&definition.default)
           else
-            definition.experimental_default_proc || definition.default
+            definition.experimental_default_proc || Core::Utils::SafeDup.frozen_or_dup(definition.default)
           end
         end
 

spec/datadog/appsec/configuration/settings_spec.rb

@@ -387,7 +387,7 @@ def patcher
         context 'is not defined' do
           let(:appsec_obfuscator_key_regex) { nil }
 
-          it { is_expected.to be described_class::DEFAULT_OBFUSCATOR_KEY_REGEX }
+          it { is_expected.to eq described_class::DEFAULT_OBFUSCATOR_KEY_REGEX }
         end
 
         context 'is defined' do
@@ -423,7 +423,7 @@ def patcher
         context 'is not defined' do
           let(:appsec_obfuscator_value_regex) { nil }
 
-          it { is_expected.to be described_class::DEFAULT_OBFUSCATOR_VALUE_REGEX }
+          it { is_expected.to eq described_class::DEFAULT_OBFUSCATOR_VALUE_REGEX }
         end
 
         context 'is defined' do

spec/datadog/core/configuration/option_spec.rb

@@ -34,6 +34,15 @@
   let(:setter_value) { double('setter_value') }
   let(:context) { double('configuration object') }
 
+  # When setting the setting value, we make sure to duplicate it to avoid unwanted modifications
+  # to make sure specs pass when comparing result ex. expect(result).to be value
+  # we ensure that frozen_or_dup returns the same instance
+  before do
+    allow(Datadog::Core::Utils::SafeDup).to receive(:frozen_or_dup) do |args, _block|
+      args
+    end
+  end
+
   describe '#initialize' do
     it { expect(option.definition).to be(definition) }
   end

spec/datadog/core/configuration/options_spec.rb

@@ -10,6 +10,15 @@
       end
     end
 
+    # When setting the setting value, we make sure to duplicate it to avoid unwanted modifications
+    # to make sure specs pass when comparing result ex. expect(result).to be value
+    # we ensure that frozen_or_dup returns the same instance
+    before do
+      allow(Datadog::Core::Utils::SafeDup).to receive(:frozen_or_dup) do |args, _block|
+        args
+      end
+    end
+
     describe 'class behavior' do
       describe '#options' do
         subject(:options) { options_class.options }