-
Notifications
You must be signed in to change notification settings - Fork 292
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
Fix integer json parsing probe definition #7957
Conversation
Following #7676 issue, numbers parsed from the json probe definition are still considered as long value which still makes then unfittable to be used as key for Integer -> Integer map. Now we parse integers (without dot numbers) as long and see if they fit into an integer, then downgrade them as DSL value with integer type.
Debugger benchmarksParameters
See matching parameters
SummaryFound 0 performance improvements and 5 performance regressions! Performance is the same for 5 metrics, 5 unstable metrics.
See unchanged results
Request duration reports for reportsgantt
title reports - request duration [CI 0.99] : candidate=None, baseline=None
dateFormat X
axisFormat %s
section baseline
noprobe (304.038 µs) : 263, 345
. : milestone, 304,
basic (298.536 µs) : 289, 308
. : milestone, 299,
loop (10.329 ms) : 10289, 10369
. : milestone, 10329,
section candidate
noprobe (293.638 µs) : 273, 315
. : milestone, 294,
basic (300.982 µs) : 292, 310
. : milestone, 301,
loop (10.806 ms) : 10757, 10855
. : milestone, 10806,
|
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 petclinicgantt
title petclinic - global startup overhead: candidate=1.43.0-SNAPSHOT~fd1bd009d1, baseline=1.43.0-SNAPSHOT~105c9dc224
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.086 s) : 0, 1085914
Total [baseline] (10.454 s) : 0, 10454475
Agent [candidate] (1.086 s) : 0, 1086235
Total [candidate] (10.459 s) : 0, 10458580
section appsec
Agent [baseline] (1.222 s) : 0, 1222193
Total [baseline] (10.72 s) : 0, 10719540
Agent [candidate] (1.218 s) : 0, 1217917
Total [candidate] (10.742 s) : 0, 10741845
section iast
Agent [baseline] (1.21 s) : 0, 1209596
Total [baseline] (10.943 s) : 0, 10942650
Agent [candidate] (1.211 s) : 0, 1210765
Total [candidate] (10.913 s) : 0, 10913109
section profiling
Agent [baseline] (1.283 s) : 0, 1282899
Total [baseline] (10.808 s) : 0, 10808410
Agent [candidate] (1.29 s) : 0, 1289979
Total [candidate] (10.864 s) : 0, 10863642
gantt
title petclinic - break down per module: candidate=1.43.0-SNAPSHOT~fd1bd009d1, baseline=1.43.0-SNAPSHOT~105c9dc224
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (689.146 ms) : 0, 689146
BytebuddyAgent [candidate] (689.062 ms) : 0, 689062
GlobalTracer [baseline] (316.637 ms) : 0, 316637
GlobalTracer [candidate] (317.923 ms) : 0, 317923
AppSec [baseline] (54.266 ms) : 0, 54266
AppSec [candidate] (54.201 ms) : 0, 54201
Remote Config [baseline] (673.785 µs) : 0, 674
Remote Config [candidate] (674.208 µs) : 0, 674
Telemetry [baseline] (11.429 ms) : 0, 11429
Telemetry [candidate] (10.562 ms) : 0, 10562
section appsec
BytebuddyAgent [baseline] (708.303 ms) : 0, 708303
BytebuddyAgent [candidate] (705.36 ms) : 0, 705360
GlobalTracer [baseline] (315.233 ms) : 0, 315233
GlobalTracer [candidate] (313.978 ms) : 0, 313978
AppSec [baseline] (165.984 ms) : 0, 165984
AppSec [candidate] (165.189 ms) : 0, 165189
Remote Config [baseline] (635.845 µs) : 0, 636
Remote Config [candidate] (637.274 µs) : 0, 637
Telemetry [baseline] (8.513 ms) : 0, 8513
Telemetry [candidate] (8.86 ms) : 0, 8860
IAST [baseline] (19.44 ms) : 0, 19440
IAST [candidate] (20.191 ms) : 0, 20191
section iast
BytebuddyAgent [baseline] (804.12 ms) : 0, 804120
BytebuddyAgent [candidate] (804.24 ms) : 0, 804240
GlobalTracer [baseline] (305.445 ms) : 0, 305445
GlobalTracer [candidate] (305.552 ms) : 0, 305552
AppSec [baseline] (57.475 ms) : 0, 57475
AppSec [candidate] (57.996 ms) : 0, 57996
Remote Config [baseline] (613.363 µs) : 0, 613
Remote Config [candidate] (617.399 µs) : 0, 617
Telemetry [baseline] (7.453 ms) : 0, 7453
Telemetry [candidate] (7.569 ms) : 0, 7569
IAST [baseline] (20.712 ms) : 0, 20712
IAST [candidate] (21.004 ms) : 0, 21004
section profiling
BytebuddyAgent [baseline] (682.282 ms) : 0, 682282
BytebuddyAgent [candidate] (686.207 ms) : 0, 686207
GlobalTracer [baseline] (400.933 ms) : 0, 400933
GlobalTracer [candidate] (402.319 ms) : 0, 402319
AppSec [baseline] (54.9 ms) : 0, 54900
AppSec [candidate] (55.275 ms) : 0, 55275
Remote Config [baseline] (664.896 µs) : 0, 665
Remote Config [candidate] (665.63 µs) : 0, 666
Telemetry [baseline] (10.673 ms) : 0, 10673
Telemetry [candidate] (12.778 ms) : 0, 12778
ProfilingAgent [baseline] (94.309 ms) : 0, 94309
ProfilingAgent [candidate] (93.377 ms) : 0, 93377
Profiling [baseline] (94.333 ms) : 0, 94333
Profiling [candidate] (93.4 ms) : 0, 93400
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.43.0-SNAPSHOT~fd1bd009d1, baseline=1.43.0-SNAPSHOT~105c9dc224
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.088 s) : 0, 1087569
Total [baseline] (8.634 s) : 0, 8633722
Agent [candidate] (1.087 s) : 0, 1087046
Total [candidate] (8.64 s) : 0, 8639666
section iast
Agent [baseline] (1.218 s) : 0, 1217598
Total [baseline] (9.18 s) : 0, 9179554
Agent [candidate] (1.21 s) : 0, 1210332
Total [candidate] (9.149 s) : 0, 9149176
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.218 s) : 0, 1217528
Total [baseline] (9.135 s) : 0, 9135500
Agent [candidate] (1.209 s) : 0, 1209259
Total [candidate] (9.123 s) : 0, 9123394
section iast_TELEMETRY_OFF
Agent [baseline] (1.217 s) : 0, 1217252
Total [baseline] (9.207 s) : 0, 9207130
Agent [candidate] (1.207 s) : 0, 1207268
Total [candidate] (9.163 s) : 0, 9163344
gantt
title insecure-bank - break down per module: candidate=1.43.0-SNAPSHOT~fd1bd009d1, baseline=1.43.0-SNAPSHOT~105c9dc224
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (688.764 ms) : 0, 688764
BytebuddyAgent [candidate] (689.321 ms) : 0, 689321
GlobalTracer [baseline] (316.707 ms) : 0, 316707
GlobalTracer [candidate] (317.57 ms) : 0, 317570
AppSec [baseline] (54.747 ms) : 0, 54747
AppSec [candidate] (54.326 ms) : 0, 54326
Remote Config [baseline] (700.474 µs) : 0, 700
Remote Config [candidate] (685.205 µs) : 0, 685
Telemetry [baseline] (12.882 ms) : 0, 12882
Telemetry [candidate] (11.362 ms) : 0, 11362
section iast
BytebuddyAgent [baseline] (809.504 ms) : 0, 809504
BytebuddyAgent [candidate] (804.621 ms) : 0, 804621
GlobalTracer [baseline] (307.039 ms) : 0, 307039
GlobalTracer [candidate] (305.288 ms) : 0, 305288
AppSec [baseline] (56.662 ms) : 0, 56662
AppSec [candidate] (57.662 ms) : 0, 57662
Remote Config [baseline] (623.093 µs) : 0, 623
Remote Config [candidate] (627.876 µs) : 0, 628
Telemetry [baseline] (7.49 ms) : 0, 7490
Telemetry [candidate] (7.502 ms) : 0, 7502
IAST [baseline] (22.452 ms) : 0, 22452
IAST [candidate] (20.826 ms) : 0, 20826
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (809.617 ms) : 0, 809617
BytebuddyAgent [candidate] (803.652 ms) : 0, 803652
GlobalTracer [baseline] (307.393 ms) : 0, 307393
GlobalTracer [candidate] (305.693 ms) : 0, 305693
AppSec [baseline] (56.167 ms) : 0, 56167
AppSec [candidate] (57.414 ms) : 0, 57414
Remote Config [baseline] (612.088 µs) : 0, 612
Remote Config [candidate] (615.762 µs) : 0, 616
Telemetry [baseline] (8.3 ms) : 0, 8300
Telemetry [candidate] (7.477 ms) : 0, 7477
IAST [baseline] (21.549 ms) : 0, 21549
IAST [candidate] (20.593 ms) : 0, 20593
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (806.68 ms) : 0, 806680
BytebuddyAgent [candidate] (799.536 ms) : 0, 799536
GlobalTracer [baseline] (309.003 ms) : 0, 309003
GlobalTracer [candidate] (306.974 ms) : 0, 306974
AppSec [baseline] (57.83 ms) : 0, 57830
AppSec [candidate] (57.76 ms) : 0, 57760
Remote Config [baseline] (632.839 µs) : 0, 633
Remote Config [candidate] (623.392 µs) : 0, 623
Telemetry [baseline] (7.538 ms) : 0, 7538
Telemetry [candidate] (7.441 ms) : 0, 7441
IAST [baseline] (21.686 ms) : 0, 21686
IAST [candidate] (21.161 ms) : 0, 21161
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 insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.43.0-SNAPSHOT~fd1bd009d1, baseline=1.43.0-SNAPSHOT~105c9dc224
dateFormat X
axisFormat %s
section baseline
no_agent (373.352 µs) : 353, 394
. : milestone, 373,
iast (485.134 µs) : 464, 506
. : milestone, 485,
iast_FULL (640.579 µs) : 619, 662
. : milestone, 641,
iast_GLOBAL (511.649 µs) : 490, 533
. : milestone, 512,
iast_HARDCODED_SECRET_DISABLED (483.666 µs) : 463, 505
. : milestone, 484,
iast_INACTIVE (446.151 µs) : 424, 468
. : milestone, 446,
iast_TELEMETRY_OFF (478.498 µs) : 457, 500
. : milestone, 478,
tracing (443.1 µs) : 422, 464
. : milestone, 443,
section candidate
no_agent (370.064 µs) : 350, 390
. : milestone, 370,
iast (495.661 µs) : 475, 517
. : milestone, 496,
iast_FULL (639.914 µs) : 619, 661
. : milestone, 640,
iast_GLOBAL (519.626 µs) : 498, 542
. : milestone, 520,
iast_HARDCODED_SECRET_DISABLED (484.941 µs) : 464, 506
. : milestone, 485,
iast_INACTIVE (451.098 µs) : 430, 472
. : milestone, 451,
iast_TELEMETRY_OFF (474.711 µs) : 453, 496
. : milestone, 475,
tracing (440.652 µs) : 420, 462
. : milestone, 441,
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.43.0-SNAPSHOT~fd1bd009d1, baseline=1.43.0-SNAPSHOT~105c9dc224
dateFormat X
axisFormat %s
section baseline
no_agent (1.34 ms) : 1320, 1360
. : milestone, 1340,
appsec (1.745 ms) : 1721, 1770
. : milestone, 1745,
appsec_no_iast (1.747 ms) : 1722, 1771
. : milestone, 1747,
iast (1.477 ms) : 1454, 1500
. : milestone, 1477,
profiling (1.522 ms) : 1498, 1546
. : milestone, 1522,
tracing (1.47 ms) : 1447, 1494
. : milestone, 1470,
section candidate
no_agent (1.343 ms) : 1323, 1363
. : milestone, 1343,
appsec (1.728 ms) : 1703, 1752
. : milestone, 1728,
appsec_no_iast (1.744 ms) : 1720, 1767
. : milestone, 1744,
iast (1.472 ms) : 1449, 1495
. : milestone, 1472,
profiling (1.491 ms) : 1468, 1514
. : milestone, 1491,
tracing (1.48 ms) : 1456, 1504
. : milestone, 1480,
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.43.0-SNAPSHOT~fd1bd009d1, baseline=1.43.0-SNAPSHOT~105c9dc224
dateFormat X
axisFormat %s
section baseline
no_agent (15.503 s) : 15503000, 15503000
. : milestone, 15503000,
appsec (15.39 s) : 15390000, 15390000
. : milestone, 15390000,
iast (18.394 s) : 18394000, 18394000
. : milestone, 18394000,
iast_GLOBAL (18.281 s) : 18281000, 18281000
. : milestone, 18281000,
profiling (15.56 s) : 15560000, 15560000
. : milestone, 15560000,
tracing (15.107 s) : 15107000, 15107000
. : milestone, 15107000,
section candidate
no_agent (15.088 s) : 15088000, 15088000
. : milestone, 15088000,
appsec (15.215 s) : 15215000, 15215000
. : milestone, 15215000,
iast (19.137 s) : 19137000, 19137000
. : milestone, 19137000,
iast_GLOBAL (18.576 s) : 18576000, 18576000
. : milestone, 18576000,
profiling (14.808 s) : 14808000, 14808000
. : milestone, 14808000,
tracing (15.323 s) : 15323000, 15323000
. : milestone, 15323000,
Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.43.0-SNAPSHOT~fd1bd009d1, baseline=1.43.0-SNAPSHOT~105c9dc224
dateFormat X
axisFormat %s
section baseline
no_agent (1.463 ms) : 1451, 1474
. : milestone, 1463,
appsec (2.325 ms) : 2284, 2366
. : milestone, 2325,
iast (2.071 ms) : 2019, 2123
. : milestone, 2071,
iast_GLOBAL (2.112 ms) : 2060, 2164
. : milestone, 2112,
profiling (2.433 ms) : 2244, 2622
. : milestone, 2433,
tracing (1.917 ms) : 1877, 1957
. : milestone, 1917,
section candidate
no_agent (1.466 ms) : 1455, 1478
. : milestone, 1466,
appsec (2.326 ms) : 2285, 2366
. : milestone, 2326,
iast (2.075 ms) : 2023, 2128
. : milestone, 2075,
iast_GLOBAL (2.108 ms) : 2056, 2160
. : milestone, 2108,
profiling (1.927 ms) : 1885, 1970
. : milestone, 1927,
tracing (1.91 ms) : 1870, 1949
. : milestone, 1910,
|
What Does This Do
Following #7676 issue, numbers parsed from the json probe definition are still considered as long value which still makes then unfittable to be used as key for Integer -> Integer map.
Now we parse integers (without dot numbers) as long and see if they fit into an integer, then downgrade them as DSL value with integer type.
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: DEBUG-3131