This is an automated email from the ASF dual-hosted git repository.
morningman pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/master by this push:
new 2a54405ca67 [fix](show frontends) show effective HTTP/HTTPS port
(#64670)
2a54405ca67 is described below
commit 2a54405ca67080a08f239eef81e6b574a7d8e424
Author: nsivarajan <[email protected]>
AuthorDate: Tue Jul 7 12:26:00 2026 +0530
[fix](show frontends) show effective HTTP/HTTPS port (#64670)
`show frontends;` always show only `http_port` even though for cases
like when enable_https=true, this PR fix the port visibility to show
HTTPS port if `enable_https` enabled.
Also, this serves as preparatory for #60921
Co-authored-by: Sivarajan Narayanan <[email protected]>
---
.../main/java/org/apache/doris/common/proc/FrontendsProcNode.java | 3 ++-
.../src/main/java/org/apache/doris/common/util/HttpURLUtil.java | 5 +++++
2 files changed, 7 insertions(+), 1 deletion(-)
diff --git
a/fe/fe-core/src/main/java/org/apache/doris/common/proc/FrontendsProcNode.java
b/fe/fe-core/src/main/java/org/apache/doris/common/proc/FrontendsProcNode.java
index 4b7adbf9638..58fc49ae4c3 100644
---
a/fe/fe-core/src/main/java/org/apache/doris/common/proc/FrontendsProcNode.java
+++
b/fe/fe-core/src/main/java/org/apache/doris/common/proc/FrontendsProcNode.java
@@ -21,6 +21,7 @@ import org.apache.doris.catalog.Env;
import org.apache.doris.common.Config;
import org.apache.doris.common.Pair;
import org.apache.doris.common.io.DiskUtils;
+import org.apache.doris.common.util.HttpURLUtil;
import org.apache.doris.common.util.TimeUtils;
import org.apache.doris.qe.ConnectContext;
import org.apache.doris.service.FeDiskInfo;
@@ -123,7 +124,7 @@ public class FrontendsProcNode implements ProcNodeInterface
{
info.add(fe.getNodeName());
info.add(fe.getHost());
info.add(Integer.toString(fe.getEditLogPort()));
- info.add(Integer.toString(Config.http_port));
+ info.add(Integer.toString(HttpURLUtil.getHttpPort()));
if (fe.getNodeName().equals(env.getNodeName())) {
info.add(Integer.toString(Config.query_port));
diff --git
a/fe/fe-core/src/main/java/org/apache/doris/common/util/HttpURLUtil.java
b/fe/fe-core/src/main/java/org/apache/doris/common/util/HttpURLUtil.java
index 70b3e68450a..baafc104bf9 100644
--- a/fe/fe-core/src/main/java/org/apache/doris/common/util/HttpURLUtil.java
+++ b/fe/fe-core/src/main/java/org/apache/doris/common/util/HttpURLUtil.java
@@ -19,6 +19,7 @@ package org.apache.doris.common.util;
import org.apache.doris.catalog.Env;
import org.apache.doris.cloud.security.SecurityChecker;
+import org.apache.doris.common.Config;
import org.apache.doris.system.SystemInfoService.HostInfo;
import com.google.common.collect.Maps;
@@ -30,6 +31,10 @@ import java.util.Map;
public class HttpURLUtil {
+ public static int getHttpPort() {
+ return Config.enable_https ? Config.https_port : Config.http_port;
+ }
+
public static HttpURLConnection getConnectionWithNodeIdent(String request)
throws IOException {
try {
SecurityChecker.getInstance().startSSRFChecking(request);
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
