-
Notifications
You must be signed in to change notification settings - Fork 291
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
WIP: added module list to pass into IITM hook #4187
base: master
Are you sure you want to change the base?
Conversation
Overall package sizeSelf size: 6.27 MB Dependency sizes
🤖 This report was automatically generated by heaviest-objects-in-the-universe |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #4187 +/- ##
==========================================
+ Coverage 85.06% 85.10% +0.03%
==========================================
Files 247 245 -2
Lines 10953 10781 -172
Branches 33 33
==========================================
- Hits 9317 9175 -142
+ Misses 1636 1606 -30 ☔ View full report in Codecov by Sentry. |
BenchmarksBenchmark execution time: 2024-03-28 15:01:49 Comparing candidate commit 41d859c in PR branch Found 2 performance improvements and 0 performance regressions! Performance is the same for 259 metrics, 5 unstable metrics. scenario:plugin-graphql-with-depth-off-18
scenario:plugin-graphql-with-depth-on-max-18
|
What does this PR do?
We are modifying IITM to be able to take in a set of modules to patch.
DataDog/import-in-the-middle#65
This PR creates a module dependency list of files that need IITM patching, and pass this list into IITM
createHook
Motivation
Plugin Checklist
Additional Notes
Security
Datadog employees:
@DataDog/security-design-and-guidance
.Unsure? Have a question? Request a review!