Make cws-tls use the same JNA dependency as instrumentations#7412
Merged
bantonsson merged 1 commit intomasterfrom Aug 14, 2024
Merged
Make cws-tls use the same JNA dependency as instrumentations#7412bantonsson merged 1 commit intomasterfrom
bantonsson merged 1 commit intomasterfrom
Conversation
65186e7 to
729a2cb
Compare
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 49 metrics, 14 unstable metrics. Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.39.0-SNAPSHOT~048905fbf0, baseline=1.39.0-SNAPSHOT~4c58087c38
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.05 s) : 0, 1050270
Total [baseline] (10.279 s) : 0, 10278896
Agent [candidate] (1.049 s) : 0, 1048760
Total [candidate] (10.337 s) : 0, 10337429
section appsec
Agent [baseline] (1.172 s) : 0, 1171624
Total [baseline] (10.45 s) : 0, 10449794
Agent [candidate] (1.172 s) : 0, 1172039
Total [candidate] (10.477 s) : 0, 10476687
section iast
Agent [baseline] (1.185 s) : 0, 1185124
Total [baseline] (10.868 s) : 0, 10868151
Agent [candidate] (1.18 s) : 0, 1180035
Total [candidate] (10.842 s) : 0, 10842439
section profiling
Agent [baseline] (1.257 s) : 0, 1257261
Total [baseline] (10.592 s) : 0, 10592318
Agent [candidate] (1.257 s) : 0, 1256820
Total [candidate] (10.664 s) : 0, 10663786
gantt
title petclinic - break down per module: candidate=1.39.0-SNAPSHOT~048905fbf0, baseline=1.39.0-SNAPSHOT~4c58087c38
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (669.73 ms) : 0, 669730
BytebuddyAgent [candidate] (668.729 ms) : 0, 668729
GlobalTracer [baseline] (307.558 ms) : 0, 307558
GlobalTracer [candidate] (307.267 ms) : 0, 307267
AppSec [baseline] (51.31 ms) : 0, 51310
AppSec [candidate] (51.151 ms) : 0, 51151
Remote Config [baseline] (687.048 µs) : 0, 687
Remote Config [candidate] (684.556 µs) : 0, 685
Telemetry [baseline] (7.489 ms) : 0, 7489
Telemetry [candidate] (7.476 ms) : 0, 7476
section appsec
BytebuddyAgent [baseline] (679.256 ms) : 0, 679256
BytebuddyAgent [candidate] (680.276 ms) : 0, 680276
GlobalTracer [baseline] (300.596 ms) : 0, 300596
GlobalTracer [candidate] (301.24 ms) : 0, 301240
AppSec [baseline] (157.344 ms) : 0, 157344
AppSec [candidate] (157.88 ms) : 0, 157880
Remote Config [baseline] (602.06 µs) : 0, 602
Remote Config [candidate] (606.906 µs) : 0, 607
Telemetry [baseline] (9.775 ms) : 0, 9775
Telemetry [candidate] (8.068 ms) : 0, 8068
IAST [baseline] (21.025 ms) : 0, 21025
IAST [candidate] (20.82 ms) : 0, 20820
section iast
BytebuddyAgent [baseline] (789.679 ms) : 0, 789679
BytebuddyAgent [candidate] (784.44 ms) : 0, 784440
GlobalTracer [baseline] (296.906 ms) : 0, 296906
GlobalTracer [candidate] (297.572 ms) : 0, 297572
AppSec [baseline] (52.202 ms) : 0, 52202
AppSec [candidate] (53.491 ms) : 0, 53491
Remote Config [baseline] (617.994 µs) : 0, 618
Remote Config [candidate] (586.977 µs) : 0, 587
Telemetry [baseline] (7.918 ms) : 0, 7918
Telemetry [candidate] (7.983 ms) : 0, 7983
IAST [baseline] (24.182 ms) : 0, 24182
IAST [candidate] (22.458 ms) : 0, 22458
section profiling
BytebuddyAgent [baseline] (669.965 ms) : 0, 669965
BytebuddyAgent [candidate] (669.21 ms) : 0, 669210
GlobalTracer [baseline] (394.439 ms) : 0, 394439
GlobalTracer [candidate] (393.646 ms) : 0, 393646
AppSec [baseline] (52.049 ms) : 0, 52049
AppSec [candidate] (52.765 ms) : 0, 52765
Remote Config [baseline] (698.933 µs) : 0, 699
Remote Config [candidate] (687.6 µs) : 0, 688
Telemetry [baseline] (7.433 ms) : 0, 7433
Telemetry [candidate] (7.502 ms) : 0, 7502
ProfilingAgent [baseline] (95.124 ms) : 0, 95124
ProfilingAgent [candidate] (95.549 ms) : 0, 95549
Profiling [baseline] (95.149 ms) : 0, 95149
Profiling [candidate] (95.574 ms) : 0, 95574
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.39.0-SNAPSHOT~048905fbf0, baseline=1.39.0-SNAPSHOT~4c58087c38
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.048 s) : 0, 1047803
Total [baseline] (8.484 s) : 0, 8484311
Agent [candidate] (1.045 s) : 0, 1045147
Total [candidate] (8.475 s) : 0, 8474662
section iast
Agent [baseline] (1.178 s) : 0, 1177591
Total [baseline] (9.026 s) : 0, 9026030
Agent [candidate] (1.186 s) : 0, 1185517
Total [candidate] (8.986 s) : 0, 8985580
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.176 s) : 0, 1175931
Total [baseline] (9.013 s) : 0, 9012834
Agent [candidate] (1.176 s) : 0, 1176341
Total [candidate] (8.991 s) : 0, 8991146
section iast_TELEMETRY_OFF
Agent [baseline] (1.18 s) : 0, 1180108
Total [baseline] (9.016 s) : 0, 9015679
Agent [candidate] (1.184 s) : 0, 1183873
Total [candidate] (8.995 s) : 0, 8994866
gantt
title insecure-bank - break down per module: candidate=1.39.0-SNAPSHOT~048905fbf0, baseline=1.39.0-SNAPSHOT~4c58087c38
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (668.451 ms) : 0, 668451
BytebuddyAgent [candidate] (666.721 ms) : 0, 666721
GlobalTracer [baseline] (306.672 ms) : 0, 306672
GlobalTracer [candidate] (305.745 ms) : 0, 305745
AppSec [baseline] (50.948 ms) : 0, 50948
AppSec [candidate] (51.066 ms) : 0, 51066
Remote Config [baseline] (688.619 µs) : 0, 689
Remote Config [candidate] (683.676 µs) : 0, 684
Telemetry [baseline] (7.612 ms) : 0, 7612
Telemetry [candidate] (7.492 ms) : 0, 7492
section iast
BytebuddyAgent [baseline] (783.777 ms) : 0, 783777
BytebuddyAgent [candidate] (789.164 ms) : 0, 789164
GlobalTracer [baseline] (295.863 ms) : 0, 295863
GlobalTracer [candidate] (298.628 ms) : 0, 298628
AppSec [baseline] (52.479 ms) : 0, 52479
AppSec [candidate] (50.532 ms) : 0, 50532
IAST [baseline] (24.147 ms) : 0, 24147
IAST [candidate] (24.047 ms) : 0, 24047
Remote Config [baseline] (588.87 µs) : 0, 589
Remote Config [candidate] (605.608 µs) : 0, 606
Telemetry [baseline] (7.159 ms) : 0, 7159
Telemetry [candidate] (8.909 ms) : 0, 8909
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (782.189 ms) : 0, 782189
BytebuddyAgent [candidate] (782.312 ms) : 0, 782312
GlobalTracer [baseline] (296.6 ms) : 0, 296600
GlobalTracer [candidate] (297.381 ms) : 0, 297381
AppSec [baseline] (47.966 ms) : 0, 47966
AppSec [candidate] (48.601 ms) : 0, 48601
IAST [baseline] (25.474 ms) : 0, 25474
IAST [candidate] (25.134 ms) : 0, 25134
Remote Config [baseline] (595.449 µs) : 0, 595
Remote Config [candidate] (581.421 µs) : 0, 581
Telemetry [baseline] (9.549 ms) : 0, 9549
Telemetry [candidate] (8.758 ms) : 0, 8758
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (784.479 ms) : 0, 784479
BytebuddyAgent [candidate] (787.444 ms) : 0, 787444
GlobalTracer [baseline] (298.148 ms) : 0, 298148
GlobalTracer [candidate] (299.469 ms) : 0, 299469
AppSec [baseline] (52.974 ms) : 0, 52974
AppSec [candidate] (52.603 ms) : 0, 52603
IAST [baseline] (23.189 ms) : 0, 23189
IAST [candidate] (23.014 ms) : 0, 23014
Remote Config [baseline] (616.998 µs) : 0, 617
Remote Config [candidate] (616.314 µs) : 0, 616
Telemetry [baseline] (7.11 ms) : 0, 7110
Telemetry [candidate] (7.055 ms) : 0, 7055
LoadParameters
See matching parameters
SummaryFound 0 performance improvements and 1 performance regressions! Performance is the same for 10 metrics, 17 unstable metrics.
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.39.0-SNAPSHOT~048905fbf0, baseline=1.39.0-SNAPSHOT~4c58087c38
dateFormat X
axisFormat %s
section baseline
no_agent (1.347 ms) : 1328, 1367
. : milestone, 1347,
appsec (1.716 ms) : 1692, 1741
. : milestone, 1716,
appsec_no_iast (1.71 ms) : 1685, 1735
. : milestone, 1710,
iast (1.469 ms) : 1446, 1492
. : milestone, 1469,
profiling (1.485 ms) : 1460, 1509
. : milestone, 1485,
tracing (1.481 ms) : 1457, 1505
. : milestone, 1481,
section candidate
no_agent (1.333 ms) : 1314, 1352
. : milestone, 1333,
appsec (1.723 ms) : 1699, 1747
. : milestone, 1723,
appsec_no_iast (1.714 ms) : 1689, 1738
. : milestone, 1714,
iast (1.483 ms) : 1461, 1505
. : milestone, 1483,
profiling (1.547 ms) : 1514, 1581
. : milestone, 1547,
tracing (1.46 ms) : 1436, 1485
. : milestone, 1460,
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.39.0-SNAPSHOT~048905fbf0, baseline=1.39.0-SNAPSHOT~4c58087c38
dateFormat X
axisFormat %s
section baseline
no_agent (368.143 µs) : 349, 388
. : milestone, 368,
iast (481.981 µs) : 461, 503
. : milestone, 482,
iast_FULL (545.812 µs) : 523, 568
. : milestone, 546,
iast_GLOBAL (506.904 µs) : 484, 529
. : milestone, 507,
iast_HARDCODED_SECRET_DISABLED (474.817 µs) : 454, 496
. : milestone, 475,
iast_INACTIVE (447.318 µs) : 425, 469
. : milestone, 447,
iast_TELEMETRY_OFF (473.586 µs) : 452, 495
. : milestone, 474,
tracing (434.992 µs) : 415, 455
. : milestone, 435,
section candidate
no_agent (366.036 µs) : 345, 387
. : milestone, 366,
iast (487.572 µs) : 465, 510
. : milestone, 488,
iast_FULL (551.713 µs) : 529, 574
. : milestone, 552,
iast_GLOBAL (504.704 µs) : 482, 527
. : milestone, 505,
iast_HARDCODED_SECRET_DISABLED (480.74 µs) : 460, 502
. : milestone, 481,
iast_INACTIVE (457.234 µs) : 434, 480
. : milestone, 457,
iast_TELEMETRY_OFF (468.002 µs) : 447, 489
. : milestone, 468,
tracing (434.081 µs) : 414, 454
. : milestone, 434,
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.39.0-SNAPSHOT~048905fbf0, baseline=1.39.0-SNAPSHOT~4c58087c38
dateFormat X
axisFormat %s
section baseline
no_agent (15.008 s) : 15008000, 15008000
. : milestone, 15008000,
appsec (15.116 s) : 15116000, 15116000
. : milestone, 15116000,
iast (18.747 s) : 18747000, 18747000
. : milestone, 18747000,
iast_GLOBAL (17.947 s) : 17947000, 17947000
. : milestone, 17947000,
profiling (15.169 s) : 15169000, 15169000
. : milestone, 15169000,
tracing (14.961 s) : 14961000, 14961000
. : milestone, 14961000,
section candidate
no_agent (15.502 s) : 15502000, 15502000
. : milestone, 15502000,
appsec (15.273 s) : 15273000, 15273000
. : milestone, 15273000,
iast (18.588 s) : 18588000, 18588000
. : milestone, 18588000,
iast_GLOBAL (17.82 s) : 17820000, 17820000
. : milestone, 17820000,
profiling (15.19 s) : 15190000, 15190000
. : milestone, 15190000,
tracing (15.125 s) : 15125000, 15125000
. : milestone, 15125000,
Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.39.0-SNAPSHOT~048905fbf0, baseline=1.39.0-SNAPSHOT~4c58087c38
dateFormat X
axisFormat %s
section baseline
no_agent (1.462 ms) : 1451, 1474
. : milestone, 1462,
appsec (2.213 ms) : 2178, 2247
. : milestone, 2213,
iast (1.951 ms) : 1910, 1992
. : milestone, 1951,
iast_GLOBAL (2.011 ms) : 1969, 2054
. : milestone, 2011,
profiling (2.344 ms) : 2160, 2529
. : milestone, 2344,
tracing (1.84 ms) : 1807, 1873
. : milestone, 1840,
section candidate
no_agent (1.455 ms) : 1444, 1467
. : milestone, 1455,
appsec (2.222 ms) : 2187, 2257
. : milestone, 2222,
iast (1.957 ms) : 1916, 1998
. : milestone, 1957,
iast_GLOBAL (2.022 ms) : 1979, 2066
. : milestone, 2022,
profiling (1.862 ms) : 1828, 1896
. : milestone, 1862,
tracing (1.83 ms) : 1797, 1862
. : milestone, 1830,
|
729a2cb to
cc29543
Compare
cc29543 to
048905f
Compare
PerfectSlayer
approved these changes
Aug 14, 2024
Contributor
PerfectSlayer
left a comment
There was a problem hiding this comment.
Looks good. Are you aware of any down side enabling cws-tls by default?
Contributor
Author
|
@PerfectSlayer It's only included in the jar file. It's still opt-in to enable. |
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
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
Makes the
cws-tlsmodule use the same JNA dependency as the instrumentations in the final packaged jar, and includes it by default.Motivation
The
cws-tlsmodule was not included by default since it was under development and added significantly to the overall jar size. The JNA dependency used has since been included by other instrumentations and thecws-tlsmodule can now be included.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]