-
Notifications
You must be signed in to change notification settings - Fork 323
Rework the listeners for AMS_FEATURES #7158
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
manuel-alvarez-alvarez
merged 2 commits into
master
from
malvarez/rc-remove-keys-from-listener
Jun 17, 2024
Merged
Rework the listeners for AMS_FEATURES #7158
manuel-alvarez-alvarez
merged 2 commits into
master
from
malvarez/rc-remove-keys-from-listener
Jun 17, 2024
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 55 metrics, 16 unstable metrics. Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.36.0-SNAPSHOT~a23b0a8e7e, baseline=1.36.0-SNAPSHOT~33a2dcd37c
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.064 s) : 0, 1063901
Total [baseline] (8.553 s) : 0, 8553405
Agent [candidate] (1.064 s) : 0, 1064120
Total [candidate] (8.554 s) : 0, 8553576
section iast
Agent [baseline] (1.172 s) : 0, 1172145
Total [baseline] (9.003 s) : 0, 9002633
Agent [candidate] (1.171 s) : 0, 1171176
Total [candidate] (9.039 s) : 0, 9039124
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.18 s) : 0, 1179952
Total [baseline] (9.022 s) : 0, 9022071
Agent [candidate] (1.172 s) : 0, 1171799
Total [candidate] (8.994 s) : 0, 8994316
section iast_TELEMETRY_OFF
Agent [baseline] (1.166 s) : 0, 1165794
Total [baseline] (8.99 s) : 0, 8990162
Agent [candidate] (1.176 s) : 0, 1176296
Total [candidate] (9.014 s) : 0, 9014438
gantt
title insecure-bank - break down per module: candidate=1.36.0-SNAPSHOT~a23b0a8e7e, baseline=1.36.0-SNAPSHOT~33a2dcd37c
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (666.313 ms) : 0, 666313
BytebuddyAgent [candidate] (666.659 ms) : 0, 666659
GlobalTracer [baseline] (304.11 ms) : 0, 304110
GlobalTracer [candidate] (304.336 ms) : 0, 304336
AppSec [baseline] (50.438 ms) : 0, 50438
AppSec [candidate] (50.098 ms) : 0, 50098
Logs Intake [baseline] (335.816 µs) : 0, 336
Logs Intake [candidate] (332.615 µs) : 0, 333
Remote Config [baseline] (676.187 µs) : 0, 676
Remote Config [candidate] (674.116 µs) : 0, 674
Telemetry [baseline] (7.544 ms) : 0, 7544
Telemetry [candidate] (7.533 ms) : 0, 7533
section iast
BytebuddyAgent [baseline] (781.252 ms) : 0, 781252
BytebuddyAgent [candidate] (780.835 ms) : 0, 780835
GlobalTracer [baseline] (293.336 ms) : 0, 293336
GlobalTracer [candidate] (293.741 ms) : 0, 293741
AppSec [baseline] (47.376 ms) : 0, 47376
AppSec [candidate] (47.085 ms) : 0, 47085
Logs Intake [baseline] (307.924 µs) : 0, 308
Logs Intake [candidate] (1.071 ms) : 0, 1071
Remote Config [baseline] (621.388 µs) : 0, 621
Remote Config [candidate] (1.41 ms) : 0, 1410
Telemetry [baseline] (6.908 ms) : 0, 6908
Telemetry [candidate] (6.987 ms) : 0, 6987
IAST [baseline] (29.006 ms) : 0, 29006
IAST [candidate] (26.734 ms) : 0, 26734
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (787.171 ms) : 0, 787171
BytebuddyAgent [candidate] (781.56 ms) : 0, 781560
GlobalTracer [baseline] (295.866 ms) : 0, 295866
GlobalTracer [candidate] (294.523 ms) : 0, 294523
AppSec [baseline] (47.904 ms) : 0, 47904
AppSec [candidate] (47.597 ms) : 0, 47597
Logs Intake [baseline] (306.772 µs) : 0, 307
Logs Intake [candidate] (304.524 µs) : 0, 305
Remote Config [baseline] (1.359 ms) : 0, 1359
Remote Config [candidate] (585.43 µs) : 0, 585
Telemetry [baseline] (7.121 ms) : 0, 7121
Telemetry [candidate] (7.76 ms) : 0, 7760
IAST [baseline] (26.755 ms) : 0, 26755
IAST [candidate] (26.074 ms) : 0, 26074
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (778.128 ms) : 0, 778128
BytebuddyAgent [candidate] (784.35 ms) : 0, 784350
GlobalTracer [baseline] (292.77 ms) : 0, 292770
GlobalTracer [candidate] (295.864 ms) : 0, 295864
AppSec [baseline] (47.303 ms) : 0, 47303
AppSec [candidate] (47.721 ms) : 0, 47721
Logs Intake [baseline] (303.038 µs) : 0, 303
Logs Intake [candidate] (307.601 µs) : 0, 308
Remote Config [baseline] (613.149 µs) : 0, 613
Remote Config [candidate] (597.523 µs) : 0, 598
Telemetry [baseline] (7.778 ms) : 0, 7778
Telemetry [candidate] (6.97 ms) : 0, 6970
IAST [baseline] (25.498 ms) : 0, 25498
IAST [candidate] (27.057 ms) : 0, 27057
Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.36.0-SNAPSHOT~a23b0a8e7e, baseline=1.36.0-SNAPSHOT~33a2dcd37c
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.064 s) : 0, 1063769
Total [baseline] (10.418 s) : 0, 10418240
Agent [candidate] (1.067 s) : 0, 1066608
Total [candidate] (10.387 s) : 0, 10386578
section appsec
Agent [baseline] (1.182 s) : 0, 1181868
Total [baseline] (10.497 s) : 0, 10497059
Agent [candidate] (1.185 s) : 0, 1184574
Total [candidate] (10.562 s) : 0, 10561915
section iast
Agent [baseline] (1.184 s) : 0, 1183824
Total [baseline] (10.733 s) : 0, 10733207
Agent [candidate] (1.174 s) : 0, 1174116
Total [candidate] (10.714 s) : 0, 10713560
section profiling
Agent [baseline] (1.275 s) : 0, 1274809
Total [baseline] (10.674 s) : 0, 10673586
Agent [candidate] (1.262 s) : 0, 1262068
Total [candidate] (10.628 s) : 0, 10627885
gantt
title petclinic - break down per module: candidate=1.36.0-SNAPSHOT~a23b0a8e7e, baseline=1.36.0-SNAPSHOT~33a2dcd37c
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (666.28 ms) : 0, 666280
BytebuddyAgent [candidate] (668.06 ms) : 0, 668060
GlobalTracer [baseline] (304.111 ms) : 0, 304111
GlobalTracer [candidate] (304.889 ms) : 0, 304889
AppSec [baseline] (50.424 ms) : 0, 50424
AppSec [candidate] (50.507 ms) : 0, 50507
Logs Intake [baseline] (334.775 µs) : 0, 335
Logs Intake [candidate] (333.295 µs) : 0, 333
Remote Config [baseline] (673.165 µs) : 0, 673
Remote Config [candidate] (672.571 µs) : 0, 673
Telemetry [baseline] (7.604 ms) : 0, 7604
Telemetry [candidate] (7.681 ms) : 0, 7681
section appsec
BytebuddyAgent [baseline] (676.133 ms) : 0, 676133
BytebuddyAgent [candidate] (676.91 ms) : 0, 676910
GlobalTracer [baseline] (297.38 ms) : 0, 297380
GlobalTracer [candidate] (297.73 ms) : 0, 297730
AppSec [baseline] (154.187 ms) : 0, 154187
AppSec [candidate] (154.221 ms) : 0, 154221
Logs Intake [baseline] (325.191 µs) : 0, 325
Logs Intake [candidate] (337.176 µs) : 0, 337
Remote Config [baseline] (620.258 µs) : 0, 620
Remote Config [candidate] (625.3 µs) : 0, 625
Telemetry [baseline] (9.222 ms) : 0, 9222
Telemetry [candidate] (9.503 ms) : 0, 9503
IAST [baseline] (19.361 ms) : 0, 19361
IAST [candidate] (21.641 ms) : 0, 21641
section iast
BytebuddyAgent [baseline] (788.432 ms) : 0, 788432
BytebuddyAgent [candidate] (782.496 ms) : 0, 782496
GlobalTracer [baseline] (296.503 ms) : 0, 296503
GlobalTracer [candidate] (294.952 ms) : 0, 294952
AppSec [baseline] (48.023 ms) : 0, 48023
AppSec [candidate] (47.385 ms) : 0, 47385
Logs Intake [baseline] (3.231 ms) : 0, 3231
Logs Intake [candidate] (316.862 µs) : 0, 317
Remote Config [baseline] (825.832 µs) : 0, 826
Remote Config [candidate] (2.132 ms) : 0, 2132
Telemetry [baseline] (7.858 ms) : 0, 7858
Telemetry [candidate] (6.973 ms) : 0, 6973
IAST [baseline] (25.534 ms) : 0, 25534
IAST [candidate] (26.544 ms) : 0, 26544
section profiling
BytebuddyAgent [baseline] (669.98 ms) : 0, 669980
BytebuddyAgent [candidate] (662.583 ms) : 0, 662583
GlobalTracer [baseline] (389.614 ms) : 0, 389614
GlobalTracer [candidate] (386.221 ms) : 0, 386221
AppSec [baseline] (51.913 ms) : 0, 51913
AppSec [candidate] (51.313 ms) : 0, 51313
Logs Intake [baseline] (345.198 µs) : 0, 345
Logs Intake [candidate] (335.463 µs) : 0, 335
Remote Config [baseline] (705.494 µs) : 0, 705
Remote Config [candidate] (716.96 µs) : 0, 717
Telemetry [baseline] (7.383 ms) : 0, 7383
Telemetry [candidate] (7.365 ms) : 0, 7365
ProfilingAgent [baseline] (97.241 ms) : 0, 97241
ProfilingAgent [candidate] (96.592 ms) : 0, 96592
Profiling [baseline] (97.267 ms) : 0, 97267
Profiling [candidate] (96.617 ms) : 0, 96617
LoadParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 11 metrics, 17 unstable metrics. Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.36.0-SNAPSHOT~a23b0a8e7e, baseline=1.36.0-SNAPSHOT~33a2dcd37c
dateFormat X
axisFormat %s
section baseline
no_agent (1.355 ms) : 1336, 1374
. : milestone, 1355,
appsec (1.721 ms) : 1697, 1746
. : milestone, 1721,
appsec_no_iast (1.712 ms) : 1686, 1737
. : milestone, 1712,
iast (1.475 ms) : 1453, 1498
. : milestone, 1475,
profiling (1.518 ms) : 1493, 1544
. : milestone, 1518,
tracing (1.448 ms) : 1424, 1473
. : milestone, 1448,
section candidate
no_agent (1.34 ms) : 1321, 1359
. : milestone, 1340,
appsec (1.728 ms) : 1705, 1751
. : milestone, 1728,
appsec_no_iast (1.703 ms) : 1677, 1728
. : milestone, 1703,
iast (1.476 ms) : 1453, 1498
. : milestone, 1476,
profiling (1.475 ms) : 1450, 1499
. : milestone, 1475,
tracing (1.452 ms) : 1427, 1476
. : milestone, 1452,
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.36.0-SNAPSHOT~a23b0a8e7e, baseline=1.36.0-SNAPSHOT~33a2dcd37c
dateFormat X
axisFormat %s
section baseline
no_agent (367.42 µs) : 348, 387
. : milestone, 367,
iast (478.605 µs) : 458, 499
. : milestone, 479,
iast_FULL (552.189 µs) : 531, 574
. : milestone, 552,
iast_GLOBAL (500.537 µs) : 479, 522
. : milestone, 501,
iast_HARDCODED_SECRET_DISABLED (486.273 µs) : 465, 508
. : milestone, 486,
iast_INACTIVE (453.139 µs) : 432, 474
. : milestone, 453,
iast_TELEMETRY_OFF (473.892 µs) : 452, 496
. : milestone, 474,
tracing (441.246 µs) : 421, 462
. : milestone, 441,
section candidate
no_agent (371.968 µs) : 352, 391
. : milestone, 372,
iast (474.983 µs) : 454, 496
. : milestone, 475,
iast_FULL (552.113 µs) : 531, 573
. : milestone, 552,
iast_GLOBAL (507.048 µs) : 486, 528
. : milestone, 507,
iast_HARDCODED_SECRET_DISABLED (482.488 µs) : 462, 503
. : milestone, 482,
iast_INACTIVE (454.324 µs) : 433, 476
. : milestone, 454,
iast_TELEMETRY_OFF (472.777 µs) : 451, 495
. : milestone, 473,
tracing (447.791 µs) : 427, 469
. : milestone, 448,
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.36.0-SNAPSHOT~a23b0a8e7e, baseline=1.36.0-SNAPSHOT~33a2dcd37c
dateFormat X
axisFormat %s
section baseline
no_agent (1.476 ms) : 1465, 1488
. : milestone, 1476,
appsec (2.215 ms) : 2181, 2249
. : milestone, 2215,
iast (1.979 ms) : 1938, 2021
. : milestone, 1979,
iast_GLOBAL (2.029 ms) : 1987, 2071
. : milestone, 2029,
profiling (1.885 ms) : 1850, 1920
. : milestone, 1885,
tracing (1.851 ms) : 1819, 1883
. : milestone, 1851,
section candidate
no_agent (1.475 ms) : 1464, 1487
. : milestone, 1475,
appsec (2.227 ms) : 2193, 2261
. : milestone, 2227,
iast (1.972 ms) : 1931, 2014
. : milestone, 1972,
iast_GLOBAL (2.015 ms) : 1973, 2056
. : milestone, 2015,
profiling (1.871 ms) : 1836, 1905
. : milestone, 1871,
tracing (1.847 ms) : 1815, 1879
. : milestone, 1847,
Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.36.0-SNAPSHOT~a23b0a8e7e, baseline=1.36.0-SNAPSHOT~33a2dcd37c
dateFormat X
axisFormat %s
section baseline
no_agent (15.549 s) : 15549000, 15549000
. : milestone, 15549000,
appsec (14.913 s) : 14913000, 14913000
. : milestone, 14913000,
iast (18.928 s) : 18928000, 18928000
. : milestone, 18928000,
iast_GLOBAL (17.776 s) : 17776000, 17776000
. : milestone, 17776000,
profiling (15.43 s) : 15430000, 15430000
. : milestone, 15430000,
tracing (14.849 s) : 14849000, 14849000
. : milestone, 14849000,
section candidate
no_agent (15.039 s) : 15039000, 15039000
. : milestone, 15039000,
appsec (15.091 s) : 15091000, 15091000
. : milestone, 15091000,
iast (19.009 s) : 19009000, 19009000
. : milestone, 19009000,
iast_GLOBAL (17.947 s) : 17947000, 17947000
. : milestone, 17947000,
profiling (15.756 s) : 15756000, 15756000
. : milestone, 15756000,
tracing (15.159 s) : 15159000, 15159000
. : milestone, 15159000,
|
fc777a1 to
7808bc0
Compare
jandro996
approved these changes
Jun 12, 2024
f117d08 to
5b2fe0d
Compare
49f9e5e to
db762ac
Compare
ValentinZakharov
approved these changes
Jun 14, 2024
195718c to
a23b0a8
Compare
a23b0a8 to
7be63f5
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
comp: asm waf
Application Security Management (WAF)
tag: no release notes
Changes to exclude from release notes
type: refactoring
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
Move remote configuration config code away from
ApiSecurityRequestSamplerand ensure we are dealing with the removal of configuration files correctly.Motivation
Additional Notes