-
Notifications
You must be signed in to change notification settings - Fork 303
Enforce size limit on application_monitoring.yaml files #8789
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
Draft
mtoffl01
wants to merge
26
commits into
master
Choose a base branch
from
mtoff/scfg_file_size_limit
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+59
−1
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
…values in template variable
BenchmarksStartupParameters
See matching parameters
SummaryFound 3 performance improvements and 1 performance regressions! Performance is the same for 51 metrics, 16 unstable metrics.
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.50.0-SNAPSHOT~0f77a40332, baseline=1.50.0-SNAPSHOT~ad6d5fef42
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.02 s) : 0, 1020017
Total [baseline] (8.659 s) : 0, 8659001
Agent [candidate] (1.02 s) : 0, 1019769
Total [candidate] (8.674 s) : 0, 8673967
section iast
Agent [baseline] (1.157 s) : 0, 1157041
Total [baseline] (9.284 s) : 0, 9283544
Agent [candidate] (1.16 s) : 0, 1160129
Total [candidate] (9.253 s) : 0, 9252893
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.158 s) : 0, 1157562
Total [baseline] (9.217 s) : 0, 9216712
Agent [candidate] (1.151 s) : 0, 1150933
Total [candidate] (9.195 s) : 0, 9195444
section iast_TELEMETRY_OFF
Agent [baseline] (1.146 s) : 0, 1145959
Total [baseline] (9.244 s) : 0, 9244364
Agent [candidate] (1.147 s) : 0, 1147165
Total [candidate] (9.169 s) : 0, 9168639
gantt
title insecure-bank - break down per module: candidate=1.50.0-SNAPSHOT~0f77a40332, baseline=1.50.0-SNAPSHOT~ad6d5fef42
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (682.156 ms) : 0, 682156
BytebuddyAgent [candidate] (682.676 ms) : 0, 682676
GlobalTracer [baseline] (239.642 ms) : 0, 239642
GlobalTracer [candidate] (240.651 ms) : 0, 240651
AppSec [baseline] (54.552 ms) : 0, 54552
AppSec [candidate] (54.665 ms) : 0, 54665
Debugger [baseline] (9.027 ms) : 0, 9027
Debugger [candidate] (7.633 ms) : 0, 7633
Remote Config [baseline] (714.72 µs) : 0, 715
Remote Config [candidate] (714.973 µs) : 0, 715
Telemetry [baseline] (10.481 ms) : 0, 10481
Telemetry [candidate] (9.943 ms) : 0, 9943
section iast
BytebuddyAgent [baseline] (806.901 ms) : 0, 806901
BytebuddyAgent [candidate] (810.209 ms) : 0, 810209
GlobalTracer [baseline] (232.456 ms) : 0, 232456
GlobalTracer [candidate] (232.463 ms) : 0, 232463
AppSec [baseline] (50.669 ms) : 0, 50669
AppSec [candidate] (48.95 ms) : 0, 48950
Debugger [baseline] (5.974 ms) : 0, 5974
Debugger [candidate] (6.032 ms) : 0, 6032
Remote Config [baseline] (611.265 µs) : 0, 611
Remote Config [candidate] (611.966 µs) : 0, 612
Telemetry [baseline] (7.98 ms) : 0, 7980
Telemetry [candidate] (7.998 ms) : 0, 7998
IAST [baseline] (28.848 ms) : 0, 28848
IAST [candidate] (30.131 ms) : 0, 30131
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (808.637 ms) : 0, 808637
BytebuddyAgent [candidate] (802.772 ms) : 0, 802772
GlobalTracer [baseline] (231.873 ms) : 0, 231873
GlobalTracer [candidate] (231.299 ms) : 0, 231299
AppSec [baseline] (47.821 ms) : 0, 47821
AppSec [candidate] (50.439 ms) : 0, 50439
Debugger [baseline] (5.969 ms) : 0, 5969
Debugger [candidate] (6.005 ms) : 0, 6005
Remote Config [baseline] (615.944 µs) : 0, 616
Remote Config [candidate] (622.107 µs) : 0, 622
Telemetry [baseline] (7.96 ms) : 0, 7960
Telemetry [candidate] (7.915 ms) : 0, 7915
IAST [baseline] (31.053 ms) : 0, 31053
IAST [candidate] (28.31 ms) : 0, 28310
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (799.008 ms) : 0, 799008
BytebuddyAgent [candidate] (798.033 ms) : 0, 798033
GlobalTracer [baseline] (230.678 ms) : 0, 230678
GlobalTracer [candidate] (232.265 ms) : 0, 232265
AppSec [baseline] (54.41 ms) : 0, 54410
AppSec [candidate] (56.221 ms) : 0, 56221
Debugger [baseline] (5.98 ms) : 0, 5980
Debugger [candidate] (5.97 ms) : 0, 5970
Remote Config [baseline] (600.846 µs) : 0, 601
Remote Config [candidate] (610.54 µs) : 0, 611
Telemetry [baseline] (7.771 ms) : 0, 7771
Telemetry [candidate] (7.821 ms) : 0, 7821
IAST [baseline] (24.003 ms) : 0, 24003
IAST [candidate] (22.642 ms) : 0, 22642
Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.50.0-SNAPSHOT~0f77a40332, baseline=1.50.0-SNAPSHOT~ad6d5fef42
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.024 s) : 0, 1024095
Total [baseline] (10.517 s) : 0, 10516902
Agent [candidate] (1.021 s) : 0, 1021125
Total [candidate] (10.446 s) : 0, 10446188
section appsec
Agent [baseline] (1.168 s) : 0, 1168197
Total [baseline] (10.744 s) : 0, 10743536
Agent [candidate] (1.162 s) : 0, 1161547
Total [candidate] (10.699 s) : 0, 10698560
section iast
Agent [baseline] (1.171 s) : 0, 1171352
Total [baseline] (10.954 s) : 0, 10953680
Agent [candidate] (1.149 s) : 0, 1149150
Total [candidate] (10.895 s) : 0, 10895495
section profiling
Agent [baseline] (1.28 s) : 0, 1279705
Total [baseline] (10.869 s) : 0, 10869102
Agent [candidate] (1.269 s) : 0, 1268556
Total [candidate] (10.868 s) : 0, 10867999
gantt
title petclinic - break down per module: candidate=1.50.0-SNAPSHOT~0f77a40332, baseline=1.50.0-SNAPSHOT~ad6d5fef42
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (687.224 ms) : 0, 687224
BytebuddyAgent [candidate] (684.358 ms) : 0, 684358
GlobalTracer [baseline] (240.893 ms) : 0, 240893
GlobalTracer [candidate] (240.525 ms) : 0, 240525
AppSec [baseline] (54.757 ms) : 0, 54757
AppSec [candidate] (54.605 ms) : 0, 54605
Debugger [baseline] (6.261 ms) : 0, 6261
Debugger [candidate] (7.613 ms) : 0, 7613
Remote Config [baseline] (727.722 µs) : 0, 728
Remote Config [candidate] (716.691 µs) : 0, 717
Telemetry [baseline] (10.657 ms) : 0, 10657
Telemetry [candidate] (9.802 ms) : 0, 9802
section appsec
BytebuddyAgent [baseline] (705.153 ms) : 0, 705153
BytebuddyAgent [candidate] (701.058 ms) : 0, 701058
GlobalTracer [baseline] (238.127 ms) : 0, 238127
GlobalTracer [candidate] (236.965 ms) : 0, 236965
AppSec [baseline] (176.185 ms) : 0, 176185
AppSec [candidate] (174.854 ms) : 0, 174854
Debugger [baseline] (6.007 ms) : 0, 6007
Debugger [candidate] (5.943 ms) : 0, 5943
Remote Config [baseline] (627.544 µs) : 0, 628
Remote Config [candidate] (627.137 µs) : 0, 627
Telemetry [baseline] (7.444 ms) : 0, 7444
Telemetry [candidate] (7.76 ms) : 0, 7760
IAST [baseline] (21.939 ms) : 0, 21939
IAST [candidate] (21.653 ms) : 0, 21653
section iast
BytebuddyAgent [baseline] (818.66 ms) : 0, 818660
BytebuddyAgent [candidate] (801.905 ms) : 0, 801905
GlobalTracer [baseline] (234.011 ms) : 0, 234011
GlobalTracer [candidate] (230.784 ms) : 0, 230784
AppSec [baseline] (52.887 ms) : 0, 52887
AppSec [candidate] (48.423 ms) : 0, 48423
Debugger [baseline] (5.99 ms) : 0, 5990
Debugger [candidate] (5.92 ms) : 0, 5920
Remote Config [baseline] (599.271 µs) : 0, 599
Remote Config [candidate] (590.715 µs) : 0, 591
Telemetry [baseline] (8.105 ms) : 0, 8105
Telemetry [candidate] (7.918 ms) : 0, 7918
IAST [baseline] (26.48 ms) : 0, 26480
IAST [candidate] (29.31 ms) : 0, 29310
section profiling
BytebuddyAgent [baseline] (673.285 ms) : 0, 673285
BytebuddyAgent [candidate] (676.106 ms) : 0, 676106
GlobalTracer [baseline] (377.952 ms) : 0, 377952
GlobalTracer [candidate] (360.458 ms) : 0, 360458
AppSec [baseline] (54.204 ms) : 0, 54204
AppSec [candidate] (61.994 ms) : 0, 61994
Debugger [baseline] (6.137 ms) : 0, 6137
Debugger [candidate] (6.231 ms) : 0, 6231
Remote Config [baseline] (645.736 µs) : 0, 646
Remote Config [candidate] (669.924 µs) : 0, 670
Telemetry [baseline] (8.038 ms) : 0, 8038
Telemetry [candidate] (8.283 ms) : 0, 8283
ProfilingAgent [baseline] (108.91 ms) : 0, 108910
ProfilingAgent [candidate] (103.762 ms) : 0, 103762
Profiling [baseline] (108.936 ms) : 0, 108936
Profiling [candidate] (103.786 ms) : 0, 103786
LoadParameters
See matching parameters
SummaryFound 1 performance improvements and 0 performance regressions! Performance is the same for 11 metrics, 18 unstable metrics.
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.50.0-SNAPSHOT~0f77a40332, baseline=1.50.0-SNAPSHOT~ad6d5fef42
dateFormat X
axisFormat %s
section baseline
no_agent (1.362 ms) : 1342, 1381
. : milestone, 1362,
appsec (1.731 ms) : 1708, 1754
. : milestone, 1731,
appsec_no_iast (1.718 ms) : 1695, 1741
. : milestone, 1718,
code_origins (1.669 ms) : 1642, 1696
. : milestone, 1669,
iast (1.513 ms) : 1490, 1537
. : milestone, 1513,
profiling (1.567 ms) : 1542, 1593
. : milestone, 1567,
tracing (1.494 ms) : 1469, 1519
. : milestone, 1494,
section candidate
no_agent (1.351 ms) : 1332, 1371
. : milestone, 1351,
appsec (1.733 ms) : 1710, 1756
. : milestone, 1733,
appsec_no_iast (1.733 ms) : 1710, 1756
. : milestone, 1733,
code_origins (1.671 ms) : 1644, 1698
. : milestone, 1671,
iast (1.51 ms) : 1486, 1534
. : milestone, 1510,
profiling (1.487 ms) : 1464, 1511
. : milestone, 1487,
tracing (1.506 ms) : 1481, 1531
. : milestone, 1506,
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.50.0-SNAPSHOT~0f77a40332, baseline=1.50.0-SNAPSHOT~ad6d5fef42
dateFormat X
axisFormat %s
section baseline
no_agent (382.812 µs) : 363, 403
. : milestone, 383,
iast (516.221 µs) : 494, 539
. : milestone, 516,
iast_FULL (734.042 µs) : 712, 757
. : milestone, 734,
iast_GLOBAL (556.782 µs) : 535, 578
. : milestone, 557,
iast_HARDCODED_SECRET_DISABLED (528.052 µs) : 505, 551
. : milestone, 528,
iast_INACTIVE (470.771 µs) : 447, 494
. : milestone, 471,
iast_TELEMETRY_OFF (515.199 µs) : 492, 538
. : milestone, 515,
tracing (459.82 µs) : 438, 482
. : milestone, 460,
section candidate
no_agent (377.994 µs) : 358, 398
. : milestone, 378,
iast (519.981 µs) : 498, 542
. : milestone, 520,
iast_FULL (731.508 µs) : 709, 754
. : milestone, 732,
iast_GLOBAL (565.638 µs) : 543, 588
. : milestone, 566,
iast_HARDCODED_SECRET_DISABLED (524.276 µs) : 501, 547
. : milestone, 524,
iast_INACTIVE (465.415 µs) : 443, 488
. : milestone, 465,
iast_TELEMETRY_OFF (499.51 µs) : 476, 523
. : milestone, 500,
tracing (457.545 µs) : 435, 480
. : milestone, 458,
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.50.0-SNAPSHOT~0f77a40332, baseline=1.50.0-SNAPSHOT~ad6d5fef42
dateFormat X
axisFormat %s
section baseline
no_agent (1.48 ms) : 1469, 1492
. : milestone, 1480,
appsec (2.408 ms) : 2359, 2457
. : milestone, 2408,
iast (2.187 ms) : 2125, 2249
. : milestone, 2187,
iast_GLOBAL (2.232 ms) : 2170, 2294
. : milestone, 2232,
profiling (2.046 ms) : 1995, 2096
. : milestone, 2046,
tracing (2.009 ms) : 1961, 2056
. : milestone, 2009,
section candidate
no_agent (1.476 ms) : 1465, 1488
. : milestone, 1476,
appsec (2.425 ms) : 2375, 2475
. : milestone, 2425,
iast (2.201 ms) : 2138, 2263
. : milestone, 2201,
iast_GLOBAL (2.225 ms) : 2163, 2288
. : milestone, 2225,
profiling (2.037 ms) : 1987, 2087
. : milestone, 2037,
tracing (2.023 ms) : 1975, 2072
. : milestone, 2023,
Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.50.0-SNAPSHOT~0f77a40332, baseline=1.50.0-SNAPSHOT~ad6d5fef42
dateFormat X
axisFormat %s
section baseline
no_agent (15.395 s) : 15395000, 15395000
. : milestone, 15395000,
appsec (15.051 s) : 15051000, 15051000
. : milestone, 15051000,
iast (18.487 s) : 18487000, 18487000
. : milestone, 18487000,
iast_GLOBAL (18.07 s) : 18070000, 18070000
. : milestone, 18070000,
profiling (14.934 s) : 14934000, 14934000
. : milestone, 14934000,
tracing (14.931 s) : 14931000, 14931000
. : milestone, 14931000,
section candidate
no_agent (15.551 s) : 15551000, 15551000
. : milestone, 15551000,
appsec (15.052 s) : 15052000, 15052000
. : milestone, 15052000,
iast (19.22 s) : 19220000, 19220000
. : milestone, 19220000,
iast_GLOBAL (18.077 s) : 18077000, 18077000
. : milestone, 18077000,
profiling (15.635 s) : 15635000, 15635000
. : milestone, 15635000,
tracing (14.968 s) : 14968000, 14968000
. : milestone, 14968000,
|
…BYTES default access
…ace-java into mtoff/scfg_file_size_limit
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
Introduces a file size limit (256 KB) to limit the maximum "hands off" configuration file size that we will read.
Motivation
Protection against corrupted files and maintain performance.
See: RFC
The value (256 KB) was determined by taking the size of a sample "hands off" configuration file with both
apm_configuration_default
andapm_configuration_rules
(phase 1 and phase 2 inputs, respectively), multiplying it by100 * 2
. This value may change, or may become configurable, in the future depending on where phase 2 specifications land.Additional Notes
Contributor Checklist
type:
and (comp:
orinst:
) labels in addition to any usefull labelsclose
,fix
or any linking keywords when referencing an issue.Use
solves
instead, and assign the PR milestone to the issueJira ticket: [PROJ-IDENT]