From e88e764871e5bb0268b9e460933508d974e135b1 Mon Sep 17 00:00:00 2001
From: Simon Stewart <simon.m.stewart@gmail.com>
Date: Tue, 17 Mar 2020 12:26:27 +0000
Subject: [PATCH] Log URL httpds are listening on when starting servers

---
 java/server/src/org/openqa/selenium/grid/commands/Hub.java  | 6 +++++-
 .../src/org/openqa/selenium/grid/commands/Standalone.java   | 6 +++++-
 .../selenium/grid/distributor/httpd/DistributorServer.java  | 6 +++++-
 .../src/org/openqa/selenium/grid/node/httpd/NodeServer.java | 6 +++++-
 .../org/openqa/selenium/grid/router/httpd/RouterServer.java | 6 +++++-
 .../selenium/grid/sessionmap/httpd/SessionMapServer.java    | 6 +++++-
 6 files changed, 30 insertions(+), 6 deletions(-)

diff --git a/java/server/src/org/openqa/selenium/grid/commands/Hub.java b/java/server/src/org/openqa/selenium/grid/commands/Hub.java
index 061592951122f..4e3ef85bb7920 100644
--- a/java/server/src/org/openqa/selenium/grid/commands/Hub.java
+++ b/java/server/src/org/openqa/selenium/grid/commands/Hub.java
@@ -131,7 +131,11 @@ public Executable configure(String... args) {
       server.start();
 
       BuildInfo info = new BuildInfo();
-      LOG.info(String.format("Started Selenium hub %s (revision %s)", info.getReleaseLabel(), info.getBuildRevision()));
+      LOG.info(String.format(
+        "Started Selenium hub %s (revision %s): %s",
+        info.getReleaseLabel(),
+        info.getBuildRevision(),
+        server.getUrl()));
     };
   }
 }
diff --git a/java/server/src/org/openqa/selenium/grid/commands/Standalone.java b/java/server/src/org/openqa/selenium/grid/commands/Standalone.java
index 0d9a41301278f..aade7cedfb002 100644
--- a/java/server/src/org/openqa/selenium/grid/commands/Standalone.java
+++ b/java/server/src/org/openqa/selenium/grid/commands/Standalone.java
@@ -168,7 +168,11 @@ public Executable configure(String... args) {
       server.start();
 
       BuildInfo info = new BuildInfo();
-      LOG.info(String.format("Started Selenium standalone %s (revision %s)", info.getReleaseLabel(), info.getBuildRevision()));
+      LOG.info(String.format(
+        "Started Selenium standalone %s (revision %s): %s",
+        info.getReleaseLabel(),
+        info.getBuildRevision(),
+        server.getUrl()));
     };
   }
 }
diff --git a/java/server/src/org/openqa/selenium/grid/distributor/httpd/DistributorServer.java b/java/server/src/org/openqa/selenium/grid/distributor/httpd/DistributorServer.java
index ae1d03005b964..1d5ba91f5e420 100644
--- a/java/server/src/org/openqa/selenium/grid/distributor/httpd/DistributorServer.java
+++ b/java/server/src/org/openqa/selenium/grid/distributor/httpd/DistributorServer.java
@@ -127,7 +127,11 @@ public Executable configure(String... args) {
       server.start();
 
       BuildInfo info = new BuildInfo();
-      LOG.info(String.format("Started Selenium distributor %s (revision %s)", info.getReleaseLabel(), info.getBuildRevision()));
+      LOG.info(String.format(
+        "Started Selenium distributor %s (revision %s): %s",
+        info.getReleaseLabel(),
+        info.getBuildRevision(),
+        server.getUrl()));
     };
   }
 }
diff --git a/java/server/src/org/openqa/selenium/grid/node/httpd/NodeServer.java b/java/server/src/org/openqa/selenium/grid/node/httpd/NodeServer.java
index be9a524e74b02..8376e4ee1949c 100644
--- a/java/server/src/org/openqa/selenium/grid/node/httpd/NodeServer.java
+++ b/java/server/src/org/openqa/selenium/grid/node/httpd/NodeServer.java
@@ -136,7 +136,11 @@ public Executable configure(String... args) {
       server.start();
 
       BuildInfo info = new BuildInfo();
-      LOG.info(String.format("Started Selenium node %s (revision %s)", info.getReleaseLabel(), info.getBuildRevision()));
+      LOG.info(String.format(
+        "Started Selenium node %s (revision %s): %s",
+        info.getReleaseLabel(),
+        info.getBuildRevision(),
+        server.getUrl()));
 
       Regularly regularly = new Regularly("Register Node with Distributor");
 
diff --git a/java/server/src/org/openqa/selenium/grid/router/httpd/RouterServer.java b/java/server/src/org/openqa/selenium/grid/router/httpd/RouterServer.java
index 9d0c2302d8499..0f97b5039f0f3 100644
--- a/java/server/src/org/openqa/selenium/grid/router/httpd/RouterServer.java
+++ b/java/server/src/org/openqa/selenium/grid/router/httpd/RouterServer.java
@@ -119,7 +119,11 @@ public Executable configure(String... args) {
       server.start();
 
       BuildInfo info = new BuildInfo();
-      LOG.info(String.format("Started Selenium router %s (revision %s)", info.getReleaseLabel(), info.getBuildRevision()));
+      LOG.info(String.format(
+        "Started Selenium router %s (revision %s): %s",
+        info.getReleaseLabel(),
+        info.getBuildRevision(),
+        server.getUrl()));
     };
   }
 }
diff --git a/java/server/src/org/openqa/selenium/grid/sessionmap/httpd/SessionMapServer.java b/java/server/src/org/openqa/selenium/grid/sessionmap/httpd/SessionMapServer.java
index e6db972dba307..62aafc36d34e6 100644
--- a/java/server/src/org/openqa/selenium/grid/sessionmap/httpd/SessionMapServer.java
+++ b/java/server/src/org/openqa/selenium/grid/sessionmap/httpd/SessionMapServer.java
@@ -121,7 +121,11 @@ public Executable configure(String... args) {
       server.start();
 
       BuildInfo info = new BuildInfo();
-      LOG.info(String.format("Started Selenium session map %s (revision %s)", info.getReleaseLabel(), info.getBuildRevision()));
+      LOG.info(String.format(
+        "Started Selenium session map %s (revision %s): %s",
+        info.getReleaseLabel(),
+        info.getBuildRevision(),
+        server.getUrl()));
     };
   }
 }