-
Notifications
You must be signed in to change notification settings - Fork 399
Add a guard for missing AppSec.telemetry in Endpoint Collection
#4966
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Typing analysisIgnored filesThere are 539 ignored files in the Steepfile out of 836. Ignored filesNote: Ignored files are excluded from the next sections.
|
BenchmarksBenchmark execution time: 2025-10-10 09:33:30 Comparing candidate commit c6d96f7 in PR branch Found 1 performance improvements and 0 performance regressions! Performance is the same for 43 metrics, 2 unstable metrics. scenario:tracing - Propagation - Datadog
|
|
✅ Tests 🎉 All green!❄️ No new flaky tests detected 🎯 Code Coverage 🔗 Commit SHA: c6d96f7 | Docs | Was this helpful? Give us feedback! |
What does this PR do?
This PR fixes an exception that might happen when
AppSec.telemetryis set tonil, and we are trying to report Rails routes. As an example, this might happen when the default ruleset file is empty or broken.Motivation:
Failing system tests for
APPSEC_CORRUPTED_RULESandAPPSEC_MISSING_RULESscenarios:https://github.com/DataDog/system-tests-dashboard/actions/runs/18364392471/job/52314703288#step:46:86
Change log entry
None. This change is internal.
Additional Notes:
APPSEC-59579
How to test the change?
CI