-
Notifications
You must be signed in to change notification settings - Fork 290
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
Add missing test for JarReader with trailing separator #7929
Merged
Conversation
This file contains 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
smola
added
tag: no release notes
Changes to exclude from release notes
comp: telemetry
Telemetry
labels
Nov 11, 2024
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 54 metrics, 9 unstable metrics. Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.43.0-SNAPSHOT~0179ac5d07, baseline=1.43.0-SNAPSHOT~9e8d0dcb45
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.08 s) : 0, 1080193
Total [baseline] (10.334 s) : 0, 10333773
Agent [candidate] (1.083 s) : 0, 1083139
Total [candidate] (10.395 s) : 0, 10394588
section appsec
Agent [baseline] (1.216 s) : 0, 1216431
Total [baseline] (10.629 s) : 0, 10629031
Agent [candidate] (1.213 s) : 0, 1213082
Total [candidate] (10.626 s) : 0, 10626165
section iast
Agent [baseline] (1.204 s) : 0, 1204014
Total [baseline] (10.903 s) : 0, 10902516
Agent [candidate] (1.204 s) : 0, 1203848
Total [candidate] (10.882 s) : 0, 10882328
section profiling
Agent [baseline] (1.284 s) : 0, 1284457
Total [baseline] (10.737 s) : 0, 10736683
Agent [candidate] (1.277 s) : 0, 1276814
Total [candidate] (10.662 s) : 0, 10661590
gantt
title petclinic - break down per module: candidate=1.43.0-SNAPSHOT~0179ac5d07, baseline=1.43.0-SNAPSHOT~9e8d0dcb45
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (686.319 ms) : 0, 686319
BytebuddyAgent [candidate] (687.087 ms) : 0, 687087
GlobalTracer [baseline] (314.808 ms) : 0, 314808
GlobalTracer [candidate] (315.548 ms) : 0, 315548
AppSec [baseline] (54.176 ms) : 0, 54176
AppSec [candidate] (54.173 ms) : 0, 54173
Remote Config [baseline] (684.685 µs) : 0, 685
Remote Config [candidate] (672.684 µs) : 0, 673
Telemetry [baseline] (10.562 ms) : 0, 10562
Telemetry [candidate] (11.99 ms) : 0, 11990
section appsec
BytebuddyAgent [baseline] (705.043 ms) : 0, 705043
BytebuddyAgent [candidate] (702.603 ms) : 0, 702603
GlobalTracer [baseline] (313.582 ms) : 0, 313582
GlobalTracer [candidate] (312.526 ms) : 0, 312526
AppSec [baseline] (166.445 ms) : 0, 166445
AppSec [candidate] (165.451 ms) : 0, 165451
Remote Config [baseline] (631.71 µs) : 0, 632
Remote Config [candidate] (633.168 µs) : 0, 633
Telemetry [baseline] (7.382 ms) : 0, 7382
Telemetry [candidate] (7.749 ms) : 0, 7749
IAST [baseline] (19.352 ms) : 0, 19352
IAST [candidate] (20.965 ms) : 0, 20965
section iast
BytebuddyAgent [baseline] (800.128 ms) : 0, 800128
BytebuddyAgent [candidate] (800.647 ms) : 0, 800647
GlobalTracer [baseline] (303.893 ms) : 0, 303893
GlobalTracer [candidate] (304.388 ms) : 0, 304388
AppSec [baseline] (56.764 ms) : 0, 56764
AppSec [candidate] (56.775 ms) : 0, 56775
Remote Config [baseline] (611.879 µs) : 0, 612
Remote Config [candidate] (623.027 µs) : 0, 623
Telemetry [baseline] (7.488 ms) : 0, 7488
Telemetry [candidate] (7.349 ms) : 0, 7349
IAST [baseline] (21.474 ms) : 0, 21474
IAST [candidate] (20.397 ms) : 0, 20397
section profiling
BytebuddyAgent [baseline] (684.453 ms) : 0, 684453
BytebuddyAgent [candidate] (680.208 ms) : 0, 680208
GlobalTracer [baseline] (400.904 ms) : 0, 400904
GlobalTracer [candidate] (398.823 ms) : 0, 398823
AppSec [baseline] (54.878 ms) : 0, 54878
AppSec [candidate] (54.434 ms) : 0, 54434
Remote Config [baseline] (670.666 µs) : 0, 671
Remote Config [candidate] (669.123 µs) : 0, 669
Telemetry [baseline] (13.968 ms) : 0, 13968
Telemetry [candidate] (12.738 ms) : 0, 12738
ProfilingAgent [baseline] (90.485 ms) : 0, 90485
ProfilingAgent [candidate] (91.08 ms) : 0, 91080
Profiling [baseline] (90.509 ms) : 0, 90509
Profiling [candidate] (91.104 ms) : 0, 91104
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.43.0-SNAPSHOT~0179ac5d07, baseline=1.43.0-SNAPSHOT~9e8d0dcb45
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.084 s) : 0, 1084035
Total [baseline] (8.63 s) : 0, 8629535
Agent [candidate] (1.08 s) : 0, 1080157
Total [candidate] (8.592 s) : 0, 8591673
section iast
Agent [baseline] (1.205 s) : 0, 1204718
Total [baseline] (9.154 s) : 0, 9154231
Agent [candidate] (1.204 s) : 0, 1204293
Total [candidate] (9.126 s) : 0, 9126181
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.205 s) : 0, 1205400
Total [baseline] (9.067 s) : 0, 9066644
Agent [candidate] (1.205 s) : 0, 1205161
Total [candidate] (9.12 s) : 0, 9120482
section iast_TELEMETRY_OFF
Agent [baseline] (1.2 s) : 0, 1199845
Total [baseline] (9.077 s) : 0, 9076959
Agent [candidate] (1.211 s) : 0, 1211164
Total [candidate] (9.109 s) : 0, 9108599
gantt
title insecure-bank - break down per module: candidate=1.43.0-SNAPSHOT~0179ac5d07, baseline=1.43.0-SNAPSHOT~9e8d0dcb45
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (687.292 ms) : 0, 687292
BytebuddyAgent [candidate] (686.437 ms) : 0, 686437
GlobalTracer [baseline] (317.035 ms) : 0, 317035
GlobalTracer [candidate] (315.342 ms) : 0, 315342
AppSec [baseline] (54.056 ms) : 0, 54056
AppSec [candidate] (54.084 ms) : 0, 54084
Remote Config [baseline] (668.414 µs) : 0, 668
Remote Config [candidate] (671.348 µs) : 0, 671
Telemetry [baseline] (11.325 ms) : 0, 11325
Telemetry [candidate] (9.925 ms) : 0, 9925
section iast
BytebuddyAgent [baseline] (801.496 ms) : 0, 801496
BytebuddyAgent [candidate] (801.002 ms) : 0, 801002
GlobalTracer [baseline] (304.187 ms) : 0, 304187
GlobalTracer [candidate] (303.677 ms) : 0, 303677
AppSec [baseline] (57.194 ms) : 0, 57194
AppSec [candidate] (56.498 ms) : 0, 56498
IAST [baseline] (20.269 ms) : 0, 20269
IAST [candidate] (21.355 ms) : 0, 21355
Remote Config [baseline] (608.154 µs) : 0, 608
Remote Config [candidate] (619.447 µs) : 0, 619
Telemetry [baseline] (7.305 ms) : 0, 7305
Telemetry [candidate] (7.469 ms) : 0, 7469
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (801.328 ms) : 0, 801328
BytebuddyAgent [candidate] (801.326 ms) : 0, 801326
GlobalTracer [baseline] (304.608 ms) : 0, 304608
GlobalTracer [candidate] (304.453 ms) : 0, 304453
AppSec [baseline] (56.548 ms) : 0, 56548
AppSec [candidate] (57.023 ms) : 0, 57023
IAST [baseline] (21.207 ms) : 0, 21207
IAST [candidate] (20.608 ms) : 0, 20608
Remote Config [baseline] (586.594 µs) : 0, 587
Remote Config [candidate] (600.706 µs) : 0, 601
Telemetry [baseline] (7.451 ms) : 0, 7451
Telemetry [candidate] (7.468 ms) : 0, 7468
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (797.338 ms) : 0, 797338
BytebuddyAgent [candidate] (805.277 ms) : 0, 805277
GlobalTracer [baseline] (303.177 ms) : 0, 303177
GlobalTracer [candidate] (306.449 ms) : 0, 306449
AppSec [baseline] (56.604 ms) : 0, 56604
AppSec [candidate] (56.911 ms) : 0, 56911
IAST [baseline] (21.126 ms) : 0, 21126
IAST [candidate] (20.799 ms) : 0, 20799
Remote Config [baseline] (614.319 µs) : 0, 614
Remote Config [candidate] (595.152 µs) : 0, 595
Telemetry [baseline] (7.368 ms) : 0, 7368
Telemetry [candidate] (7.344 ms) : 0, 7344
LoadParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 12 metrics, 16 unstable metrics. Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.43.0-SNAPSHOT~0179ac5d07, baseline=1.43.0-SNAPSHOT~9e8d0dcb45
dateFormat X
axisFormat %s
section baseline
no_agent (366.788 µs) : 347, 386
. : milestone, 367,
iast (488.908 µs) : 468, 510
. : milestone, 489,
iast_FULL (644.264 µs) : 623, 666
. : milestone, 644,
iast_GLOBAL (513.987 µs) : 493, 535
. : milestone, 514,
iast_HARDCODED_SECRET_DISABLED (495.599 µs) : 474, 517
. : milestone, 496,
iast_INACTIVE (451.55 µs) : 430, 473
. : milestone, 452,
iast_TELEMETRY_OFF (472.058 µs) : 451, 493
. : milestone, 472,
tracing (443.842 µs) : 423, 464
. : milestone, 444,
section candidate
no_agent (374.605 µs) : 355, 394
. : milestone, 375,
iast (486.843 µs) : 465, 508
. : milestone, 487,
iast_FULL (646.75 µs) : 625, 668
. : milestone, 647,
iast_GLOBAL (509.772 µs) : 489, 531
. : milestone, 510,
iast_HARDCODED_SECRET_DISABLED (485.588 µs) : 464, 507
. : milestone, 486,
iast_INACTIVE (446.458 µs) : 426, 467
. : milestone, 446,
iast_TELEMETRY_OFF (477.854 µs) : 456, 499
. : milestone, 478,
tracing (451.674 µs) : 430, 473
. : milestone, 452,
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.43.0-SNAPSHOT~0179ac5d07, baseline=1.43.0-SNAPSHOT~9e8d0dcb45
dateFormat X
axisFormat %s
section baseline
no_agent (1.334 ms) : 1315, 1352
. : milestone, 1334,
appsec (1.725 ms) : 1701, 1749
. : milestone, 1725,
appsec_no_iast (1.729 ms) : 1704, 1753
. : milestone, 1729,
iast (1.483 ms) : 1461, 1506
. : milestone, 1483,
profiling (1.491 ms) : 1468, 1514
. : milestone, 1491,
tracing (1.494 ms) : 1469, 1520
. : milestone, 1494,
section candidate
no_agent (1.351 ms) : 1332, 1370
. : milestone, 1351,
appsec (1.734 ms) : 1710, 1759
. : milestone, 1734,
appsec_no_iast (1.723 ms) : 1699, 1747
. : milestone, 1723,
iast (1.489 ms) : 1466, 1512
. : milestone, 1489,
profiling (1.504 ms) : 1480, 1529
. : milestone, 1504,
tracing (1.466 ms) : 1442, 1490
. : milestone, 1466,
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.43.0-SNAPSHOT~0179ac5d07, baseline=1.43.0-SNAPSHOT~9e8d0dcb45
dateFormat X
axisFormat %s
section baseline
no_agent (1.469 ms) : 1458, 1481
. : milestone, 1469,
appsec (2.341 ms) : 2300, 2382
. : milestone, 2341,
iast (2.078 ms) : 2026, 2131
. : milestone, 2078,
iast_GLOBAL (2.123 ms) : 2070, 2175
. : milestone, 2123,
profiling (1.939 ms) : 1896, 1981
. : milestone, 1939,
tracing (1.918 ms) : 1878, 1958
. : milestone, 1918,
section candidate
no_agent (1.475 ms) : 1463, 1486
. : milestone, 1475,
appsec (2.349 ms) : 2308, 2391
. : milestone, 2349,
iast (2.084 ms) : 2032, 2136
. : milestone, 2084,
iast_GLOBAL (2.123 ms) : 2070, 2175
. : milestone, 2123,
profiling (1.949 ms) : 1907, 1991
. : milestone, 1949,
tracing (1.929 ms) : 1889, 1969
. : milestone, 1929,
Execution time for biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.43.0-SNAPSHOT~0179ac5d07, baseline=1.43.0-SNAPSHOT~9e8d0dcb45
dateFormat X
axisFormat %s
section baseline
no_agent (15.094 s) : 15094000, 15094000
. : milestone, 15094000,
appsec (15.363 s) : 15363000, 15363000
. : milestone, 15363000,
iast (18.253 s) : 18253000, 18253000
. : milestone, 18253000,
iast_GLOBAL (18.013 s) : 18013000, 18013000
. : milestone, 18013000,
profiling (15.569 s) : 15569000, 15569000
. : milestone, 15569000,
tracing (15.35 s) : 15350000, 15350000
. : milestone, 15350000,
section candidate
no_agent (15.256 s) : 15256000, 15256000
. : milestone, 15256000,
appsec (15.331 s) : 15331000, 15331000
. : milestone, 15331000,
iast (18.398 s) : 18398000, 18398000
. : milestone, 18398000,
iast_GLOBAL (17.973 s) : 17973000, 17973000
. : milestone, 17973000,
profiling (14.849 s) : 14849000, 14849000
. : milestone, 14849000,
tracing (14.917 s) : 14917000, 14917000
. : milestone, 14917000,
|
Mariovido
approved these changes
Nov 11, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
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
Add missing test case for
JarReader
for nested jar path with trailing!/
.Motivation
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: APPSEC-55773 (related)