-
Notifications
You must be signed in to change notification settings - Fork 319
Refactor: Introduce overloaded tryCommitBlockingResponse accepting RequestBlockingAction #10125
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
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 59 metrics, 6 unstable metrics. Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.57.0-SNAPSHOT~5f62fa11c7, baseline=1.57.0-SNAPSHOT~f38cc2ee7a
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.092 s) : 0, 1092165
Total [baseline] (10.911 s) : 0, 10911324
Agent [candidate] (1.081 s) : 0, 1081465
Total [candidate] (10.753 s) : 0, 10753485
section appsec
Agent [baseline] (1.266 s) : 0, 1266057
Total [baseline] (11.12 s) : 0, 11119621
Agent [candidate] (1.273 s) : 0, 1272840
Total [candidate] (11.109 s) : 0, 11108596
section iast
Agent [baseline] (1.223 s) : 0, 1223316
Total [baseline] (11.133 s) : 0, 11132712
Agent [candidate] (1.227 s) : 0, 1227351
Total [candidate] (11.077 s) : 0, 11076639
section profiling
Agent [baseline] (1.221 s) : 0, 1220875
Total [baseline] (11.056 s) : 0, 11055549
Agent [candidate] (1.205 s) : 0, 1205320
Total [candidate] (10.941 s) : 0, 10941346
gantt
title petclinic - break down per module: candidate=1.57.0-SNAPSHOT~5f62fa11c7, baseline=1.57.0-SNAPSHOT~f38cc2ee7a
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.214 ms) : 0, 1214
crashtracking [candidate] (1.188 ms) : 0, 1188
BytebuddyAgent [baseline] (656.126 ms) : 0, 656126
BytebuddyAgent [candidate] (648.957 ms) : 0, 648957
GlobalTracer [baseline] (283.798 ms) : 0, 283798
GlobalTracer [candidate] (281.927 ms) : 0, 281927
AppSec [baseline] (32.672 ms) : 0, 32672
AppSec [candidate] (32.352 ms) : 0, 32352
Debugger [baseline] (68.946 ms) : 0, 68946
Debugger [candidate] (68.199 ms) : 0, 68199
Remote Config [baseline] (655.31 µs) : 0, 655
Remote Config [candidate] (624.943 µs) : 0, 625
Telemetry [baseline] (9.155 ms) : 0, 9155
Telemetry [candidate] (9.051 ms) : 0, 9051
Flare Poller [baseline] (3.771 ms) : 0, 3771
Flare Poller [candidate] (3.711 ms) : 0, 3711
section appsec
crashtracking [baseline] (1.219 ms) : 0, 1219
crashtracking [candidate] (1.219 ms) : 0, 1219
BytebuddyAgent [baseline] (690.173 ms) : 0, 690173
BytebuddyAgent [candidate] (694.864 ms) : 0, 694864
GlobalTracer [baseline] (258.955 ms) : 0, 258955
GlobalTracer [candidate] (261.14 ms) : 0, 261140
IAST [baseline] (24.8 ms) : 0, 24800
IAST [candidate] (24.926 ms) : 0, 24926
AppSec [baseline] (174.325 ms) : 0, 174325
AppSec [candidate] (175.355 ms) : 0, 175355
Debugger [baseline] (67.292 ms) : 0, 67292
Debugger [candidate] (66.005 ms) : 0, 66005
Remote Config [baseline] (727.461 µs) : 0, 727
Remote Config [candidate] (709.458 µs) : 0, 709
Telemetry [baseline] (9.024 ms) : 0, 9024
Telemetry [candidate] (9.123 ms) : 0, 9123
Flare Poller [baseline] (3.96 ms) : 0, 3960
Flare Poller [candidate] (3.943 ms) : 0, 3943
section iast
crashtracking [baseline] (1.211 ms) : 0, 1211
crashtracking [candidate] (1.2 ms) : 0, 1200
BytebuddyAgent [baseline] (791.149 ms) : 0, 791149
BytebuddyAgent [candidate] (795.43 ms) : 0, 795430
GlobalTracer [baseline] (255.633 ms) : 0, 255633
GlobalTracer [candidate] (255.734 ms) : 0, 255734
IAST [baseline] (26.906 ms) : 0, 26906
IAST [candidate] (26.755 ms) : 0, 26755
AppSec [baseline] (35.311 ms) : 0, 35311
AppSec [candidate] (34.259 ms) : 0, 34259
Debugger [baseline] (65.417 ms) : 0, 65417
Debugger [candidate] (66.031 ms) : 0, 66031
Remote Config [baseline] (546.274 µs) : 0, 546
Remote Config [candidate] (530.296 µs) : 0, 530
Telemetry [baseline] (8.41 ms) : 0, 8410
Telemetry [candidate] (8.435 ms) : 0, 8435
Flare Poller [baseline] (3.457 ms) : 0, 3457
Flare Poller [candidate] (3.467 ms) : 0, 3467
section profiling
crashtracking [baseline] (1.21 ms) : 0, 1210
crashtracking [candidate] (1.183 ms) : 0, 1183
BytebuddyAgent [baseline] (711.96 ms) : 0, 711960
BytebuddyAgent [candidate] (701.963 ms) : 0, 701963
GlobalTracer [baseline] (223.512 ms) : 0, 223512
GlobalTracer [candidate] (220.682 ms) : 0, 220682
AppSec [baseline] (33.028 ms) : 0, 33028
AppSec [candidate] (32.258 ms) : 0, 32258
Debugger [baseline] (69.141 ms) : 0, 69141
Debugger [candidate] (68.615 ms) : 0, 68615
Remote Config [baseline] (644.63 µs) : 0, 645
Remote Config [candidate] (642.291 µs) : 0, 642
Telemetry [baseline] (8.976 ms) : 0, 8976
Telemetry [candidate] (8.93 ms) : 0, 8930
Flare Poller [baseline] (3.793 ms) : 0, 3793
Flare Poller [candidate] (3.755 ms) : 0, 3755
ProfilingAgent [baseline] (98.177 ms) : 0, 98177
ProfilingAgent [candidate] (97.911 ms) : 0, 97911
Profiling [baseline] (98.764 ms) : 0, 98764
Profiling [candidate] (98.485 ms) : 0, 98485
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.57.0-SNAPSHOT~5f62fa11c7, baseline=1.57.0-SNAPSHOT~f38cc2ee7a
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.088 s) : 0, 1087884
Total [baseline] (8.796 s) : 0, 8795589
Agent [candidate] (1.091 s) : 0, 1090654
Total [candidate] (8.794 s) : 0, 8794043
section iast
Agent [baseline] (1.22 s) : 0, 1220277
Total [baseline] (9.46 s) : 0, 9460388
Agent [candidate] (1.223 s) : 0, 1222833
Total [candidate] (9.436 s) : 0, 9435515
gantt
title insecure-bank - break down per module: candidate=1.57.0-SNAPSHOT~5f62fa11c7, baseline=1.57.0-SNAPSHOT~f38cc2ee7a
dateFormat X
axisFormat %s
section tracing
crashtracking [baseline] (1.191 ms) : 0, 1191
crashtracking [candidate] (1.2 ms) : 0, 1200
BytebuddyAgent [baseline] (653.751 ms) : 0, 653751
BytebuddyAgent [candidate] (655.111 ms) : 0, 655111
GlobalTracer [baseline] (283.609 ms) : 0, 283609
GlobalTracer [candidate] (284.553 ms) : 0, 284553
AppSec [baseline] (32.564 ms) : 0, 32564
AppSec [candidate] (32.718 ms) : 0, 32718
Debugger [baseline] (67.65 ms) : 0, 67650
Debugger [candidate] (67.558 ms) : 0, 67558
Remote Config [baseline] (657.408 µs) : 0, 657
Remote Config [candidate] (669.517 µs) : 0, 670
Telemetry [baseline] (9.107 ms) : 0, 9107
Telemetry [candidate] (9.178 ms) : 0, 9178
Flare Poller [baseline] (3.771 ms) : 0, 3771
Flare Poller [candidate] (3.911 ms) : 0, 3911
section iast
crashtracking [baseline] (1.187 ms) : 0, 1187
crashtracking [candidate] (1.188 ms) : 0, 1188
BytebuddyAgent [baseline] (788.52 ms) : 0, 788520
BytebuddyAgent [candidate] (791.384 ms) : 0, 791384
GlobalTracer [baseline] (255.856 ms) : 0, 255856
GlobalTracer [candidate] (255.546 ms) : 0, 255546
IAST [baseline] (26.869 ms) : 0, 26869
IAST [candidate] (26.988 ms) : 0, 26988
AppSec [baseline] (34.338 ms) : 0, 34338
AppSec [candidate] (35.468 ms) : 0, 35468
Debugger [baseline] (65.585 ms) : 0, 65585
Debugger [candidate] (64.324 ms) : 0, 64324
Remote Config [baseline] (560.806 µs) : 0, 561
Remote Config [candidate] (535.145 µs) : 0, 535
Telemetry [baseline] (8.427 ms) : 0, 8427
Telemetry [candidate] (8.444 ms) : 0, 8444
Flare Poller [baseline] (3.457 ms) : 0, 3457
Flare Poller [candidate] (3.526 ms) : 0, 3526
LoadParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 19 metrics, 17 unstable metrics. Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.57.0-SNAPSHOT~5f62fa11c7, baseline=1.57.0-SNAPSHOT~f38cc2ee7a
dateFormat X
axisFormat %s
section baseline
no_agent (18.898 ms) : 18705, 19091
. : milestone, 18898,
appsec (18.603 ms) : 18417, 18789
. : milestone, 18603,
code_origins (17.706 ms) : 17527, 17885
. : milestone, 17706,
iast (17.843 ms) : 17666, 18019
. : milestone, 17843,
profiling (18.476 ms) : 18292, 18659
. : milestone, 18476,
tracing (17.679 ms) : 17507, 17851
. : milestone, 17679,
section candidate
no_agent (17.3 ms) : 17123, 17478
. : milestone, 17300,
appsec (18.555 ms) : 18366, 18744
. : milestone, 18555,
code_origins (17.613 ms) : 17439, 17788
. : milestone, 17613,
iast (17.786 ms) : 17609, 17964
. : milestone, 17786,
profiling (18.682 ms) : 18494, 18870
. : milestone, 18682,
tracing (17.717 ms) : 17539, 17896
. : milestone, 17717,
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.57.0-SNAPSHOT~5f62fa11c7, baseline=1.57.0-SNAPSHOT~f38cc2ee7a
dateFormat X
axisFormat %s
section baseline
no_agent (1.197 ms) : 1185, 1209
. : milestone, 1197,
iast (3.238 ms) : 3194, 3281
. : milestone, 3238,
iast_FULL (5.753 ms) : 5695, 5811
. : milestone, 5753,
iast_GLOBAL (3.574 ms) : 3518, 3629
. : milestone, 3574,
profiling (2.102 ms) : 2083, 2122
. : milestone, 2102,
tracing (1.803 ms) : 1787, 1820
. : milestone, 1803,
section candidate
no_agent (1.197 ms) : 1185, 1208
. : milestone, 1197,
iast (3.328 ms) : 3279, 3377
. : milestone, 3328,
iast_FULL (5.841 ms) : 5782, 5900
. : milestone, 5841,
iast_GLOBAL (3.599 ms) : 3541, 3657
. : milestone, 3599,
profiling (2.051 ms) : 2033, 2069
. : milestone, 2051,
tracing (1.774 ms) : 1760, 1788
. : milestone, 1774,
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.57.0-SNAPSHOT~5f62fa11c7, baseline=1.57.0-SNAPSHOT~f38cc2ee7a
dateFormat X
axisFormat %s
section baseline
no_agent (14.984 s) : 14984000, 14984000
. : milestone, 14984000,
appsec (14.64 s) : 14640000, 14640000
. : milestone, 14640000,
iast (18.55 s) : 18550000, 18550000
. : milestone, 18550000,
iast_GLOBAL (17.739 s) : 17739000, 17739000
. : milestone, 17739000,
profiling (15.113 s) : 15113000, 15113000
. : milestone, 15113000,
tracing (14.766 s) : 14766000, 14766000
. : milestone, 14766000,
section candidate
no_agent (14.972 s) : 14972000, 14972000
. : milestone, 14972000,
appsec (14.709 s) : 14709000, 14709000
. : milestone, 14709000,
iast (18.142 s) : 18142000, 18142000
. : milestone, 18142000,
iast_GLOBAL (17.851 s) : 17851000, 17851000
. : milestone, 17851000,
profiling (14.62 s) : 14620000, 14620000
. : milestone, 14620000,
tracing (14.634 s) : 14634000, 14634000
. : milestone, 14634000,
Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.57.0-SNAPSHOT~5f62fa11c7, baseline=1.57.0-SNAPSHOT~f38cc2ee7a
dateFormat X
axisFormat %s
section baseline
no_agent (1.482 ms) : 1471, 1494
. : milestone, 1482,
appsec (3.723 ms) : 3504, 3942
. : milestone, 3723,
iast (2.211 ms) : 2146, 2275
. : milestone, 2211,
iast_GLOBAL (2.253 ms) : 2188, 2318
. : milestone, 2253,
profiling (2.065 ms) : 2013, 2118
. : milestone, 2065,
tracing (2.05 ms) : 1999, 2101
. : milestone, 2050,
section candidate
no_agent (1.48 ms) : 1469, 1492
. : milestone, 1480,
appsec (3.707 ms) : 3488, 3927
. : milestone, 3707,
iast (2.219 ms) : 2154, 2283
. : milestone, 2219,
iast_GLOBAL (2.262 ms) : 2197, 2327
. : milestone, 2262,
profiling (2.099 ms) : 2045, 2153
. : milestone, 2099,
tracing (2.048 ms) : 1997, 2099
. : milestone, 2048,
|
smola
approved these changes
Dec 11, 2025
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: enhancement
Enhancements and improvements
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
BlockResponseFunctionMotivation
This PR prepares the codebase for adding
securityResponseIdtoRequestBlockingActionin a follow-up PR. Currently, tons of classes manually extract individual fields (statusCode,blockingContentType,extraHeaders) fromRequestBlockingActionbefore callingtryCommitBlockingResponse. This pattern makes it difficult to extendRequestBlockingActionwith new fields, as every call site would need to be updated.Additional Notes
Contributor Checklist
type:and (comp:orinst:) labels in addition to any useful labelsclose,fixor any linking keywords when referencing an issue.Use
solvesinstead, and assign the PR milestone to the issueJira ticket: [PROJ-IDENT]