Skip to content

Commit 19fc217

Browse files
committed
[java][js] Add Firefox CDP deprecation warnings
1 parent 2bd18eb commit 19fc217

File tree

3 files changed

+16
-0
lines changed

3 files changed

+16
-0
lines changed

java/src/org/openqa/selenium/devtools/CdpEndpointFinder.java

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -91,6 +91,9 @@ public static Optional<URI> getReportedUri(Capabilities caps) {
9191
break;
9292
case "firefox":
9393
key = "moz:debuggerAddress";
94+
LOG.warning(
95+
"CDP support for Firefox is deprecated and will be removed in future versions. "
96+
+ "Please switch to WebDriver BiDi.");
9497
break;
9598
default:
9699
return Optional.empty();

java/src/org/openqa/selenium/firefox/FirefoxDriver.java

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -164,6 +164,13 @@ private FirefoxDriver(
164164

165165
Optional<URI> reportedUri =
166166
CdpEndpointFinder.getReportedUri("moz:debuggerAddress", capabilities);
167+
168+
if (reportedUri.isPresent()) {
169+
LOG.warning(
170+
"CDP support for Firefox is deprecated and will be removed in future versions. "
171+
+ "Please switch to WebDriver BiDi.");
172+
}
173+
167174
Optional<HttpClient> client =
168175
reportedUri.map(uri -> CdpEndpointFinder.getHttpClient(factory, uri));
169176
Optional<URI> cdpUri;

javascript/node/selenium-webdriver/lib/webdriver.js

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1230,6 +1230,12 @@ class WebDriver {
12301230

12311231
const caps = await this.getCapabilities()
12321232

1233+
if (caps['map_'].get('browserName') === 'firefox') {
1234+
console.warn(
1235+
'CDP support for Firefox is deprecated and will be removed in future versions. Please switch to WebDriver BiDi.',
1236+
)
1237+
}
1238+
12331239
if (process.env.SELENIUM_REMOTE_URL) {
12341240
const host = new URL(process.env.SELENIUM_REMOTE_URL).host
12351241
const sessionId = await this.getSession().then((session) => session.getId())

0 commit comments

Comments
 (0)