data_shifter 0.3.2.1 → 0.3.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a2c27a241f01fd293278fdd45a9b8d5961d3a9ba304a95f657fd591ab5e2b4d6
-  data.tar.gz: 4db9925bd3e55bc8420bab92d9ed92a9e86e868ac0a1151f00b5245bace24319
+  metadata.gz: 8f9ab924aaeb0638f2061907484bc7ba4f5e795a1ea58e96f1ae8b0a7542b150
+  data.tar.gz: 24a81ebe9a5e1f0d27d4cec1494fd26c4c8d1209ddca2aba6cd606dec14e4dda
 SHA512:
-  metadata.gz: 151626dbec0bae6d652a71cfcf814ccd79ee442211275dcfa4e62a8d0de04838376a6c67385826f3c4320126ef28d2ac37889e61d59b12f2d29a794296cc921d
-  data.tar.gz: 2a696741fbfde60b730cc29470b4c47063813cd2dd3995726961c1405283f32e38203659625c56316d5dd5d06abe3bb251ad455a96d298ec1f38129e558bb4e0
+  metadata.gz: f5834ec7a2545de4342cc3f9205c8ce0f3a0b53fdfbebacd1452b7196c1b7cb606c4fa4d32e8e7d92532ff7e958f5537d2cba51e0176f9d760cdad9fa8b31498
+  data.tar.gz: cefd020aedd8a63f65f0bb507bcf8843a65fac12debfb78e47cd0e74b485e98746a8d3b805e0c2be6071254488308a1c10b47fb9e5280f13425d6eade6f4113d

data/CHANGELOG.md

@@ -4,6 +4,10 @@
 
 * N/A
 
+## [0.3.3]
+
+* [Bugfix] Dry-run net guard now allows loopback hosts (`127.0.0.1`, `::1`, `localhost`) by default. Previously, any tracing/metrics sidecar listening on loopback (Datadog agent on 8126, statsd on 8125, OTLP collectors, etc.) would trip `WebMock::NetConnectNotAllowedError` from a background tracer thread and abort the rake task — even when the shift itself made no HTTP calls. Opt out with `DataShifter.config.allow_loopback_requests = false` for strict net blocking.
+
 ## [0.3.2.1]
 
 * [Bugfix] Dry-run Sidekiq guard restores the previous `Sidekiq::Testing` mode (`fake!`, `inline!`, or `disable!`) instead of always calling `disable!`, which leaked global state and could leave later specs enqueueing work to real Redis.

data/README.md

@@ -111,7 +111,7 @@ In **dry run** mode, DataShifter automatically blocks or fakes these side effect
 
 | Service      | Behavior in dry run |
 |-------------|----------------------|
-| **HTTP**    | Blocked via WebMock (`disable_net_connect!`). Allow specific hosts with `allow_external_requests [...]` or `DataShifter.config.allow_external_requests`. |
+| **HTTP**    | Blocked via WebMock (`disable_net_connect!`). Loopback (`127.0.0.1`, `::1`, `localhost`) is allowed by default so tracing/metrics sidecars (Datadog agent, statsd, OTLP collector, etc.) keep working; opt out with `DataShifter.config.allow_loopback_requests = false`. Allow other hosts with `allow_external_requests [...]` or `DataShifter.config.allow_external_requests`. |
 | **ActionMailer** | `perform_deliveries = false` (restored after run). |
 | **ActiveJob**    | Queue adapter set to `:test` (restored after run). |
 | **Sidekiq**      | `Sidekiq::Testing.fake!` for the run; previous mode (`fake!`, `inline!`, or `disable!`) is restored after. Only applied if `Sidekiq::Testing` is already loaded. |
@@ -222,6 +222,11 @@ DataShifter.configure do |config|
   # Hosts allowed for HTTP during dry run only (no effect in commit mode)
   config.allow_external_requests = ["api.readonly.example.com"]
 
+  # Allow loopback HTTP (127.0.0.1, ::1, localhost) during dry runs (default: true).
+  # Needed so tracing/metrics sidecars (Datadog agent on 8126, statsd on 8125, OTLP
+  # collector, etc.) keep working in dry-run mode.
+  config.allow_loopback_requests = true
+
   # Suppress repeated log messages during a shift run (default: true)
   config.suppress_repeated_logs = true
 

data/lib/data_shifter/configuration.rb

@@ -16,6 +16,11 @@ module DataShifter
     # Has no effect in commit mode — HTTP is unrestricted when dry_run is false.
     attr_accessor :allow_external_requests
 
+    # Whether to allow loopback HTTP (127.0.0.1, ::1, localhost) during dry runs. Default: true.
+    # Loopback is rarely "external" and is needed for tracing/metrics sidecars (Datadog agent on
+    # 8126, statsd on 8125, OTLP collector, etc.). Set to false if you want strict net blocking.
+    attr_accessor :allow_loopback_requests
+
     # Whether to suppress repeated log messages during a shift run. Default: true.
     # Can be overridden per shift with `suppress_repeated_logs true/false`.
     attr_accessor :suppress_repeated_logs
@@ -33,6 +38,7 @@ module DataShifter
 
     def initialize
       @allow_external_requests = []
+      @allow_loopback_requests = true
       @suppress_repeated_logs = true
       @repeated_log_cap = 1000
       @progress_enabled = true

data/lib/data_shifter/internal/side_effect_guards.rb

@@ -14,6 +14,13 @@ module DataShifter
     # - ActionMailer / ActiveJob / Sidekiq: no extra loading; we only toggle existing config
     #   for the duration of the block and restore in ensure (Sidekiq restores prior fake/inline/disable).
     module SideEffectGuards
+      # Loopback hosts allowed by default during dry-run net blocking (unless opted out via
+      # DataShifter.config.allow_loopback_requests = false). Localhost is by definition local —
+      # the guard's purpose is to prevent accidental *external* state mutations, not to fight
+      # tracing/metrics sidecars (Datadog agent on 8126, statsd on 8125, OTLP collector, etc.)
+      # which commonly listen on these hosts.
+      DEFAULT_LOOPBACK_HOSTS = %w[127.0.0.1 ::1 localhost].freeze
+
       class << self
         # Applies side-effect guards, yields, then restores. Call only when running in dry run.
         def with_guards(shift_class:, &block)
@@ -62,7 +69,8 @@ module DataShifter
         def allowed_net_hosts(shift_class)
           per_shift = shift_class.respond_to?(:_allow_external_requests) ? shift_class._allow_external_requests : []
           global = DataShifter.config.allow_external_requests
-          Array(per_shift) + Array(global)
+          loopback = DataShifter.config.allow_loopback_requests ? DEFAULT_LOOPBACK_HOSTS : []
+          Array(per_shift) + Array(global) + loopback
         end
 
         def webmock_net_connect_error

data/lib/data_shifter/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module DataShifter
-  VERSION = "0.3.2.1"
+  VERSION = "0.3.3"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: data_shifter
 version: !ruby/object:Gem::Version
-  version: 0.3.2.1
+  version: 0.3.3
 platform: ruby
 authors:
 - Kali Donovan