-
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
Add LFI exploit prevention support #7487
Conversation
BenchmarksStartupParameters
See matching parameters
SummaryFound 3 performance improvements and 0 performance regressions! Performance is the same for 47 metrics, 13 unstable metrics.
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.40.0-SNAPSHOT~b26c58d0bb, baseline=1.40.0-SNAPSHOT~afdbcba335
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.074 s) : 0, 1074029
Total [baseline] (8.601 s) : 0, 8601359
Agent [candidate] (1.072 s) : 0, 1071752
Total [candidate] (8.561 s) : 0, 8561353
section iast
Agent [baseline] (1.2 s) : 0, 1199710
Total [baseline] (8.992 s) : 0, 8992067
Agent [candidate] (1.192 s) : 0, 1191790
Total [candidate] (8.989 s) : 0, 8989275
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.19 s) : 0, 1190006
Total [baseline] (8.976 s) : 0, 8976441
Agent [candidate] (1.199 s) : 0, 1199466
Total [candidate] (8.985 s) : 0, 8985258
section iast_TELEMETRY_OFF
Agent [baseline] (1.199 s) : 0, 1199222
Total [baseline] (8.994 s) : 0, 8993912
Agent [candidate] (1.187 s) : 0, 1187082
Total [candidate] (8.963 s) : 0, 8962992
gantt
title insecure-bank - break down per module: candidate=1.40.0-SNAPSHOT~b26c58d0bb, baseline=1.40.0-SNAPSHOT~afdbcba335
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (686.344 ms) : 0, 686344
BytebuddyAgent [candidate] (684.263 ms) : 0, 684263
GlobalTracer [baseline] (312.853 ms) : 0, 312853
GlobalTracer [candidate] (313.02 ms) : 0, 313020
AppSec [baseline] (52.612 ms) : 0, 52612
AppSec [candidate] (52.377 ms) : 0, 52377
Remote Config [baseline] (734.517 µs) : 0, 735
Remote Config [candidate] (661.558 µs) : 0, 662
Telemetry [baseline] (7.715 ms) : 0, 7715
Telemetry [candidate] (7.634 ms) : 0, 7634
section iast
BytebuddyAgent [baseline] (799.195 ms) : 0, 799195
BytebuddyAgent [candidate] (793.522 ms) : 0, 793522
GlobalTracer [baseline] (301.809 ms) : 0, 301809
GlobalTracer [candidate] (300.418 ms) : 0, 300418
AppSec [baseline] (53.21 ms) : 0, 53210
AppSec [candidate] (53.141 ms) : 0, 53141
Remote Config [baseline] (682.656 µs) : 0, 683
Remote Config [candidate] (632.266 µs) : 0, 632
Telemetry [baseline] (7.374 ms) : 0, 7374
Telemetry [candidate] (7.437 ms) : 0, 7437
IAST [baseline] (23.613 ms) : 0, 23613
IAST [candidate] (22.879 ms) : 0, 22879
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (791.856 ms) : 0, 791856
BytebuddyAgent [candidate] (797.616 ms) : 0, 797616
GlobalTracer [baseline] (299.502 ms) : 0, 299502
GlobalTracer [candidate] (302.176 ms) : 0, 302176
AppSec [baseline] (54.148 ms) : 0, 54148
AppSec [candidate] (54.702 ms) : 0, 54702
Remote Config [baseline] (613.811 µs) : 0, 614
Remote Config [candidate] (608.712 µs) : 0, 609
Telemetry [baseline] (7.395 ms) : 0, 7395
Telemetry [candidate] (8.294 ms) : 0, 8294
IAST [baseline] (22.774 ms) : 0, 22774
IAST [candidate] (22.214 ms) : 0, 22214
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (797.423 ms) : 0, 797423
BytebuddyAgent [candidate] (788.702 ms) : 0, 788702
GlobalTracer [baseline] (302.328 ms) : 0, 302328
GlobalTracer [candidate] (299.989 ms) : 0, 299989
AppSec [baseline] (52.126 ms) : 0, 52126
AppSec [candidate] (52.527 ms) : 0, 52527
Remote Config [baseline] (624.624 µs) : 0, 625
Remote Config [candidate] (605.79 µs) : 0, 606
Telemetry [baseline] (8.161 ms) : 0, 8161
Telemetry [candidate] (8.108 ms) : 0, 8108
IAST [baseline] (24.645 ms) : 0, 24645
IAST [candidate] (23.406 ms) : 0, 23406
Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.40.0-SNAPSHOT~b26c58d0bb, baseline=1.40.0-SNAPSHOT~afdbcba335
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.068 s) : 0, 1067618
Total [baseline] (10.36 s) : 0, 10359578
Agent [candidate] (1.068 s) : 0, 1067715
Total [candidate] (10.415 s) : 0, 10414810
section appsec
Agent [baseline] (1.202 s) : 0, 1201867
Total [baseline] (10.627 s) : 0, 10626748
Agent [candidate] (1.205 s) : 0, 1205261
Total [candidate] (10.621 s) : 0, 10621352
section iast
Agent [baseline] (1.201 s) : 0, 1200751
Total [baseline] (10.846 s) : 0, 10846151
Agent [candidate] (1.202 s) : 0, 1202119
Total [candidate] (10.879 s) : 0, 10879465
section profiling
Agent [baseline] (1.266 s) : 0, 1265990
Total [baseline] (10.588 s) : 0, 10588477
Agent [candidate] (1.269 s) : 0, 1269337
Total [candidate] (10.601 s) : 0, 10601280
gantt
title petclinic - break down per module: candidate=1.40.0-SNAPSHOT~b26c58d0bb, baseline=1.40.0-SNAPSHOT~afdbcba335
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (682.063 ms) : 0, 682063
BytebuddyAgent [candidate] (681.545 ms) : 0, 681545
GlobalTracer [baseline] (311.742 ms) : 0, 311742
GlobalTracer [candidate] (311.551 ms) : 0, 311551
AppSec [baseline] (51.798 ms) : 0, 51798
AppSec [candidate] (52.581 ms) : 0, 52581
Remote Config [baseline] (724.418 µs) : 0, 724
Remote Config [candidate] (682.628 µs) : 0, 683
Telemetry [baseline] (7.547 ms) : 0, 7547
Telemetry [candidate] (7.615 ms) : 0, 7615
section appsec
BytebuddyAgent [baseline] (702.33 ms) : 0, 702330
BytebuddyAgent [candidate] (705.777 ms) : 0, 705777
GlobalTracer [baseline] (304.396 ms) : 0, 304396
GlobalTracer [candidate] (305.053 ms) : 0, 305053
AppSec [baseline] (162.779 ms) : 0, 162779
AppSec [candidate] (161.499 ms) : 0, 161499
Remote Config [baseline] (641.966 µs) : 0, 642
Remote Config [candidate] (647.739 µs) : 0, 648
Telemetry [baseline] (8.282 ms) : 0, 8282
Telemetry [candidate] (8.832 ms) : 0, 8832
IAST [baseline] (19.406 ms) : 0, 19406
IAST [candidate] (19.395 ms) : 0, 19395
section iast
BytebuddyAgent [baseline] (798.71 ms) : 0, 798710
BytebuddyAgent [candidate] (798.616 ms) : 0, 798616
GlobalTracer [baseline] (302.395 ms) : 0, 302395
GlobalTracer [candidate] (303.448 ms) : 0, 303448
AppSec [baseline] (52.163 ms) : 0, 52163
AppSec [candidate] (54.045 ms) : 0, 54045
Remote Config [baseline] (614.296 µs) : 0, 614
Remote Config [candidate] (622.536 µs) : 0, 623
Telemetry [baseline] (7.341 ms) : 0, 7341
Telemetry [candidate] (7.483 ms) : 0, 7483
IAST [baseline] (25.669 ms) : 0, 25669
IAST [candidate] (24.048 ms) : 0, 24048
section profiling
BytebuddyAgent [baseline] (675.833 ms) : 0, 675833
BytebuddyAgent [candidate] (677.016 ms) : 0, 677016
GlobalTracer [baseline] (394.806 ms) : 0, 394806
GlobalTracer [candidate] (395.746 ms) : 0, 395746
AppSec [baseline] (52.575 ms) : 0, 52575
AppSec [candidate] (52.966 ms) : 0, 52966
Remote Config [baseline] (711.493 µs) : 0, 711
Remote Config [candidate] (671.535 µs) : 0, 672
Telemetry [baseline] (7.491 ms) : 0, 7491
Telemetry [candidate] (7.463 ms) : 0, 7463
ProfilingAgent [baseline] (96.369 ms) : 0, 96369
ProfilingAgent [candidate] (97.159 ms) : 0, 97159
Profiling [baseline] (96.392 ms) : 0, 96392
Profiling [candidate] (97.182 ms) : 0, 97182
LoadParameters
See matching parameters
SummaryFound 1 performance improvements and 0 performance regressions! Performance is the same for 9 metrics, 18 unstable metrics.
Request duration reports for petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.40.0-SNAPSHOT~b26c58d0bb, baseline=1.40.0-SNAPSHOT~afdbcba335
dateFormat X
axisFormat %s
section baseline
no_agent (1.351 ms) : 1332, 1371
. : milestone, 1351,
appsec (1.726 ms) : 1701, 1750
. : milestone, 1726,
appsec_no_iast (1.7 ms) : 1675, 1725
. : milestone, 1700,
iast (1.476 ms) : 1454, 1499
. : milestone, 1476,
profiling (1.534 ms) : 1510, 1558
. : milestone, 1534,
tracing (1.466 ms) : 1442, 1491
. : milestone, 1466,
section candidate
no_agent (1.342 ms) : 1323, 1361
. : milestone, 1342,
appsec (1.693 ms) : 1670, 1717
. : milestone, 1693,
appsec_no_iast (1.725 ms) : 1701, 1749
. : milestone, 1725,
iast (1.459 ms) : 1436, 1481
. : milestone, 1459,
profiling (1.462 ms) : 1439, 1485
. : milestone, 1462,
tracing (1.45 ms) : 1425, 1475
. : milestone, 1450,
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.40.0-SNAPSHOT~b26c58d0bb, baseline=1.40.0-SNAPSHOT~afdbcba335
dateFormat X
axisFormat %s
section baseline
no_agent (374.004 µs) : 355, 393
. : milestone, 374,
iast (487.262 µs) : 465, 510
. : milestone, 487,
iast_FULL (553.547 µs) : 532, 575
. : milestone, 554,
iast_GLOBAL (515.243 µs) : 493, 537
. : milestone, 515,
iast_HARDCODED_SECRET_DISABLED (485.391 µs) : 463, 507
. : milestone, 485,
iast_INACTIVE (442.074 µs) : 421, 463
. : milestone, 442,
iast_TELEMETRY_OFF (477.732 µs) : 454, 501
. : milestone, 478,
tracing (454.675 µs) : 434, 475
. : milestone, 455,
section candidate
no_agent (372.674 µs) : 352, 393
. : milestone, 373,
iast (480.848 µs) : 459, 503
. : milestone, 481,
iast_FULL (558.826 µs) : 538, 580
. : milestone, 559,
iast_GLOBAL (498.568 µs) : 478, 520
. : milestone, 499,
iast_HARDCODED_SECRET_DISABLED (486.226 µs) : 464, 508
. : milestone, 486,
iast_INACTIVE (453.697 µs) : 432, 475
. : milestone, 454,
iast_TELEMETRY_OFF (475.492 µs) : 453, 498
. : milestone, 475,
tracing (444.107 µs) : 422, 466
. : milestone, 444,
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.40.0-SNAPSHOT~b26c58d0bb, baseline=1.40.0-SNAPSHOT~afdbcba335
dateFormat X
axisFormat %s
section baseline
no_agent (15.663 s) : 15663000, 15663000
. : milestone, 15663000,
appsec (15.063 s) : 15063000, 15063000
. : milestone, 15063000,
iast (19.038 s) : 19038000, 19038000
. : milestone, 19038000,
iast_GLOBAL (18.032 s) : 18032000, 18032000
. : milestone, 18032000,
profiling (15.678 s) : 15678000, 15678000
. : milestone, 15678000,
tracing (15.153 s) : 15153000, 15153000
. : milestone, 15153000,
section candidate
no_agent (15.074 s) : 15074000, 15074000
. : milestone, 15074000,
appsec (15.212 s) : 15212000, 15212000
. : milestone, 15212000,
iast (18.947 s) : 18947000, 18947000
. : milestone, 18947000,
iast_GLOBAL (18.115 s) : 18115000, 18115000
. : milestone, 18115000,
profiling (15.088 s) : 15088000, 15088000
. : milestone, 15088000,
tracing (15.165 s) : 15165000, 15165000
. : milestone, 15165000,
Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.40.0-SNAPSHOT~b26c58d0bb, baseline=1.40.0-SNAPSHOT~afdbcba335
dateFormat X
axisFormat %s
section baseline
no_agent (1.469 ms) : 1457, 1480
. : milestone, 1469,
appsec (2.292 ms) : 2251, 2333
. : milestone, 2292,
iast (2.042 ms) : 1993, 2090
. : milestone, 2042,
iast_GLOBAL (2.092 ms) : 2042, 2142
. : milestone, 2092,
profiling (1.922 ms) : 1881, 1964
. : milestone, 1922,
tracing (1.908 ms) : 1870, 1947
. : milestone, 1908,
section candidate
no_agent (1.465 ms) : 1454, 1477
. : milestone, 1465,
appsec (2.285 ms) : 2244, 2325
. : milestone, 2285,
iast (2.059 ms) : 2009, 2110
. : milestone, 2059,
iast_GLOBAL (2.104 ms) : 2054, 2154
. : milestone, 2104,
profiling (1.955 ms) : 1913, 1997
. : milestone, 1955,
tracing (1.914 ms) : 1876, 1953
. : milestone, 1914,
|
afe802c
to
5b03afd
Compare
c01fdaf
to
d6c3191
Compare
d10aca5
to
fe72611
Compare
0a20d74
to
7d57134
Compare
e4f07a8
to
ee21476
Compare
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.
Needs to announce RC capability.
ff7afeb
to
0cf5cba
Compare
.../src/test/groovy/datadog/trace/instrumentation/java/io/FileLoadedRaspHelperForkedTest.groovy
Outdated
Show resolved
Hide resolved
dd-java-agent/appsec/src/main/java/com/datadog/appsec/gateway/GatewayBridge.java
Outdated
Show resolved
Hide resolved
@@ -1,7 +1,7 @@ | |||
package datadog.trace.api.telemetry; | |||
|
|||
public enum RuleType { | |||
LIF("lfi"), | |||
LFI("lfi"), |
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.
😆
@@ -248,6 +248,17 @@ public EventType<BiFunction<RequestContext, String, Flow<Void>>> networkConnecti | |||
return (EventType<BiFunction<RequestContext, String, Flow<Void>>>) NETWORK_CONNECTION; | |||
} | |||
|
|||
static final int FILE_LOADED_ID = 20; |
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.
We might have some conflicts here with my PR
...va-agent/appsec/src/test/groovy/com/datadog/appsec/gateway/GatewayBridgeSpecification.groovy
Outdated
Show resolved
Hide resolved
…GatewayBridge.java Co-authored-by: Manuel Álvarez Álvarez <[email protected]>
…y/GatewayBridgeSpecification.groovy Co-authored-by: Manuel Álvarez Álvarez <[email protected]>
What Does This Do
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-46822