Skip to content

Commit

Permalink
Disable crashtracking by default
Browse files Browse the repository at this point in the history
Crashtracking has been identified as causing issues with `ECHILD` and
`waitpid`, and possibly anything involving `SIGCHLD` with obscure
failure modes of hard to anticipate consequences.

Disable by default as a cautionary measure.

Mitigates #3954
  • Loading branch information
lloeki committed Oct 9, 2024
1 parent e5a774a commit c8a2165
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion lib/datadog/core/configuration/settings.rb
Original file line number Diff line number Diff line change
Expand Up @@ -886,7 +886,7 @@ def initialize(*_)
# Enables reporting of information when Ruby VM crashes.
option :enabled do |o|
o.type :bool
o.default true
o.default false
o.env 'DD_CRASHTRACKING_ENABLED'
end
end
Expand Down
6 changes: 3 additions & 3 deletions spec/datadog/core/configuration/settings_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -1967,7 +1967,7 @@
context 'is not defined' do
let(:environment) { nil }

it { is_expected.to be true }
it { is_expected.to be false }
end

[true, false].each do |value|
Expand All @@ -1982,9 +1982,9 @@

describe '#enabled=' do
it 'updates the #enabled setting' do
expect { settings.crashtracking.enabled = false }
expect { settings.crashtracking.enabled = true }
.to change { settings.crashtracking.enabled }
.from(true).to(false)
.from(false).to(true)
end
end
end
Expand Down

0 comments on commit c8a2165

Please sign in to comment.