Conversation
|
Hi! 👋 Thanks for your pull request! 🎉 To help us review it, please make sure to:
If you need help, please check our contributing guidelines. |
|
🎯 Code Coverage 🔗 Commit SHA: 658120f | Docs | Was this helpful? Give us feedback! |
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 43 metrics, 16 unstable metrics. Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.53.0-SNAPSHOT~658120f76e, baseline=1.53.0-SNAPSHOT~91f463686c
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.049 s) : 0, 1049090
Total [baseline] (10.641 s) : 0, 10641356
Agent [candidate] (1.05 s) : 0, 1049624
Total [candidate] (10.696 s) : 0, 10695882
section appsec
Agent [baseline] (1.223 s) : 0, 1222818
Total [baseline] (10.782 s) : 0, 10781837
Agent [candidate] (1.233 s) : 0, 1232587
Total [candidate] (10.828 s) : 0, 10828176
section iast
Agent [baseline] (1.188 s) : 0, 1188383
Total [baseline] (11.09 s) : 0, 11089593
Agent [candidate] (1.184 s) : 0, 1183548
Total [candidate] (11.032 s) : 0, 11032171
section profiling
Agent [baseline] (1.199 s) : 0, 1198921
Total [baseline] (10.892 s) : 0, 10891528
Agent [candidate] (1.201 s) : 0, 1200748
Total [candidate] (10.875 s) : 0, 10874882
gantt
title petclinic - break down per module: candidate=1.53.0-SNAPSHOT~658120f76e, baseline=1.53.0-SNAPSHOT~91f463686c
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.466 ms) : 0, 1466
crashtracking [candidate] (1.442 ms) : 0, 1442
BytebuddyAgent [baseline] (733.575 ms) : 0, 733575
BytebuddyAgent [candidate] (733.747 ms) : 0, 733747
GlobalTracer [baseline] (242.265 ms) : 0, 242265
GlobalTracer [candidate] (243.457 ms) : 0, 243457
AppSec [baseline] (30.094 ms) : 0, 30094
AppSec [candidate] (30.177 ms) : 0, 30177
Debugger [baseline] (6.019 ms) : 0, 6019
Debugger [candidate] (6.08 ms) : 0, 6080
Remote Config [baseline] (687.92 µs) : 0, 688
Remote Config [candidate] (669.865 µs) : 0, 670
Telemetry [baseline] (13.816 ms) : 0, 13816
Telemetry [candidate] (12.924 ms) : 0, 12924
section appsec
crashtracking [baseline] (1.456 ms) : 0, 1456
crashtracking [candidate] (1.451 ms) : 0, 1451
BytebuddyAgent [baseline] (755.996 ms) : 0, 755996
BytebuddyAgent [candidate] (761.615 ms) : 0, 761615
GlobalTracer [baseline] (235.302 ms) : 0, 235302
GlobalTracer [candidate] (237.118 ms) : 0, 237118
AppSec [baseline] (170.508 ms) : 0, 170508
AppSec [candidate] (168.624 ms) : 0, 168624
Debugger [baseline] (5.778 ms) : 0, 5778
Debugger [candidate] (6.661 ms) : 0, 6661
Remote Config [baseline] (636.889 µs) : 0, 637
Remote Config [candidate] (654.894 µs) : 0, 655
Telemetry [baseline] (8.47 ms) : 0, 8470
Telemetry [candidate] (11.562 ms) : 0, 11562
IAST [baseline] (23.499 ms) : 0, 23499
IAST [candidate] (23.743 ms) : 0, 23743
section iast
crashtracking [baseline] (1.467 ms) : 0, 1467
crashtracking [candidate] (1.446 ms) : 0, 1446
BytebuddyAgent [baseline] (858.296 ms) : 0, 858296
BytebuddyAgent [candidate] (852.836 ms) : 0, 852836
GlobalTracer [baseline] (234.242 ms) : 0, 234242
GlobalTracer [candidate] (235.133 ms) : 0, 235133
AppSec [baseline] (26.394 ms) : 0, 26394
AppSec [candidate] (28.677 ms) : 0, 28677
Debugger [baseline] (7.525 ms) : 0, 7525
Debugger [candidate] (5.751 ms) : 0, 5751
Remote Config [baseline] (628.238 µs) : 0, 628
Remote Config [candidate] (601.491 µs) : 0, 601
Telemetry [baseline] (8.38 ms) : 0, 8380
Telemetry [candidate] (8.426 ms) : 0, 8426
IAST [baseline] (30.298 ms) : 0, 30298
IAST [candidate] (29.601 ms) : 0, 29601
section profiling
crashtracking [baseline] (1.429 ms) : 0, 1429
crashtracking [candidate] (1.436 ms) : 0, 1436
BytebuddyAgent [baseline] (763.047 ms) : 0, 763047
BytebuddyAgent [candidate] (764.748 ms) : 0, 764748
GlobalTracer [baseline] (223.227 ms) : 0, 223227
GlobalTracer [candidate] (223.546 ms) : 0, 223546
AppSec [baseline] (30.028 ms) : 0, 30028
AppSec [candidate] (30.025 ms) : 0, 30025
Debugger [baseline] (6.291 ms) : 0, 6291
Debugger [candidate] (7.016 ms) : 0, 7016
Remote Config [baseline] (728.025 µs) : 0, 728
Remote Config [candidate] (685.561 µs) : 0, 686
Telemetry [baseline] (16.535 ms) : 0, 16535
Telemetry [candidate] (15.574 ms) : 0, 15574
ProfilingAgent [baseline] (107.246 ms) : 0, 107246
ProfilingAgent [candidate] (107.291 ms) : 0, 107291
Profiling [baseline] (107.911 ms) : 0, 107911
Profiling [candidate] (107.958 ms) : 0, 107958
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.53.0-SNAPSHOT~658120f76e, baseline=1.53.0-SNAPSHOT~91f463686c
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.048 s) : 0, 1048123
Total [baseline] (8.644 s) : 0, 8643764
Agent [candidate] (1.05 s) : 0, 1050330
Total [candidate] (8.634 s) : 0, 8634071
section iast
Agent [baseline] (1.182 s) : 0, 1182134
Total [baseline] (9.348 s) : 0, 9348095
Agent [candidate] (1.198 s) : 0, 1197747
Total [candidate] (9.322 s) : 0, 9322214
gantt
title insecure-bank - break down per module: candidate=1.53.0-SNAPSHOT~658120f76e, baseline=1.53.0-SNAPSHOT~91f463686c
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.448 ms) : 0, 1448
crashtracking [candidate] (1.475 ms) : 0, 1475
BytebuddyAgent [baseline] (733.197 ms) : 0, 733197
BytebuddyAgent [candidate] (737.402 ms) : 0, 737402
GlobalTracer [baseline] (242.211 ms) : 0, 242211
GlobalTracer [candidate] (242.992 ms) : 0, 242992
AppSec [baseline] (30.06 ms) : 0, 30060
AppSec [candidate] (30.013 ms) : 0, 30013
Debugger [baseline] (6.039 ms) : 0, 6039
Debugger [candidate] (6.037 ms) : 0, 6037
Remote Config [baseline] (671.076 µs) : 0, 671
Remote Config [candidate] (672.429 µs) : 0, 672
Telemetry [baseline] (13.575 ms) : 0, 13575
Telemetry [candidate] (10.615 ms) : 0, 10615
section iast
crashtracking [baseline] (1.461 ms) : 0, 1461
crashtracking [candidate] (1.464 ms) : 0, 1464
BytebuddyAgent [baseline] (852.799 ms) : 0, 852799
BytebuddyAgent [candidate] (866.07 ms) : 0, 866070
GlobalTracer [baseline] (234.596 ms) : 0, 234596
GlobalTracer [candidate] (235.586 ms) : 0, 235586
AppSec [baseline] (27.159 ms) : 0, 27159
AppSec [candidate] (27.115 ms) : 0, 27115
Debugger [baseline] (6.594 ms) : 0, 6594
Debugger [candidate] (7.437 ms) : 0, 7437
Remote Config [baseline] (608.378 µs) : 0, 608
Remote Config [candidate] (601.986 µs) : 0, 602
Telemetry [baseline] (8.269 ms) : 0, 8269
Telemetry [candidate] (9.182 ms) : 0, 9182
IAST [baseline] (29.611 ms) : 0, 29611
IAST [candidate] (29.003 ms) : 0, 29003
LoadParameters
See matching parameters
SummaryFound 2 performance improvements and 3 performance regressions! Performance is the same for 7 metrics, 12 unstable metrics.
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.53.0-SNAPSHOT~658120f76e, baseline=1.53.0-SNAPSHOT~91f463686c
dateFormat X
axisFormat %s
section baseline
no_agent (36.459 ms) : 36163, 36754
. : milestone, 36459,
appsec (48.45 ms) : 48020, 48880
. : milestone, 48450,
code_origins (43.617 ms) : 43235, 43999
. : milestone, 43617,
iast (42.533 ms) : 42175, 42890
. : milestone, 42533,
profiling (48.162 ms) : 47710, 48614
. : milestone, 48162,
tracing (45.05 ms) : 44672, 45428
. : milestone, 45050,
section candidate
no_agent (35.614 ms) : 35326, 35902
. : milestone, 35614,
appsec (46.277 ms) : 45864, 46689
. : milestone, 46277,
code_origins (45.175 ms) : 44786, 45564
. : milestone, 45175,
iast (44.613 ms) : 44234, 44993
. : milestone, 44613,
profiling (48.318 ms) : 47871, 48764
. : milestone, 48318,
tracing (41.984 ms) : 41630, 42339
. : milestone, 41984,
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.53.0-SNAPSHOT~658120f76e, baseline=1.53.0-SNAPSHOT~91f463686c
dateFormat X
axisFormat %s
section baseline
no_agent (4.306 ms) : 4258, 4353
. : milestone, 4306,
iast (9.262 ms) : 9103, 9422
. : milestone, 9262,
iast_FULL (14.174 ms) : 13887, 14461
. : milestone, 14174,
iast_GLOBAL (10.134 ms) : 9954, 10313
. : milestone, 10134,
profiling (8.659 ms) : 8521, 8797
. : milestone, 8659,
tracing (7.636 ms) : 7529, 7744
. : milestone, 7636,
section candidate
no_agent (4.43 ms) : 4380, 4479
. : milestone, 4430,
iast (9.393 ms) : 9241, 9545
. : milestone, 9393,
iast_FULL (14.351 ms) : 14057, 14645
. : milestone, 14351,
iast_GLOBAL (10.59 ms) : 10403, 10777
. : milestone, 10590,
profiling (8.881 ms) : 8744, 9017
. : milestone, 8881,
tracing (7.753 ms) : 7643, 7863
. : milestone, 7753,
DacapoParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 10 metrics, 2 unstable metrics. Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.53.0-SNAPSHOT~658120f76e, baseline=1.53.0-SNAPSHOT~91f463686c
dateFormat X
axisFormat %s
section baseline
no_agent (1.467 ms) : 1456, 1479
. : milestone, 1467,
appsec (3.591 ms) : 3378, 3804
. : milestone, 3591,
iast (2.196 ms) : 2133, 2259
. : milestone, 2196,
iast_GLOBAL (2.241 ms) : 2178, 2305
. : milestone, 2241,
profiling (2.448 ms) : 2291, 2605
. : milestone, 2448,
tracing (2.014 ms) : 1965, 2064
. : milestone, 2014,
section candidate
no_agent (1.471 ms) : 1459, 1482
. : milestone, 1471,
appsec (3.654 ms) : 3432, 3877
. : milestone, 3654,
iast (2.203 ms) : 2140, 2266
. : milestone, 2203,
iast_GLOBAL (2.236 ms) : 2173, 2299
. : milestone, 2236,
profiling (2.047 ms) : 1996, 2099
. : milestone, 2047,
tracing (2.025 ms) : 1975, 2074
. : milestone, 2025,
Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.53.0-SNAPSHOT~658120f76e, baseline=1.53.0-SNAPSHOT~91f463686c
dateFormat X
axisFormat %s
section baseline
no_agent (14.902 s) : 14902000, 14902000
. : milestone, 14902000,
appsec (15.059 s) : 15059000, 15059000
. : milestone, 15059000,
iast (18.727 s) : 18727000, 18727000
. : milestone, 18727000,
iast_GLOBAL (17.791 s) : 17791000, 17791000
. : milestone, 17791000,
profiling (15.495 s) : 15495000, 15495000
. : milestone, 15495000,
tracing (14.817 s) : 14817000, 14817000
. : milestone, 14817000,
section candidate
no_agent (15.679 s) : 15679000, 15679000
. : milestone, 15679000,
appsec (14.948 s) : 14948000, 14948000
. : milestone, 14948000,
iast (18.384 s) : 18384000, 18384000
. : milestone, 18384000,
iast_GLOBAL (18.178 s) : 18178000, 18178000
. : milestone, 18178000,
profiling (15.338 s) : 15338000, 15338000
. : milestone, 15338000,
tracing (14.986 s) : 14986000, 14986000
. : milestone, 14986000,
|
e7c2f3e to
1c949d2
Compare
1c949d2 to
658120f
Compare
| "Failed to inject helper classes - instrumentation.class=" | ||
| + requestingName | ||
| + " instrumentation.target.classloader={} instrumentation.target.class={}", | ||
| requestingName, |
There was a problem hiding this comment.
The requestingName was embedded in the message for telemetry purposes, but the argument wasn't removed.
There was a problem hiding this comment.
Thanks. This is exactly the example I had in mind as an approved exception for dynamic messages.
| Package | Type | Package file | Manager | Update | Change | |---|---|---|---|---|---| | [com.datadoghq:dd-trace-api](https://github.com/datadog/dd-trace-java) | dependencies | misk/gradle/libs.versions.toml | gradle | minor | `1.52.1` -> `1.53.0` | --- ### Release Notes <details> <summary>datadog/dd-trace-java (com.datadoghq:dd-trace-api)</summary> ### [`v1.53.0`](https://github.com/DataDog/dd-trace-java/releases/tag/v1.53.0): 1.53.0 ### Components #### Application Security Management (WAF) - 🐛 Fix NullPointerException log in AppSec ([#​9355](DataDog/dd-trace-java#9355) - [@​jandro996](https://github.com/jandro996)) - ✨ Add Forwarded header as a source to client IP resolution ([#​9310](DataDog/dd-trace-java#9310) - [@​jandro996](https://github.com/jandro996)) #### Build & Tooling - 📖 Update CODEOWNERS ([#​9294](DataDog/dd-trace-java#9294) - [@​mhlidd](https://github.com/mhlidd)) - ✨ Add dependency on Maven Central deployment to OCI publish jobs ([#​9204](DataDog/dd-trace-java#9204) - [@​sarahchen6](https://github.com/sarahchen6)) #### Continuous Integration Visibility - ✨ Upload code coverage reports to Datadog ([#​9425](DataDog/dd-trace-java#9425) - [@​nikita-tkachenko-datadog](https://github.com/nikita-tkachenko-datadog)) - ✨⚡ Do not follow symlinks by default when building repository index ([#​9318](DataDog/dd-trace-java#9318) - [@​nikita-tkachenko-datadog](https://github.com/nikita-tkachenko-datadog)) - ✨ Improve tag propagation between test event levels ([#​9278](DataDog/dd-trace-java#9278) - [@​daniel-mohedano](https://github.com/daniel-mohedano)) #### Data Streams Monitoring - ✨ Create container hash tags propagation ([#​9282](DataDog/dd-trace-java#9282) - [@​ygree](https://github.com/ygree)) #### Database Monitoring - 🐛 Prevent crash in SQL Server's JDBC when tracing execute methods with generated keys ([#​9321](DataDog/dd-trace-java#9321) - [@​na-ji](https://github.com/na-ji)) - ✨ Create container hash tags propagation ([#​9282](DataDog/dd-trace-java#9282) - [@​ygree](https://github.com/ygree)) #### GraalVM native-image - 🐛 Update GraalVM config to reflect TempLocationManager's new package ([#​9337](DataDog/dd-trace-java#9337) - [@​luneo7](https://github.com/luneo7) - thanks for the contribution!) #### Metrics - 🐛 Add reflect config for CSS and graalvm ([#​9449](DataDog/dd-trace-java#9449) - [@​amarziali](https://github.com/amarziali)) - ✨ Align CSS health metrics ([#​9444](DataDog/dd-trace-java#9444) - [@​amarziali](https://github.com/amarziali)) - ✨ Aggregate peer tags for consumer span kind ([#​9442](DataDog/dd-trace-java#9442) - [@​amarziali](https://github.com/amarziali)) - ✨ Implement health metrics for client stats ([#​9377](DataDog/dd-trace-java#9377) - [@​amarziali](https://github.com/amarziali)) - ✨ Add peer tags, span kind and trace root flag to MetricKey bucket ([#​9178](DataDog/dd-trace-java#9178) - [@​bric3](https://github.com/bric3)) #### Profiling - ✨ Create a profile flare reporter (requires tracing or CI vis to be enabled) ([#​9433](DataDog/dd-trace-java#9433) - [@​MattAlp](https://github.com/MattAlp)) - 🐛 Properly fall-back on UDS profiling URL ([#​9401](DataDog/dd-trace-java#9401) - [@​jbachorik](https://github.com/jbachorik)) - 🐛 Properly handle trace agent IPv6 URL in profiling ([#​9334](DataDog/dd-trace-java#9334) - [@​jbachorik](https://github.com/jbachorik)) #### Realtime User Monitoring - 🐛 Avoid multiple injections on dispatch on jboss ([#​9392](DataDog/dd-trace-java#9392) - [@​amarziali](https://github.com/amarziali)) - ✨ Support async servlet for RUM injection ([#​9333](DataDog/dd-trace-java#9333) - [@​amarziali](https://github.com/amarziali)) - 🐛 Improve RUM injection matching and avoid truncating responses ([#​9323](DataDog/dd-trace-java#9323) - [@​amarziali](https://github.com/amarziali)) - 💡 Add telemetry for the RUM injector ([#​9267](DataDog/dd-trace-java#9267) - [@​sarahchen6](https://github.com/sarahchen6)) - ✨ Make rum injector stream/writer more resilient to errors ([#​9184](DataDog/dd-trace-java#9184) - [@​amarziali](https://github.com/amarziali)) #### Telemetry - 🐛 Remediate error logs ([#​9459](DataDog/dd-trace-java#9459) - [@​ygree](https://github.com/ygree)) - ✨ Report config\_id for Hands Off Config files ([#​9299](DataDog/dd-trace-java#9299) - [@​mtoffl01](https://github.com/mtoffl01)) - ✨ Adding Config Inversion Telemetry component ([#​9244](DataDog/dd-trace-java#9244) - [@​mhlidd](https://github.com/mhlidd)) #### Trace context propagation - 🧹 Migrate instrumentations to Context API ([#​9384](DataDog/dd-trace-java#9384), [#​9378](DataDog/dd-trace-java#9378), [#​9358](DataDog/dd-trace-java#9358) - [@​PerfectSlayer](https://github.com/PerfectSlayer)) - ✨🔍 Add baggage propagation telemetry ([#​9289](DataDog/dd-trace-java#9289) - [@​rachelyangdog](https://github.com/rachelyangdog)) - 🧹 Improve Instrumenter API to use Context instead of Span ([#​9211](DataDog/dd-trace-java#9211) - [@​PerfectSlayer](https://github.com/PerfectSlayer)) #### Tracer core - 💡:test\_tube: Enable process level tags for java 21 applications ([#​9458](DataDog/dd-trace-java#9458) - [@​amarziali](https://github.com/amarziali)) - ✨ Move base and peer service adder before span is published ([#​9408](DataDog/dd-trace-java#9408) - [@​amarziali](https://github.com/amarziali)) - ✨ Add ErrorPriorities level for manual instrumentation ([#​9387](DataDog/dd-trace-java#9387) - [@​cecile75](https://github.com/cecile75)) - ✨ Throw InvalidBooleanValueException in ConfigConverter.booleanValueOf ([#​9370](DataDog/dd-trace-java#9370) - [@​mtoffl01](https://github.com/mtoffl01)) - 🐛 Avoid NPE on featureDiscovery creation ([#​9353](DataDog/dd-trace-java#9353) - [@​amarziali](https://github.com/amarziali)) - ✨ Stable Config improvements ([#​9259](DataDog/dd-trace-java#9259) - [@​mtoffl01](https://github.com/mtoffl01)) ### Instrumentations #### gRPC instrumentation - 🐛 Changing GRPC instrumentation codes from `status.code` to `grpc.status.code` ([#​9367](DataDog/dd-trace-java#9367) - [@​mhlidd](https://github.com/mhlidd)) #### JDBC instrumentation - 🐛 Prevent crash in SQL Server's JDBC when tracing execute methods with generated keys ([#​9321](DataDog/dd-trace-java#9321) - [@​na-ji](https://github.com/na-ji)) #### Kotlin instrumentation - 🐛 Take defensive copy of original scope stack when restoring on different thread ([#​9403](DataDog/dd-trace-java#9403) - [@​mcculls](https://github.com/mcculls)) #### OpenTelemetry instrumentation - 🐛 Fix using SpanAttribute annotation on multiple parameters ([#​9412](DataDog/dd-trace-java#9412) - [@​cecile75](https://github.com/cecile75)) </details> --- ### Configuration 📅 **Schedule**: Branch creation - "after 6pm every weekday,before 2am every weekday" in timezone Australia/Melbourne, Automerge - At any time (no schedule defined). 🚦 **Automerge**: Enabled. ♻ **Rebasing**: Never, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate). GitOrigin-RevId: e23ec866cef91afc6b5226d5f0dc7da538d947e8
What Does This Do
Prevents passing exception messages and dynamically concatenated strings as an error message to the telemetry
Motivation
Sensitive data leaks prevention
Additional Notes
Contributor Checklist
type:and (comp:orinst:) labels in addition to any usefull labelsclose,fixor any linking keywords when referencing an issue.Use
solvesinstead, and assign the PR milestone to the issueJira ticket: [PROJ-IDENT]