-
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
Fix Selenium instrumentation to avoid trying to set cookies for about:blank and other similar pages #6973
Merged
nikita-tkachenko-datadog
merged 1 commit into
master
from
nikita-tkachenko/selenium-about-blank
May 3, 2024
Merged
Fix Selenium instrumentation to avoid trying to set cookies for about:blank and other similar pages #6973
nikita-tkachenko-datadog
merged 1 commit into
master
from
nikita-tkachenko/selenium-about-blank
May 3, 2024
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
…:blank and other similar pages
nikita-tkachenko-datadog
added
type: bug
comp: ci visibility
Continuous Integration Visibility
labels
Apr 30, 2024
BenchmarksStartupParameters
See matching parameters
SummaryFound 0 performance improvements and 0 performance regressions! Performance is the same for 48 metrics, 15 unstable metrics. Startup time reports for petclinicgantt
title petclinic - global startup overhead: candidate=1.34.0-SNAPSHOT~948cedc714, baseline=1.34.0-SNAPSHOT~2b0502cacf
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.079 s) : 0, 1078546
Total [baseline] (10.38 s) : 0, 10380126
Agent [candidate] (1.078 s) : 0, 1077945
Total [candidate] (10.372 s) : 0, 10371872
section appsec
Agent [baseline] (1.194 s) : 0, 1193950
Total [baseline] (10.53 s) : 0, 10530066
Agent [candidate] (1.19 s) : 0, 1189705
Total [candidate] (10.475 s) : 0, 10475093
section iast
Agent [baseline] (1.206 s) : 0, 1205677
Total [baseline] (10.754 s) : 0, 10754084
Agent [candidate] (1.205 s) : 0, 1205268
Total [candidate] (10.762 s) : 0, 10761724
section profiling
Agent [baseline] (1.267 s) : 0, 1266500
Total [baseline] (10.578 s) : 0, 10578334
Agent [candidate] (1.267 s) : 0, 1267357
Total [candidate] (10.616 s) : 0, 10615645
gantt
title petclinic - break down per module: candidate=1.34.0-SNAPSHOT~948cedc714, baseline=1.34.0-SNAPSHOT~2b0502cacf
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (677.071 ms) : 0, 677071
BytebuddyAgent [candidate] (675.617 ms) : 0, 675617
GlobalTracer [baseline] (309.644 ms) : 0, 309644
GlobalTracer [candidate] (309.799 ms) : 0, 309799
AppSec [baseline] (49.208 ms) : 0, 49208
AppSec [candidate] (49.611 ms) : 0, 49611
Remote Config [baseline] (653.322 µs) : 0, 653
Remote Config [candidate] (657.51 µs) : 0, 658
Telemetry [baseline] (7.507 ms) : 0, 7507
Telemetry [candidate] (7.58 ms) : 0, 7580
section appsec
BytebuddyAgent [baseline] (698.448 ms) : 0, 698448
BytebuddyAgent [candidate] (696.349 ms) : 0, 696349
GlobalTracer [baseline] (292.139 ms) : 0, 292139
GlobalTracer [candidate] (291.945 ms) : 0, 291945
AppSec [baseline] (149.946 ms) : 0, 149946
AppSec [candidate] (148.949 ms) : 0, 148949
Remote Config [baseline] (635.372 µs) : 0, 635
Remote Config [candidate] (617.659 µs) : 0, 618
Telemetry [baseline] (8.598 ms) : 0, 8598
Telemetry [candidate] (7.892 ms) : 0, 7892
IAST [baseline] (19.407 ms) : 0, 19407
IAST [candidate] (19.14 ms) : 0, 19140
section iast
BytebuddyAgent [baseline] (799.14 ms) : 0, 799140
BytebuddyAgent [candidate] (799.014 ms) : 0, 799014
GlobalTracer [baseline] (289.294 ms) : 0, 289294
GlobalTracer [candidate] (289.897 ms) : 0, 289897
AppSec [baseline] (50.366 ms) : 0, 50366
AppSec [candidate] (50.333 ms) : 0, 50333
Remote Config [baseline] (580.828 µs) : 0, 581
Remote Config [candidate] (1.471 ms) : 0, 1471
Telemetry [baseline] (6.623 ms) : 0, 6623
Telemetry [candidate] (6.677 ms) : 0, 6677
IAST [baseline] (25.155 ms) : 0, 25155
IAST [candidate] (23.248 ms) : 0, 23248
section profiling
BytebuddyAgent [baseline] (677.008 ms) : 0, 677008
BytebuddyAgent [candidate] (677.187 ms) : 0, 677187
GlobalTracer [baseline] (379.929 ms) : 0, 379929
GlobalTracer [candidate] (380.001 ms) : 0, 380001
AppSec [baseline] (49.965 ms) : 0, 49965
AppSec [candidate] (50.12 ms) : 0, 50120
Remote Config [baseline] (710.112 µs) : 0, 710
Remote Config [candidate] (704.265 µs) : 0, 704
Telemetry [baseline] (7.464 ms) : 0, 7464
Telemetry [candidate] (7.526 ms) : 0, 7526
ProfilingAgent [baseline] (95.146 ms) : 0, 95146
ProfilingAgent [candidate] (95.404 ms) : 0, 95404
Profiling [baseline] (95.17 ms) : 0, 95170
Profiling [candidate] (95.428 ms) : 0, 95428
Startup time reports for insecure-bankgantt
title insecure-bank - global startup overhead: candidate=1.34.0-SNAPSHOT~948cedc714, baseline=1.34.0-SNAPSHOT~2b0502cacf
dateFormat X
axisFormat %s
section tracing
Agent [baseline] (1.081 s) : 0, 1080505
Total [baseline] (8.546 s) : 0, 8545752
Agent [candidate] (1.081 s) : 0, 1081139
Total [candidate] (8.54 s) : 0, 8539970
section iast
Agent [baseline] (1.196 s) : 0, 1196165
Total [baseline] (9.022 s) : 0, 9021668
Agent [candidate] (1.198 s) : 0, 1197562
Total [candidate] (8.971 s) : 0, 8971249
section iast_HARDCODED_SECRET_DISABLED
Agent [baseline] (1.213 s) : 0, 1212813
Total [baseline] (9.009 s) : 0, 9008553
Agent [candidate] (1.216 s) : 0, 1216055
Total [candidate] (8.994 s) : 0, 8994314
section iast_TELEMETRY_OFF
Agent [baseline] (1.203 s) : 0, 1203272
Total [baseline] (8.989 s) : 0, 8988523
Agent [candidate] (1.21 s) : 0, 1209722
Total [candidate] (9.015 s) : 0, 9014604
gantt
title insecure-bank - break down per module: candidate=1.34.0-SNAPSHOT~948cedc714, baseline=1.34.0-SNAPSHOT~2b0502cacf
dateFormat X
axisFormat %s
section tracing
BytebuddyAgent [baseline] (676.835 ms) : 0, 676835
BytebuddyAgent [candidate] (677.322 ms) : 0, 677322
GlobalTracer [baseline] (310.913 ms) : 0, 310913
GlobalTracer [candidate] (311.266 ms) : 0, 311266
AppSec [baseline] (49.678 ms) : 0, 49678
AppSec [candidate] (49.844 ms) : 0, 49844
Remote Config [baseline] (680.389 µs) : 0, 680
Remote Config [candidate] (664.88 µs) : 0, 665
Telemetry [baseline] (7.708 ms) : 0, 7708
Telemetry [candidate] (7.582 ms) : 0, 7582
section iast
BytebuddyAgent [baseline] (792.045 ms) : 0, 792045
BytebuddyAgent [candidate] (793.645 ms) : 0, 793645
GlobalTracer [baseline] (287.455 ms) : 0, 287455
GlobalTracer [candidate] (287.809 ms) : 0, 287809
AppSec [baseline] (50.941 ms) : 0, 50941
AppSec [candidate] (48.846 ms) : 0, 48846
IAST [baseline] (24.267 ms) : 0, 24267
IAST [candidate] (24.789 ms) : 0, 24789
Remote Config [baseline] (631.271 µs) : 0, 631
Remote Config [candidate] (669.188 µs) : 0, 669
Telemetry [baseline] (6.687 ms) : 0, 6687
Telemetry [candidate] (7.424 ms) : 0, 7424
section iast_HARDCODED_SECRET_DISABLED
BytebuddyAgent [baseline] (803.977 ms) : 0, 803977
BytebuddyAgent [candidate] (805.584 ms) : 0, 805584
GlobalTracer [baseline] (290.918 ms) : 0, 290918
GlobalTracer [candidate] (291.848 ms) : 0, 291848
AppSec [baseline] (53.131 ms) : 0, 53131
AppSec [candidate] (50.952 ms) : 0, 50952
IAST [baseline] (21.944 ms) : 0, 21944
IAST [candidate] (23.162 ms) : 0, 23162
Remote Config [baseline] (579.856 µs) : 0, 580
Remote Config [candidate] (1.322 ms) : 0, 1322
Telemetry [baseline] (7.54 ms) : 0, 7540
Telemetry [candidate] (8.423 ms) : 0, 8423
section iast_TELEMETRY_OFF
BytebuddyAgent [baseline] (796.447 ms) : 0, 796447
BytebuddyAgent [candidate] (799.724 ms) : 0, 799724
GlobalTracer [baseline] (289.523 ms) : 0, 289523
GlobalTracer [candidate] (292.292 ms) : 0, 292292
AppSec [baseline] (50.861 ms) : 0, 50861
AppSec [candidate] (53.286 ms) : 0, 53286
IAST [baseline] (22.527 ms) : 0, 22527
IAST [candidate] (21.735 ms) : 0, 21735
Remote Config [baseline] (585.812 µs) : 0, 586
Remote Config [candidate] (586.667 µs) : 0, 587
Telemetry [baseline] (8.878 ms) : 0, 8878
Telemetry [candidate] (7.454 ms) : 0, 7454
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 petclinicgantt
title petclinic - request duration [CI 0.99] : candidate=1.34.0-SNAPSHOT~948cedc714, baseline=1.34.0-SNAPSHOT~2b0502cacf
dateFormat X
axisFormat %s
section baseline
no_agent (1.341 ms) : 1322, 1360
. : milestone, 1341,
appsec (1.73 ms) : 1706, 1753
. : milestone, 1730,
appsec_no_iast (1.732 ms) : 1707, 1757
. : milestone, 1732,
iast (1.486 ms) : 1463, 1508
. : milestone, 1486,
profiling (1.523 ms) : 1497, 1549
. : milestone, 1523,
tracing (1.455 ms) : 1431, 1478
. : milestone, 1455,
section candidate
no_agent (1.344 ms) : 1325, 1363
. : milestone, 1344,
appsec (1.732 ms) : 1708, 1756
. : milestone, 1732,
appsec_no_iast (1.746 ms) : 1722, 1771
. : milestone, 1746,
iast (1.493 ms) : 1470, 1516
. : milestone, 1493,
profiling (1.568 ms) : 1543, 1592
. : milestone, 1568,
tracing (1.478 ms) : 1453, 1502
. : milestone, 1478,
Request duration reports for insecure-bankgantt
title insecure-bank - request duration [CI 0.99] : candidate=1.34.0-SNAPSHOT~948cedc714, baseline=1.34.0-SNAPSHOT~2b0502cacf
dateFormat X
axisFormat %s
section baseline
no_agent (372.427 µs) : 352, 392
. : milestone, 372,
iast (478.635 µs) : 458, 499
. : milestone, 479,
iast_FULL (546.317 µs) : 525, 567
. : milestone, 546,
iast_GLOBAL (493.777 µs) : 473, 514
. : milestone, 494,
iast_HARDCODED_SECRET_DISABLED (481.519 µs) : 460, 503
. : milestone, 482,
iast_INACTIVE (455.463 µs) : 433, 477
. : milestone, 455,
iast_TELEMETRY_OFF (477.728 µs) : 457, 499
. : milestone, 478,
tracing (446.366 µs) : 425, 467
. : milestone, 446,
section candidate
no_agent (366.491 µs) : 346, 387
. : milestone, 366,
iast (477.546 µs) : 456, 499
. : milestone, 478,
iast_FULL (548.113 µs) : 528, 569
. : milestone, 548,
iast_GLOBAL (492.797 µs) : 472, 513
. : milestone, 493,
iast_HARDCODED_SECRET_DISABLED (473.9 µs) : 453, 494
. : milestone, 474,
iast_INACTIVE (454.784 µs) : 434, 475
. : milestone, 455,
iast_TELEMETRY_OFF (469.45 µs) : 448, 491
. : milestone, 469,
tracing (445.282 µs) : 425, 465
. : milestone, 445,
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.34.0-SNAPSHOT~948cedc714, baseline=1.34.0-SNAPSHOT~2b0502cacf
dateFormat X
axisFormat %s
section baseline
no_agent (15.221 s) : 15221000, 15221000
. : milestone, 15221000,
appsec (15.324 s) : 15324000, 15324000
. : milestone, 15324000,
iast (18.764 s) : 18764000, 18764000
. : milestone, 18764000,
iast_GLOBAL (17.87 s) : 17870000, 17870000
. : milestone, 17870000,
profiling (15.05 s) : 15050000, 15050000
. : milestone, 15050000,
tracing (15.258 s) : 15258000, 15258000
. : milestone, 15258000,
section candidate
no_agent (14.981 s) : 14981000, 14981000
. : milestone, 14981000,
appsec (14.951 s) : 14951000, 14951000
. : milestone, 14951000,
iast (18.804 s) : 18804000, 18804000
. : milestone, 18804000,
iast_GLOBAL (17.965 s) : 17965000, 17965000
. : milestone, 17965000,
profiling (15.263 s) : 15263000, 15263000
. : milestone, 15263000,
tracing (14.796 s) : 14796000, 14796000
. : milestone, 14796000,
Execution time for tomcatgantt
title tomcat - execution time [CI 0.99] : candidate=1.34.0-SNAPSHOT~948cedc714, baseline=1.34.0-SNAPSHOT~2b0502cacf
dateFormat X
axisFormat %s
section baseline
no_agent (1.467 ms) : 1455, 1478
. : milestone, 1467,
appsec (2.209 ms) : 2175, 2243
. : milestone, 2209,
iast (1.886 ms) : 1851, 1920
. : milestone, 1886,
iast_GLOBAL (1.936 ms) : 1901, 1972
. : milestone, 1936,
profiling (1.857 ms) : 1823, 1892
. : milestone, 1857,
tracing (1.843 ms) : 1811, 1875
. : milestone, 1843,
section candidate
no_agent (1.466 ms) : 1455, 1478
. : milestone, 1466,
appsec (2.206 ms) : 2172, 2240
. : milestone, 2206,
iast (1.887 ms) : 1852, 1923
. : milestone, 1887,
iast_GLOBAL (1.935 ms) : 1900, 1971
. : milestone, 1935,
profiling (2.287 ms) : 2121, 2452
. : milestone, 2287,
tracing (1.854 ms) : 1821, 1887
. : milestone, 1854,
|
juan-fernandez
approved these changes
May 3, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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
Updates Selenium instrumentation to avoid setting cookies for pages whose URL cannot be parsed (
about:blank
is one example of such page).Motivation
Some Selenium tests explicitly open
about:blank
before starting a test case.Trying to set a cookie for this page will lead to a Selenium driver exception.
Additional Notes
Selenium instrumentation adds cookies for loaded pages in order to integrate with DD RUM (if the page has RUM instrumentation configured, RUM will use the cookie to establish correspondence between a RUM session and a CI Visibility test case).
Jira ticket: CIVIS-9921