MODCLUSTER-798 Add support for Tomcat's server "portOffset".

rhusar · Dec 11, 2023 · 3a01cdb · 3a01cdb
1 parent 2f7924e
commit 3a01cdb
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 7 deletions.
diff --git a/container/spi/src/main/java/org/jboss/modcluster/container/Connector.java b/container/spi/src/main/java/org/jboss/modcluster/container/Connector.java
@@ -73,7 +73,8 @@ public String toString() {
     void setAddress(InetAddress address);
 
     /**
-     * Returns the port on which this connector operates.
+     * Returns the port number on which this connector listens. Note that the implementation needs to account for any form
+     * of port-offsetting that might have an effect on the configured port.
      *
      * @return a port number
      */

diff --git a/...iner/tomcat-10.1/src/main/java/org/jboss/modcluster/container/tomcat/TomcatConnector.java b/...iner/tomcat-10.1/src/main/java/org/jboss/modcluster/container/tomcat/TomcatConnector.java
@@ -78,7 +78,7 @@ public void setAddress(InetAddress address) {
 
     @Override
     public int getPort() {
-        return (this.externalPort == null) ? this.connector.getPort() : this.externalPort;
+        return (this.externalPort == null) ? this.connector.getPortWithOffset() : this.externalPort;
     }
 
     @Override
@@ -112,7 +112,7 @@ public int hashCode() {
     @Override
     public String toString() {
         InetAddress address = this.getAddress();
-        return String.format("%s://%s:%d", this.getType(), (address != null) ? address.getHostAddress() : "<undefined>", this.connector.getPort());
+        return String.format("%s://%s:%d", this.getType(), (address != null) ? address.getHostAddress() : "<undefined>", this.connector.getPortWithOffset());
     }
 
     /**

diff --git a/...ainer/tomcat-8.5/src/main/java/org/jboss/modcluster/container/tomcat/TomcatConnector.java b/...ainer/tomcat-8.5/src/main/java/org/jboss/modcluster/container/tomcat/TomcatConnector.java
@@ -78,7 +78,7 @@ public void setAddress(InetAddress address) {
 
     @Override
     public int getPort() {
-        return (this.externalPort == null) ? this.connector.getPort() : this.externalPort;
+        return (this.externalPort == null) ? this.connector.getPortWithOffset() : this.externalPort;
     }
 
     @Override
@@ -112,7 +112,7 @@ public int hashCode() {
     @Override
     public String toString() {
         InetAddress address = this.getAddress();
-        return String.format("%s://%s:%d", this.getType(), (address != null) ? address.getHostAddress() : "<undefined>", this.connector.getPort());
+        return String.format("%s://%s:%d", this.getType(), (address != null) ? address.getHostAddress() : "<undefined>", this.connector.getPortWithOffset());
     }
 
     /**

diff --git a/...ainer/tomcat-9.0/src/main/java/org/jboss/modcluster/container/tomcat/TomcatConnector.java b/...ainer/tomcat-9.0/src/main/java/org/jboss/modcluster/container/tomcat/TomcatConnector.java
@@ -78,7 +78,7 @@ public void setAddress(InetAddress address) {
 
     @Override
     public int getPort() {
-        return (this.externalPort == null) ? this.connector.getPort() : this.externalPort;
+        return (this.externalPort == null) ? this.connector.getPortWithOffset() : this.externalPort;
     }
 
     @Override
@@ -112,7 +112,7 @@ public int hashCode() {
     @Override
     public String toString() {
         InetAddress address = this.getAddress();
-        return String.format("%s://%s:%d", this.getType(), (address != null) ? address.getHostAddress() : "<undefined>", this.connector.getPort());
+        return String.format("%s://%s:%d", this.getType(), (address != null) ? address.getHostAddress() : "<undefined>", this.connector.getPortWithOffset());
     }
 
     /**