Skip to content
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

guanceV1120 #36

Merged
merged 385 commits into from
Apr 19, 2023
Merged

guanceV1120 #36

merged 385 commits into from
Apr 19, 2023

Conversation

songlonqi-java
Copy link
Collaborator

What Does This Do

Motivation

Additional Notes

merge datadog 最新的tag 1.12.0

richardstartin and others added 30 commits March 15, 2023 14:25
…tegration

add service name and resource name to profiling context
reduce the number of noisy log statements
Main goal of this refactoring is to support duplicated probe as first
class citizen instead of having it on the side lines.
We shift the way snapshots are constructed to focus on CapturedContext
as main data collection object and evaluation of log template and
condition. The commit phase is considering the captured contexts and
decide if we need to send a snapshot and build it by probe with all
information required (entry/exit/line context, duration, evaluation
errors).
Each context has a status compiling information about the evaluation
results and help make the decision at commit time. Those statuses are
per probe.
add _dd.trace prefix to built in tracing based profiling attributes
Fix merge of rule data provided in the config file
remove gRPC queueing time instrumentation
…me time (start JMX at same time to exercise runtime metrics) (DataDog#4924)
… java.net.http.HttpRequest are visible when checking against Java11 (DataDog#4920)
…r.error for Jetty.

Add Spring MVC Jetty smoke test
Instead of adding `$$_WeldSubclass`, ignore all classes containing `$$_Weld`.
jpbempel and others added 26 commits April 12, 2023 17:00
…tails

Rename ProbeDetails and move out Snapshot
…-evaluateat

move resolveEvaluateAt method to CapturedContext
…nsformations

Remove parametric testTransformations
We still have deprecations in the configuration stage, so the project is
not ready for this mode.
Remove abandoned test sets plugin for gradle 8
This state enables IAST instrumentation, but does not enable the IAST
subsystem itself. It is meant for instrumentation validation at the
moment.
In TaintedMap, insert at the end of the chain, instead of at the head.
This allows us to test for duplicates. This prevents the pathological
case where the same object is tainted many times, making the linked list
of a bucket have an unbound length and distorting the size estimate.

This should be, in theory, slower. But microbenchmarks do not show any
noticeable difference. Perhaps because given our parameters, less than
0.2% of the chains are expect to be longer than 2 elements.
Upgrade to libddwaf 1.9.0 (libsqreen 6.2.0)
remove hand-crafted JSON payload and remove double quotes from profiler tags
@guance-review-bot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: songlonqi-java

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
0 out of 13 committers have signed the CLA.

❌ amarziali
❌ Yumasi
❌ nikita-tkachenko-datadog
❌ bantonsson
❌ jpbempel
❌ mcculls
❌ cataphract
❌ PerfectSlayer
❌ manuel-alvarez-alvarez
❌ jbachorik
❌ richardstartin
❌ songlonqi-java
❌ smola
You have signed the CLA already but the status is still pending? Let us recheck it.

@songlonqi-java songlonqi-java merged commit 5dc0c28 into guance Apr 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet