-
Notifications
You must be signed in to change notification settings - Fork 417
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
feat(asm): cmdi patch refactor + rasp support #11870
Conversation
|
Datadog ReportBranch report: ✅ 0 Failed, 87 Passed, 1468 Skipped, 3m 56.06s Total duration (35m 33.79s time saved) |
BenchmarksBenchmark execution time: 2025-01-09 14:27:42 Comparing candidate commit 1e3c3bf in PR branch Found 0 performance improvements and 3 performance regressions! Performance is the same for 391 metrics, 2 unstable metrics. scenario:iast_aspects-ospathdirname_aspect
scenario:iast_aspects-ospathjoin_aspect
scenario:iast_aspects-ospathsplitext_aspect
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work!
subprocess
module andsystem.spawn*
functionssubprocess
module (withshell=True
)once merged this will also be tested with DataDog/system-tests#3791
APPSEC-56275
Checklist
Reviewer Checklist