-
Notifications
You must be signed in to change notification settings - Fork 315
Fix debugger batch upload URL when using UDS #7166
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
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 54 metrics, 17 unstable metrics. Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.36.0-SNAPSHOT~7145fa8107, baseline=1.36.0-SNAPSHOT~54461eefea
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.578 s) : 0, 1578417
Total [baseline] (11.917 s) : 0, 11917389
Agent [candidate] (1.577 s) : 0, 1577447
Total [candidate] (11.9 s) : 0, 11900100
section iast
Agent [baseline] (1.706 s) : 0, 1705724
Total [baseline] (12.544 s) : 0, 12543683
Agent [candidate] (1.709 s) : 0, 1708712
Total [candidate] (12.561 s) : 0, 12561397
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.719 s) : 0, 1719120
Total [baseline] (12.534 s) : 0, 12533865
Agent [candidate] (1.714 s) : 0, 1714420
Total [candidate] (12.505 s) : 0, 12505429
section iast_TELEMETRY_OFF
Agent [baseline] (1.707 s) : 0, 1707049
Total [baseline] (12.544 s) : 0, 12544382
Agent [candidate] (1.711 s) : 0, 1710812
Total [candidate] (12.573 s) : 0, 12572823
gantt
title insecure-bank - break down per module: candidate=1.36.0-SNAPSHOT~7145fa8107, baseline=1.36.0-SNAPSHOT~54461eefea
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (984.745 ms) : 0, 984745
BytebuddyAgent [candidate] (984.687 ms) : 0, 984687
GlobalTracer [baseline] (450.409 ms) : 0, 450409
GlobalTracer [candidate] (449.784 ms) : 0, 449784
AppSec [baseline] (71.65 ms) : 0, 71650
AppSec [candidate] (71.403 ms) : 0, 71403
Logs Intake [baseline] (419.946 µs) : 0, 420
Logs Intake [candidate] (410.967 µs) : 0, 411
Remote Config [baseline] (870.268 µs) : 0, 870
Remote Config [candidate] (897.999 µs) : 0, 898
Telemetry [baseline] (9.845 ms) : 0, 9845
Telemetry [candidate] (9.778 ms) : 0, 9778
section iast
BytebuddyAgent [baseline] (1.144 s) : 0, 1144457
BytebuddyAgent [candidate] (1.146 s) : 0, 1146194
GlobalTracer [baseline] (431.47 ms) : 0, 431470
GlobalTracer [candidate] (432.583 ms) : 0, 432583
AppSec [baseline] (67.126 ms) : 0, 67126
AppSec [candidate] (66.348 ms) : 0, 66348
IAST [baseline] (27.445 ms) : 0, 27445
IAST [candidate] (30.208 ms) : 0, 30208
Logs Intake [baseline] (398.57 µs) : 0, 399
Logs Intake [candidate] (389.702 µs) : 0, 390
Remote Config [baseline] (761.475 µs) : 0, 761
Remote Config [candidate] (1.518 ms) : 0, 1518
Telemetry [baseline] (14.397 ms) : 0, 14397
Telemetry [candidate] (11.72 ms) : 0, 11720
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (1.152 s) : 0, 1152231
BytebuddyAgent [candidate] (1.149 s) : 0, 1149154
GlobalTracer [baseline] (434.172 ms) : 0, 434172
GlobalTracer [candidate] (433.719 ms) : 0, 433719
AppSec [baseline] (67.364 ms) : 0, 67364
AppSec [candidate] (67.488 ms) : 0, 67488
IAST [baseline] (32.675 ms) : 0, 32675
IAST [candidate] (33.948 ms) : 0, 33948
Logs Intake [baseline] (400.787 µs) : 0, 401
Logs Intake [candidate] (387.003 µs) : 0, 387
Remote Config [baseline] (773.256 µs) : 0, 773
Remote Config [candidate] (760.907 µs) : 0, 761
Telemetry [baseline] (11.612 ms) : 0, 11612
Telemetry [candidate] (9.077 ms) : 0, 9077
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (1.143 s) : 0, 1143298
BytebuddyAgent [candidate] (1.146 s) : 0, 1145840
GlobalTracer [baseline] (434.207 ms) : 0, 434207
GlobalTracer [candidate] (433.236 ms) : 0, 433236
AppSec [baseline] (68.292 ms) : 0, 68292
AppSec [candidate] (68.161 ms) : 0, 68161
IAST [baseline] (29.726 ms) : 0, 29726
IAST [candidate] (33.534 ms) : 0, 33534
Logs Intake [baseline] (392.957 µs) : 0, 393
Logs Intake [candidate] (389.054 µs) : 0, 389
Remote Config [baseline] (752.696 µs) : 0, 753
Remote Config [candidate] (750.693 µs) : 0, 751
Telemetry [baseline] (10.611 ms) : 0, 10611
Telemetry [candidate] (8.94 ms) : 0, 8940
Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.36.0-SNAPSHOT~7145fa8107, baseline=1.36.0-SNAPSHOT~54461eefea
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.58 s) : 0, 1579729
Total [baseline] (14.384 s) : 0, 14383946
Agent [candidate] (1.583 s) : 0, 1583262
Total [candidate] (14.421 s) : 0, 14420777
section appsec
Agent [baseline] (1.759 s) : 0, 1759323
Total [baseline] (14.45 s) : 0, 14449749
Agent [candidate] (1.757 s) : 0, 1757222
Total [candidate] (14.449 s) : 0, 14448552
section iast
Agent [baseline] (1.705 s) : 0, 1704907
Total [baseline] (14.721 s) : 0, 14720857
Agent [candidate] (1.725 s) : 0, 1724665
Total [candidate] (14.891 s) : 0, 14891297
section profiling
Agent [baseline] (1.894 s) : 0, 1893716
Total [baseline] (14.745 s) : 0, 14744533
Agent [candidate] (1.895 s) : 0, 1894926
Total [candidate] (14.665 s) : 0, 14665313
gantt
title petclinic - break down per module: candidate=1.36.0-SNAPSHOT~7145fa8107, baseline=1.36.0-SNAPSHOT~54461eefea
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (985.644 ms) : 0, 985644
BytebuddyAgent [candidate] (989.182 ms) : 0, 989182
GlobalTracer [baseline] (451.056 ms) : 0, 451056
GlobalTracer [candidate] (451.097 ms) : 0, 451097
AppSec [baseline] (71.527 ms) : 0, 71527
AppSec [candidate] (71.167 ms) : 0, 71167
Logs Intake [baseline] (413.659 µs) : 0, 414
Logs Intake [candidate] (416.608 µs) : 0, 417
Remote Config [baseline] (859.185 µs) : 0, 859
Remote Config [candidate] (863.499 µs) : 0, 863
Telemetry [baseline] (9.754 ms) : 0, 9754
Telemetry [candidate] (9.831 ms) : 0, 9831
section appsec
BytebuddyAgent [baseline] (1.003 s) : 0, 1003186
BytebuddyAgent [candidate] (1.001 s) : 0, 1000986
GlobalTracer [baseline] (441.248 ms) : 0, 441248
GlobalTracer [candidate] (441.679 ms) : 0, 441679
AppSec [baseline] (231.758 ms) : 0, 231758
AppSec [candidate] (231.457 ms) : 0, 231457
Logs Intake [baseline] (416.917 µs) : 0, 417
Logs Intake [candidate] (413.375 µs) : 0, 413
Remote Config [baseline] (787.299 µs) : 0, 787
Remote Config [candidate] (770.837 µs) : 0, 771
Telemetry [baseline] (8.903 ms) : 0, 8903
Telemetry [candidate] (9.226 ms) : 0, 9226
IAST [baseline] (24.913 ms) : 0, 24913
IAST [candidate] (24.142 ms) : 0, 24142
section iast
BytebuddyAgent [baseline] (1.145 s) : 0, 1145299
BytebuddyAgent [candidate] (1.158 s) : 0, 1157531
GlobalTracer [baseline] (431.948 ms) : 0, 431948
GlobalTracer [candidate] (436.409 ms) : 0, 436409
AppSec [baseline] (65.807 ms) : 0, 65807
AppSec [candidate] (67.354 ms) : 0, 67354
Logs Intake [baseline] (390.707 µs) : 0, 391
Logs Intake [candidate] (415.541 µs) : 0, 416
Remote Config [baseline] (757.969 µs) : 0, 758
Remote Config [candidate] (788.038 µs) : 0, 788
Telemetry [baseline] (10.961 ms) : 0, 10961
Telemetry [candidate] (11.788 ms) : 0, 11788
IAST [baseline] (30.095 ms) : 0, 30095
IAST [candidate] (30.474 ms) : 0, 30474
section profiling
ProfilingAgent [baseline] (153.617 ms) : 0, 153617
ProfilingAgent [candidate] (155.696 ms) : 0, 155696
BytebuddyAgent [baseline] (983.08 ms) : 0, 983080
BytebuddyAgent [candidate] (983.256 ms) : 0, 983256
GlobalTracer [baseline] (579.526 ms) : 0, 579526
GlobalTracer [candidate] (578.689 ms) : 0, 578689
AppSec [baseline] (73.121 ms) : 0, 73121
AppSec [candidate] (72.932 ms) : 0, 72932
Logs Intake [baseline] (419.246 µs) : 0, 419
Logs Intake [candidate] (412.435 µs) : 0, 412
Remote Config [baseline] (950.676 µs) : 0, 951
Remote Config [candidate] (961.305 µs) : 0, 961
Telemetry [baseline] (9.514 ms) : 0, 9514
Telemetry [candidate] (9.442 ms) : 0, 9442
Profiling [baseline] (153.685 ms) : 0, 153685
Profiling [candidate] (155.772 ms) : 0, 155772
LoadParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 6 metrics, 22 unstable metrics. Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.36.0-SNAPSHOT~7145fa8107, baseline=1.36.0-SNAPSHOT~54461eefea
dateFormat X
axisFormat %s
section baseline
no_agent (1.707 ms) : 1682, 1731
. : milestone, 1707,
appsec (2.179 ms) : 2148, 2210
. : milestone, 2179,
appsec_no_iast (2.191 ms) : 2159, 2223
. : milestone, 2191,
iast (1.881 ms) : 1852, 1911
. : milestone, 1881,
profiling (1.929 ms) : 1892, 1965
. : milestone, 1929,
tracing (1.86 ms) : 1828, 1892
. : milestone, 1860,
section candidate
no_agent (1.708 ms) : 1683, 1733
. : milestone, 1708,
appsec (2.195 ms) : 2164, 2226
. : milestone, 2195,
appsec_no_iast (2.169 ms) : 2138, 2201
. : milestone, 2169,
iast (1.854 ms) : 1824, 1884
. : milestone, 1854,
profiling (1.954 ms) : 1917, 1991
. : milestone, 1954,
tracing (1.879 ms) : 1847, 1910
. : milestone, 1879,
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.36.0-SNAPSHOT~7145fa8107, baseline=1.36.0-SNAPSHOT~54461eefea
dateFormat X
axisFormat %s
section baseline
no_agent (451.894 µs) : 423, 481
. : milestone, 452,
iast (588.942 µs) : 557, 621
. : milestone, 589,
iast_FULL (684.696 µs) : 653, 716
. : milestone, 685,
iast_GLOBAL (623.608 µs) : 592, 655
. : milestone, 624,
iast_HARDCODED_SECRET_DISABLED (589.237 µs) : 558, 621
. : milestone, 589,
iast_INACTIVE (553.164 µs) : 522, 584
. : milestone, 553,
iast_TELEMETRY_OFF (578.504 µs) : 547, 610
. : milestone, 579,
tracing (539.03 µs) : 509, 569
. : milestone, 539,
section candidate
no_agent (448.502 µs) : 420, 477
. : milestone, 449,
iast (587.207 µs) : 555, 619
. : milestone, 587,
iast_FULL (680.987 µs) : 649, 713
. : milestone, 681,
iast_GLOBAL (622.328 µs) : 591, 654
. : milestone, 622,
iast_HARDCODED_SECRET_DISABLED (584.97 µs) : 554, 616
. : milestone, 585,
iast_INACTIVE (559.999 µs) : 529, 591
. : milestone, 560,
iast_TELEMETRY_OFF (576.907 µs) : 545, 608
. : milestone, 577,
tracing (543.973 µs) : 514, 574
. : milestone, 544,
DacapoParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 11 metrics, 1 unstable metrics. Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.36.0-SNAPSHOT~7145fa8107, baseline=1.36.0-SNAPSHOT~54461eefea
dateFormat X
axisFormat %s
section baseline
no_agent (21.086 s) : 21086000, 21086000
. : milestone, 21086000,
appsec (21.648 s) : 21648000, 21648000
. : milestone, 21648000,
iast (24.649 s) : 24649000, 24649000
. : milestone, 24649000,
iast_GLOBAL (25.065 s) : 25065000, 25065000
. : milestone, 25065000,
profiling (21.173 s) : 21173000, 21173000
. : milestone, 21173000,
tracing (21.346 s) : 21346000, 21346000
. : milestone, 21346000,
section candidate
no_agent (20.882 s) : 20882000, 20882000
. : milestone, 20882000,
appsec (21.789 s) : 21789000, 21789000
. : milestone, 21789000,
iast (24.221 s) : 24221000, 24221000
. : milestone, 24221000,
iast_GLOBAL (24.38 s) : 24380000, 24380000
. : milestone, 24380000,
profiling (21.015 s) : 21015000, 21015000
. : milestone, 21015000,
tracing (21.737 s) : 21737000, 21737000
. : milestone, 21737000,
Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.36.0-SNAPSHOT~7145fa8107, baseline=1.36.0-SNAPSHOT~54461eefea
dateFormat X
axisFormat %s
section baseline
no_agent (1.544 ms) : 1531, 1557
. : milestone, 1544,
appsec (2.685 ms) : 2624, 2745
. : milestone, 2685,
iast (2.346 ms) : 2275, 2417
. : milestone, 2346,
iast_GLOBAL (2.399 ms) : 2328, 2470
. : milestone, 2399,
profiling (2.22 ms) : 2159, 2281
. : milestone, 2220,
tracing (2.174 ms) : 2116, 2232
. : milestone, 2174,
section candidate
no_agent (1.542 ms) : 1530, 1555
. : milestone, 1542,
appsec (2.682 ms) : 2621, 2742
. : milestone, 2682,
iast (2.34 ms) : 2270, 2410
. : milestone, 2340,
iast_GLOBAL (2.412 ms) : 2339, 2485
. : milestone, 2412,
profiling (2.887 ms) : 2629, 3146
. : milestone, 2887,
tracing (2.163 ms) : 2106, 2221
. : milestone, 2163,
|
jpbempel
approved these changes
Jun 12, 2024
mcculls
added a commit
that referenced
this pull request
Jun 12, 2024
mcculls
added a commit
that referenced
this pull request
Jun 12, 2024
mcculls
added a commit
that referenced
this pull request
Jun 12, 2024
mcculls
added a commit
that referenced
this pull request
Jun 12, 2024
5 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
These direct uses of
Config.agentUrlwere missed in #7094This PR fixes them the same way we fixed uses of
Config.getAgentUrl()in the original PR; i.e. when using UDS any endpoint usingConfig.agentUrldirectly rather thanSharedCommunicationObjectsshould use a placeholder ofhttp://${agentHost}:${agentPort}instead ofunix:...to satisfy OkHttp'sHttpUrl.get.This is necessary because the
agentUrlcaptured inConfigwill now preserve the original connection string, which when using UDS will start withunix:. This protocol will be rejected by OkHttp'sHttpUrl.get, so to satisfy that call we use the abovehttp:placeholder - note the actual connection will be tunnelled over UDS, which is why the URL is a placeholder.Ideally all remote connections to the agent will eventually migrate to use
SharedCommunicationObjectsto avoid having to repeat this.