[java] ensure the original RemoteNode stays DOWN #13646

SeleniumHQ · Mar 1, 2024 · 75d4eaf · 75d4eaf
1 parent d65e38e
commit 75d4eaf
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/java/src/org/openqa/selenium/grid/node/remote/RemoteNode.java b/java/src/org/openqa/selenium/grid/node/remote/RemoteNode.java
@@ -279,6 +279,12 @@ public Result check() {
       try {
         NodeStatus status = getStatus();
 
+        if (!Objects.equals(getId(), status.getNodeId())) {
+          // ensure the original RemoteNode stays DOWN when it has been restarted and registered
+          // again as another RemoteNode with the same externalUri
+          return new Result(DOWN, externalUri + " has unexpected node id");
+        }
+
         switch (status.getAvailability()) {
           case DOWN:
             return new Result(DOWN, externalUri + " is down");