-
Notifications
You must be signed in to change notification settings - Fork 303
Fix error mark on http status for IBM liberty #8822
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
Conversation
Hi! 👋 Thanks for your pull request! 🎉 To help us review it, please make sure to:
If you need help, please check our contributing guidelines. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 57 metrics, 14 unstable metrics. Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.50.0-SNAPSHOT~3b7aba26a8, baseline=1.50.0-SNAPSHOT~b4aebf8b31
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.019 s) : 0, 1019124
Total [baseline] (10.462 s) : 0, 10461641
Agent [candidate] (1.02 s) : 0, 1019961
Total [candidate] (10.439 s) : 0, 10439394
section appsec
Agent [baseline] (1.16 s) : 0, 1160163
Total [baseline] (10.648 s) : 0, 10648487
Agent [candidate] (1.167 s) : 0, 1166973
Total [candidate] (10.682 s) : 0, 10682270
section iast
Agent [baseline] (1.156 s) : 0, 1155936
Total [baseline] (10.86 s) : 0, 10859600
Agent [candidate] (1.148 s) : 0, 1147670
Total [candidate] (10.888 s) : 0, 10888173
section profiling
Agent [baseline] (1.279 s) : 0, 1279072
Total [baseline] (10.837 s) : 0, 10836890
Agent [candidate] (1.277 s) : 0, 1277456
Total [candidate] (10.77 s) : 0, 10769992
gantt
title petclinic - break down per module: candidate=1.50.0-SNAPSHOT~3b7aba26a8, baseline=1.50.0-SNAPSHOT~b4aebf8b31
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (682.471 ms) : 0, 682471
BytebuddyAgent [candidate] (682.201 ms) : 0, 682201
GlobalTracer [baseline] (240.489 ms) : 0, 240489
GlobalTracer [candidate] (240.426 ms) : 0, 240426
AppSec [baseline] (54.493 ms) : 0, 54493
AppSec [candidate] (54.718 ms) : 0, 54718
Debugger [baseline] (9.083 ms) : 0, 9083
Debugger [candidate] (8.442 ms) : 0, 8442
Remote Config [baseline] (707.078 µs) : 0, 707
Remote Config [candidate] (703.82 µs) : 0, 704
Telemetry [baseline] (8.358 ms) : 0, 8358
Telemetry [candidate] (9.84 ms) : 0, 9840
section appsec
BytebuddyAgent [baseline] (699.688 ms) : 0, 699688
BytebuddyAgent [candidate] (705.369 ms) : 0, 705369
GlobalTracer [baseline] (236.465 ms) : 0, 236465
GlobalTracer [candidate] (237.421 ms) : 0, 237421
AppSec [baseline] (175.738 ms) : 0, 175738
AppSec [candidate] (175.721 ms) : 0, 175721
Debugger [baseline] (5.948 ms) : 0, 5948
Debugger [candidate] (5.928 ms) : 0, 5928
Remote Config [baseline] (630.325 µs) : 0, 630
Remote Config [candidate] (617.731 µs) : 0, 618
Telemetry [baseline] (7.431 ms) : 0, 7431
Telemetry [candidate] (7.397 ms) : 0, 7397
IAST [baseline] (21.588 ms) : 0, 21588
IAST [candidate] (21.668 ms) : 0, 21668
section iast
BytebuddyAgent [baseline] (807.052 ms) : 0, 807052
BytebuddyAgent [candidate] (801.217 ms) : 0, 801217
GlobalTracer [baseline] (231.633 ms) : 0, 231633
GlobalTracer [candidate] (230.403 ms) : 0, 230403
AppSec [baseline] (51.458 ms) : 0, 51458
AppSec [candidate] (49.283 ms) : 0, 49283
Debugger [baseline] (5.95 ms) : 0, 5950
Debugger [candidate] (5.827 ms) : 0, 5827
Remote Config [baseline] (597.654 µs) : 0, 598
Remote Config [candidate] (587.144 µs) : 0, 587
Telemetry [baseline] (7.928 ms) : 0, 7928
Telemetry [candidate] (7.79 ms) : 0, 7790
IAST [baseline] (27.776 ms) : 0, 27776
IAST [candidate] (29.221 ms) : 0, 29221
section profiling
BytebuddyAgent [baseline] (673.084 ms) : 0, 673084
BytebuddyAgent [candidate] (672.592 ms) : 0, 672592
GlobalTracer [baseline] (374.8 ms) : 0, 374800
GlobalTracer [candidate] (374.078 ms) : 0, 374078
AppSec [baseline] (61.817 ms) : 0, 61817
AppSec [candidate] (61.713 ms) : 0, 61713
Debugger [baseline] (6.298 ms) : 0, 6298
Debugger [candidate] (6.299 ms) : 0, 6299
Remote Config [baseline] (651.83 µs) : 0, 652
Remote Config [candidate] (657.438 µs) : 0, 657
Telemetry [baseline] (8.159 ms) : 0, 8159
Telemetry [candidate] (8.186 ms) : 0, 8186
ProfilingAgent [baseline] (103.47 ms) : 0, 103470
ProfilingAgent [candidate] (103.07 ms) : 0, 103070
Profiling [baseline] (103.495 ms) : 0, 103495
Profiling [candidate] (103.093 ms) : 0, 103093
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.50.0-SNAPSHOT~3b7aba26a8, baseline=1.50.0-SNAPSHOT~b4aebf8b31
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.02 s) : 0, 1019649
Total [baseline] (8.654 s) : 0, 8653657
Agent [candidate] (1.019 s) : 0, 1018869
Total [candidate] (8.629 s) : 0, 8628707
section iast
Agent [baseline] (1.154 s) : 0, 1154239
Total [baseline] (9.226 s) : 0, 9225680
Agent [candidate] (1.167 s) : 0, 1167353
Total [candidate] (9.261 s) : 0, 9261259
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.149 s) : 0, 1148555
Total [baseline] (9.207 s) : 0, 9207140
Agent [candidate] (1.147 s) : 0, 1146995
Total [candidate] (9.209 s) : 0, 9208797
section iast_TELEMETRY_OFF
Agent [baseline] (1.15 s) : 0, 1150191
Total [baseline] (9.259 s) : 0, 9259237
Agent [candidate] (1.144 s) : 0, 1144487
Total [candidate] (9.243 s) : 0, 9243397
gantt
title insecure-bank - break down per module: candidate=1.50.0-SNAPSHOT~3b7aba26a8, baseline=1.50.0-SNAPSHOT~b4aebf8b31
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (682.026 ms) : 0, 682026
BytebuddyAgent [candidate] (681.69 ms) : 0, 681690
GlobalTracer [baseline] (239.69 ms) : 0, 239690
GlobalTracer [candidate] (240.168 ms) : 0, 240168
AppSec [baseline] (54.592 ms) : 0, 54592
AppSec [candidate] (54.708 ms) : 0, 54708
Debugger [baseline] (8.377 ms) : 0, 8377
Debugger [candidate] (7.59 ms) : 0, 7590
Remote Config [baseline] (702.924 µs) : 0, 703
Remote Config [candidate] (718.792 µs) : 0, 719
Telemetry [baseline] (10.707 ms) : 0, 10707
Telemetry [candidate] (10.522 ms) : 0, 10522
section iast
BytebuddyAgent [baseline] (806.101 ms) : 0, 806101
BytebuddyAgent [candidate] (815.125 ms) : 0, 815125
GlobalTracer [baseline] (231.495 ms) : 0, 231495
GlobalTracer [candidate] (233.838 ms) : 0, 233838
AppSec [baseline] (48.593 ms) : 0, 48593
AppSec [candidate] (50.264 ms) : 0, 50264
Debugger [baseline] (5.878 ms) : 0, 5878
Debugger [candidate] (6.023 ms) : 0, 6023
Remote Config [baseline] (599.73 µs) : 0, 600
Remote Config [candidate] (617.419 µs) : 0, 617
Telemetry [baseline] (7.865 ms) : 0, 7865
Telemetry [candidate] (8.017 ms) : 0, 8017
IAST [baseline] (29.348 ms) : 0, 29348
IAST [candidate] (29.704 ms) : 0, 29704
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (801.237 ms) : 0, 801237
BytebuddyAgent [candidate] (799.836 ms) : 0, 799836
GlobalTracer [baseline] (230.724 ms) : 0, 230724
GlobalTracer [candidate] (230.548 ms) : 0, 230548
AppSec [baseline] (49.39 ms) : 0, 49390
AppSec [candidate] (49.404 ms) : 0, 49404
Debugger [baseline] (5.87 ms) : 0, 5870
Debugger [candidate] (5.877 ms) : 0, 5877
Remote Config [baseline] (604.33 µs) : 0, 604
Remote Config [candidate] (591.087 µs) : 0, 591
Telemetry [baseline] (7.906 ms) : 0, 7906
Telemetry [candidate] (7.929 ms) : 0, 7929
IAST [baseline] (29.249 ms) : 0, 29249
IAST [candidate] (29.232 ms) : 0, 29232
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (802.438 ms) : 0, 802438
BytebuddyAgent [candidate] (796.825 ms) : 0, 796825
GlobalTracer [baseline] (231.302 ms) : 0, 231302
GlobalTracer [candidate] (231.365 ms) : 0, 231365
AppSec [baseline] (54.433 ms) : 0, 54433
AppSec [candidate] (55.24 ms) : 0, 55240
Debugger [baseline] (5.948 ms) : 0, 5948
Debugger [candidate] (5.983 ms) : 0, 5983
Remote Config [baseline] (608.202 µs) : 0, 608
Remote Config [candidate] (598.197 µs) : 0, 598
Telemetry [baseline] (7.812 ms) : 0, 7812
Telemetry [candidate] (7.824 ms) : 0, 7824
IAST [baseline] (24.029 ms) : 0, 24029
IAST [candidate] (23.229 ms) : 0, 23229
LoadParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 12 metrics, 18 unstable metrics. Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.50.0-SNAPSHOT~3b7aba26a8, baseline=1.50.0-SNAPSHOT~b4aebf8b31
dateFormat X
axisFormat %s
section baseline
no_agent (1.369 ms) : 1349, 1388
. : milestone, 1369,
appsec (1.732 ms) : 1708, 1756
. : milestone, 1732,
appsec_no_iast (1.731 ms) : 1707, 1755
. : milestone, 1731,
code_origins (1.654 ms) : 1627, 1681
. : milestone, 1654,
iast (1.511 ms) : 1488, 1535
. : milestone, 1511,
profiling (1.508 ms) : 1484, 1532
. : milestone, 1508,
tracing (1.509 ms) : 1484, 1534
. : milestone, 1509,
section candidate
no_agent (1.372 ms) : 1352, 1391
. : milestone, 1372,
appsec (1.736 ms) : 1713, 1760
. : milestone, 1736,
appsec_no_iast (1.744 ms) : 1721, 1767
. : milestone, 1744,
code_origins (1.684 ms) : 1657, 1712
. : milestone, 1684,
iast (1.536 ms) : 1511, 1560
. : milestone, 1536,
profiling (1.541 ms) : 1517, 1564
. : milestone, 1541,
tracing (1.496 ms) : 1472, 1521
. : milestone, 1496,
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.50.0-SNAPSHOT~3b7aba26a8, baseline=1.50.0-SNAPSHOT~b4aebf8b31
dateFormat X
axisFormat %s
section baseline
no_agent (382.533 µs) : 362, 403
. : milestone, 383,
iast (521.871 µs) : 500, 544
. : milestone, 522,
iast_FULL (738.751 µs) : 717, 761
. : milestone, 739,
iast_GLOBAL (569.816 µs) : 548, 592
. : milestone, 570,
iast_HARDCODED_SECRET_DISABLED (520.488 µs) : 498, 543
. : milestone, 520,
iast_INACTIVE (469.725 µs) : 447, 493
. : milestone, 470,
iast_TELEMETRY_OFF (502.466 µs) : 479, 526
. : milestone, 502,
tracing (468.563 µs) : 446, 491
. : milestone, 469,
section candidate
no_agent (379.364 µs) : 359, 399
. : milestone, 379,
iast (516.96 µs) : 495, 539
. : milestone, 517,
iast_FULL (737.292 µs) : 715, 759
. : milestone, 737,
iast_GLOBAL (571.373 µs) : 549, 594
. : milestone, 571,
iast_HARDCODED_SECRET_DISABLED (526.774 µs) : 504, 549
. : milestone, 527,
iast_INACTIVE (469.967 µs) : 447, 493
. : milestone, 470,
iast_TELEMETRY_OFF (512.687 µs) : 490, 536
. : milestone, 513,
tracing (461.492 µs) : 439, 484
. : milestone, 461,
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 biojavagantt
title biojava - execution time [CI 0.99] : candidate=1.50.0-SNAPSHOT~3b7aba26a8, baseline=1.50.0-SNAPSHOT~b4aebf8b31
dateFormat X
axisFormat %s
section baseline
no_agent (15.079 s) : 15079000, 15079000
. : milestone, 15079000,
appsec (15.285 s) : 15285000, 15285000
. : milestone, 15285000,
iast (18.54 s) : 18540000, 18540000
. : milestone, 18540000,
iast_GLOBAL (17.7 s) : 17700000, 17700000
. : milestone, 17700000,
profiling (15.189 s) : 15189000, 15189000
. : milestone, 15189000,
tracing (15.093 s) : 15093000, 15093000
. : milestone, 15093000,
section candidate
no_agent (15.242 s) : 15242000, 15242000
. : milestone, 15242000,
appsec (15.0 s) : 15000000, 15000000
. : milestone, 15000000,
iast (18.448 s) : 18448000, 18448000
. : milestone, 18448000,
iast_GLOBAL (17.844 s) : 17844000, 17844000
. : milestone, 17844000,
profiling (15.115 s) : 15115000, 15115000
. : milestone, 15115000,
tracing (15.037 s) : 15037000, 15037000
. : milestone, 15037000,
Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.50.0-SNAPSHOT~3b7aba26a8, baseline=1.50.0-SNAPSHOT~b4aebf8b31
dateFormat X
axisFormat %s
section baseline
no_agent (1.474 ms) : 1462, 1485
. : milestone, 1474,
appsec (2.393 ms) : 2344, 2442
. : milestone, 2393,
iast (2.185 ms) : 2123, 2247
. : milestone, 2185,
iast_GLOBAL (2.232 ms) : 2169, 2295
. : milestone, 2232,
profiling (2.045 ms) : 1994, 2096
. : milestone, 2045,
tracing (2.0 ms) : 1952, 2048
. : milestone, 2000,
section candidate
no_agent (1.467 ms) : 1456, 1479
. : milestone, 1467,
appsec (2.402 ms) : 2352, 2451
. : milestone, 2402,
iast (2.173 ms) : 2111, 2235
. : milestone, 2173,
iast_GLOBAL (2.224 ms) : 2161, 2287
. : milestone, 2224,
profiling (2.038 ms) : 1988, 2089
. : milestone, 2038,
tracing (2.014 ms) : 1966, 2062
. : milestone, 2014,
|
04d2bea
to
e2f2c9a
Compare
What Does This Do
The libery decorator does not take into account flagging the span as errored depending on the http status code.
In fact the code on the HttpServerDecorator that handles that was not called so that logic was skipped.
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: APMS-15737