-
Notifications
You must be signed in to change notification settings - Fork 315
Preserve unix: agent URLs
#7094
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 52 metrics, 11 unstable metrics. Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.35.0-SNAPSHOT~096483556f, baseline=1.35.0-SNAPSHOT~4b5cad9604
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.58 s) : 0, 1580344
Total [baseline] (11.952 s) : 0, 11951843
Agent [candidate] (1.571 s) : 0, 1571225
Total [candidate] (11.947 s) : 0, 11947383
section iast
Agent [baseline] (1.704 s) : 0, 1703858
Total [baseline] (12.559 s) : 0, 12559282
Agent [candidate] (1.708 s) : 0, 1708491
Total [candidate] (12.557 s) : 0, 12557230
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.71 s) : 0, 1709633
Total [baseline] (12.574 s) : 0, 12574236
Agent [candidate] (1.702 s) : 0, 1702107
Total [candidate] (12.496 s) : 0, 12495569
section iast_TELEMETRY_OFF
Agent [baseline] (1.731 s) : 0, 1731146
Total [baseline] (12.605 s) : 0, 12605087
Agent [candidate] (1.712 s) : 0, 1712103
Total [candidate] (12.578 s) : 0, 12578345
gantt
title insecure-bank - break down per module: candidate=1.35.0-SNAPSHOT~096483556f, baseline=1.35.0-SNAPSHOT~4b5cad9604
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (972.548 ms) : 0, 972548
BytebuddyAgent [candidate] (967.663 ms) : 0, 967663
GlobalTracer [baseline] (464.22 ms) : 0, 464220
GlobalTracer [candidate] (461.332 ms) : 0, 461332
AppSec [baseline] (71.81 ms) : 0, 71810
AppSec [candidate] (70.898 ms) : 0, 70898
Remote Config [baseline] (859.539 µs) : 0, 860
Remote Config [candidate] (836.933 µs) : 0, 837
Telemetry [baseline] (10.12 ms) : 0, 10120
Telemetry [candidate] (9.876 ms) : 0, 9876
section iast
BytebuddyAgent [baseline] (1.145 s) : 0, 1144899
BytebuddyAgent [candidate] (1.146 s) : 0, 1146006
GlobalTracer [baseline] (430.395 ms) : 0, 430395
GlobalTracer [candidate] (431.145 ms) : 0, 431145
AppSec [baseline] (66.653 ms) : 0, 66653
AppSec [candidate] (66.217 ms) : 0, 66217
IAST [baseline] (29.852 ms) : 0, 29852
IAST [candidate] (33.886 ms) : 0, 33886
Remote Config [baseline] (783.939 µs) : 0, 784
Remote Config [candidate] (778.314 µs) : 0, 778
Telemetry [baseline] (11.532 ms) : 0, 11532
Telemetry [candidate] (10.652 ms) : 0, 10652
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (1.149 s) : 0, 1148994
BytebuddyAgent [candidate] (1.142 s) : 0, 1141630
GlobalTracer [baseline] (430.361 ms) : 0, 430361
GlobalTracer [candidate] (430.259 ms) : 0, 430259
AppSec [baseline] (67.234 ms) : 0, 67234
AppSec [candidate] (67.121 ms) : 0, 67121
IAST [baseline] (30.753 ms) : 0, 30753
IAST [candidate] (32.482 ms) : 0, 32482
Remote Config [baseline] (768.145 µs) : 0, 768
Remote Config [candidate] (792.906 µs) : 0, 793
Telemetry [baseline] (11.611 ms) : 0, 11611
Telemetry [candidate] (10.009 ms) : 0, 10009
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (1.162 s) : 0, 1161810
BytebuddyAgent [candidate] (1.151 s) : 0, 1150729
GlobalTracer [baseline] (437.194 ms) : 0, 437194
GlobalTracer [candidate] (432.464 ms) : 0, 432464
AppSec [baseline] (69.318 ms) : 0, 69318
AppSec [candidate] (67.763 ms) : 0, 67763
IAST [baseline] (31.014 ms) : 0, 31014
IAST [candidate] (28.015 ms) : 0, 28015
Remote Config [baseline] (785.999 µs) : 0, 786
Remote Config [candidate] (779.928 µs) : 0, 780
Telemetry [baseline] (10.867 ms) : 0, 10867
Telemetry [candidate] (12.35 ms) : 0, 12350
Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.35.0-SNAPSHOT~096483556f, baseline=1.35.0-SNAPSHOT~4b5cad9604
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.577 s) : 0, 1577282
Total [baseline] (14.237 s) : 0, 14236786
Agent [candidate] (1.573 s) : 0, 1573275
Total [candidate] (14.26 s) : 0, 14260226
section appsec
Agent [baseline] (1.752 s) : 0, 1752058
Total [baseline] (14.496 s) : 0, 14495687
Agent [candidate] (1.754 s) : 0, 1754299
Total [candidate] (14.323 s) : 0, 14322904
section iast
Agent [baseline] (1.714 s) : 0, 1714223
Total [baseline] (14.713 s) : 0, 14712799
Agent [candidate] (1.705 s) : 0, 1705287
Total [candidate] (14.778 s) : 0, 14778038
section profiling
Agent [baseline] (1.881 s) : 0, 1881264
Total [baseline] (14.674 s) : 0, 14673673
Agent [candidate] (1.883 s) : 0, 1882646
Total [candidate] (14.548 s) : 0, 14547629
gantt
title petclinic - break down per module: candidate=1.35.0-SNAPSHOT~096483556f, baseline=1.35.0-SNAPSHOT~4b5cad9604
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (971.826 ms) : 0, 971826
BytebuddyAgent [candidate] (969.3 ms) : 0, 969300
GlobalTracer [baseline] (463.058 ms) : 0, 463058
GlobalTracer [candidate] (461.823 ms) : 0, 461823
AppSec [baseline] (71.016 ms) : 0, 71016
AppSec [candidate] (70.785 ms) : 0, 70785
Remote Config [baseline] (840.492 µs) : 0, 840
Remote Config [candidate] (846.207 µs) : 0, 846
Telemetry [baseline] (9.893 ms) : 0, 9893
Telemetry [candidate] (9.884 ms) : 0, 9884
section appsec
BytebuddyAgent [baseline] (1.0 s) : 0, 1000476
BytebuddyAgent [candidate] (1.003 s) : 0, 1002667
GlobalTracer [baseline] (437.286 ms) : 0, 437286
GlobalTracer [candidate] (437.744 ms) : 0, 437744
AppSec [baseline] (231.728 ms) : 0, 231728
AppSec [candidate] (231.113 ms) : 0, 231113
Remote Config [baseline] (785.294 µs) : 0, 785
Remote Config [candidate] (784.141 µs) : 0, 784
Telemetry [baseline] (9.19 ms) : 0, 9190
Telemetry [candidate] (9.227 ms) : 0, 9227
IAST [baseline] (24.19 ms) : 0, 24190
IAST [candidate] (24.26 ms) : 0, 24260
section iast
BytebuddyAgent [baseline] (1.15 s) : 0, 1149801
BytebuddyAgent [candidate] (1.145 s) : 0, 1144829
GlobalTracer [baseline] (432.678 ms) : 0, 432678
GlobalTracer [candidate] (431.096 ms) : 0, 431096
AppSec [baseline] (67.888 ms) : 0, 67888
AppSec [candidate] (66.499 ms) : 0, 66499
Remote Config [baseline] (797.761 µs) : 0, 798
Remote Config [candidate] (762.155 µs) : 0, 762
Telemetry [baseline] (12.365 ms) : 0, 12365
Telemetry [candidate] (10.651 ms) : 0, 10651
IAST [baseline] (30.973 ms) : 0, 30973
IAST [candidate] (31.702 ms) : 0, 31702
section profiling
ProfilingAgent [baseline] (147.194 ms) : 0, 147194
ProfilingAgent [candidate] (148.693 ms) : 0, 148693
BytebuddyAgent [baseline] (980.062 ms) : 0, 980062
BytebuddyAgent [candidate] (980.826 ms) : 0, 980826
GlobalTracer [baseline] (577.312 ms) : 0, 577312
GlobalTracer [candidate] (577.24 ms) : 0, 577240
AppSec [baseline] (72.563 ms) : 0, 72563
AppSec [candidate] (71.687 ms) : 0, 71687
Remote Config [baseline] (1.083 ms) : 0, 1083
Remote Config [candidate] (1.107 ms) : 0, 1107
Telemetry [baseline] (9.794 ms) : 0, 9794
Telemetry [candidate] (9.838 ms) : 0, 9838
Profiling [baseline] (147.25 ms) : 0, 147250
Profiling [candidate] (148.751 ms) : 0, 148751
LoadParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 7 metrics, 21 unstable metrics. Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.35.0-SNAPSHOT~096483556f, baseline=1.35.0-SNAPSHOT~4b5cad9604
dateFormat X
axisFormat %s
section baseline
no_agent (1.703 ms) : 1679, 1728
. : milestone, 1703,
appsec (2.187 ms) : 2153, 2220
. : milestone, 2187,
appsec_no_iast (2.186 ms) : 2153, 2220
. : milestone, 2186,
iast (1.901 ms) : 1872, 1930
. : milestone, 1901,
profiling (1.895 ms) : 1864, 1926
. : milestone, 1895,
tracing (1.894 ms) : 1861, 1926
. : milestone, 1894,
section candidate
no_agent (1.706 ms) : 1680, 1732
. : milestone, 1706,
appsec (2.173 ms) : 2140, 2206
. : milestone, 2173,
appsec_no_iast (2.191 ms) : 2159, 2224
. : milestone, 2191,
iast (1.9 ms) : 1871, 1930
. : milestone, 1900,
profiling (1.923 ms) : 1890, 1957
. : milestone, 1923,
tracing (1.875 ms) : 1844, 1907
. : milestone, 1875,
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.35.0-SNAPSHOT~096483556f, baseline=1.35.0-SNAPSHOT~4b5cad9604
dateFormat X
axisFormat %s
section baseline
no_agent (451.127 µs) : 422, 480
. : milestone, 451,
iast (588.027 µs) : 556, 620
. : milestone, 588,
iast_FULL (691.802 µs) : 660, 723
. : milestone, 692,
iast_GLOBAL (619.536 µs) : 588, 652
. : milestone, 620,
iast_HARDCODED_SECRET_DISABLED (584.296 µs) : 552, 616
. : milestone, 584,
iast_INACTIVE (558.429 µs) : 528, 589
. : milestone, 558,
iast_TELEMETRY_OFF (571.797 µs) : 539, 605
. : milestone, 572,
tracing (538.322 µs) : 508, 569
. : milestone, 538,
section candidate
no_agent (450.779 µs) : 422, 480
. : milestone, 451,
iast (589.476 µs) : 558, 621
. : milestone, 589,
iast_FULL (682.486 µs) : 651, 714
. : milestone, 682,
iast_GLOBAL (617.564 µs) : 587, 649
. : milestone, 618,
iast_HARDCODED_SECRET_DISABLED (590.873 µs) : 557, 625
. : milestone, 591,
iast_INACTIVE (553.463 µs) : 522, 585
. : milestone, 553,
iast_TELEMETRY_OFF (578.652 µs) : 548, 609
. : milestone, 579,
tracing (539.306 µs) : 509, 569
. : milestone, 539,
DacapoParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 12 metrics, 0 unstable metrics. Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.35.0-SNAPSHOT~096483556f, baseline=1.35.0-SNAPSHOT~4b5cad9604
dateFormat X
axisFormat %s
section baseline
no_agent (1.47 ms) : 1459, 1482
. : milestone, 1470,
appsec (2.2 ms) : 2166, 2233
. : milestone, 2200,
iast (1.957 ms) : 1916, 1997
. : milestone, 1957,
iast_GLOBAL (1.981 ms) : 1941, 2021
. : milestone, 1981,
profiling (1.85 ms) : 1818, 1883
. : milestone, 1850,
tracing (1.828 ms) : 1796, 1861
. : milestone, 1828,
section candidate
no_agent (1.46 ms) : 1448, 1471
. : milestone, 1460,
appsec (2.204 ms) : 2170, 2238
. : milestone, 2204,
iast (1.957 ms) : 1916, 1997
. : milestone, 1957,
iast_GLOBAL (2.003 ms) : 1963, 2044
. : milestone, 2003,
profiling (1.85 ms) : 1817, 1884
. : milestone, 1850,
tracing (1.828 ms) : 1796, 1860
. : milestone, 1828,
Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.35.0-SNAPSHOT~096483556f, baseline=1.35.0-SNAPSHOT~4b5cad9604
dateFormat X
axisFormat %s
section baseline
no_agent (14.998 s) : 14998000, 14998000
. : milestone, 14998000,
appsec (15.319 s) : 15319000, 15319000
. : milestone, 15319000,
iast (19.025 s) : 19025000, 19025000
. : milestone, 19025000,
iast_GLOBAL (18.092 s) : 18092000, 18092000
. : milestone, 18092000,
profiling (15.809 s) : 15809000, 15809000
. : milestone, 15809000,
tracing (15.102 s) : 15102000, 15102000
. : milestone, 15102000,
section candidate
no_agent (15.042 s) : 15042000, 15042000
. : milestone, 15042000,
appsec (15.037 s) : 15037000, 15037000
. : milestone, 15037000,
iast (18.543 s) : 18543000, 18543000
. : milestone, 18543000,
iast_GLOBAL (17.997 s) : 17997000, 17997000
. : milestone, 17997000,
profiling (15.899 s) : 15899000, 15899000
. : milestone, 15899000,
tracing (14.979 s) : 14979000, 14979000
. : milestone, 14979000,
|
678a09e to
0964835
Compare
jpbempel
approved these changes
May 30, 2024
dougqh
approved these changes
May 30, 2024
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.
What Does This Do
Avoids merging in any additional host/port settings when dealing with
unix:URLs.If the user has configured a
unix:agent URL that means they want the tracer to communicate over the given UDS file, in which case we shouldn't then check for any additional agent host or agent port settings, because they won't be applicable. Instead it should just use the default host/port and not rebuild the agent URL.This does mean that consumers of the agent URL should handle that it might now start with
unix:- in practice this only affects one place,SharedCommunicationObjectswhich now uses a placeholderhttp://{agent.host}:{agent.port}forunix:URLs. Note this is just to satisfy the OkHttp API, the actual communication will be tunnelled over UDS.The other updates were to a couple of products still using the raw agent URL - they now use
SharedCommunicationObjects/DDAgentFeaturesDiscoveryto get the parsed agent URL for use with OkHttp.Motivation
Merging in additional host/port settings triggers a rebuild of the agent URL in
Config, which is misleading because the actual agent URL is the original one that starts withunix:.