From d8c095200a6a18c44614e195deacc9c8687faee0 Mon Sep 17 00:00:00 2001
From: Kewe63 <kewe63@users.noreply.github.com>
Date: Mon, 11 May 2026 15:32:45 +0300
Subject: [PATCH] feat: auto-detect public IP for geth P2P peer discovery

Previously, HOST_IP was hardcoded to empty string and users had to
manually edit the entrypoint to enable --nat=extip. This change:

- Adds the same get_public_ip() function used by base-consensus-entrypoint
  and op-node-entrypoint (tries ifconfig.me, api.ipify.org, ipecho.net,
  v4.ident.me in sequence)
- Auto-detects public IP if HOST_IP is not manually set
- Falls back gracefully with a warning if detection fails
- Respects manual HOST_IP override when set via environment
---
 geth/geth-entrypoint | 33 +++++++++++++++++++++++++++++++--
 1 file changed, 31 insertions(+), 2 deletions(-)

diff --git a/geth/geth-entrypoint b/geth/geth-entrypoint
index b598cb3f4..9cafe9c7b 100755
--- a/geth/geth-entrypoint
+++ b/geth/geth-entrypoint
@@ -7,7 +7,7 @@ RPC_PORT="${RPC_PORT:-8545}"
 WS_PORT="${WS_PORT:-8546}"
 AUTHRPC_PORT="${AUTHRPC_PORT:-8551}"
 METRICS_PORT="${METRICS_PORT:-6060}"
-HOST_IP="" # put your external IP address here and open port 30303 to improve peer connectivity
+HOST_IP="" # Will be auto-detected; set manually to override
 P2P_PORT="${P2P_PORT:-30303}"
 DISCOVERY_PORT="${DISCOVERY_PORT:-30303}"
 ADDITIONAL_ARGS=""
@@ -26,7 +26,36 @@ if [[ -z "$OP_NODE_NETWORK" ]]; then
     exit 1
 fi
 
-mkdir -p $GETH_DATA_DIR
+# Auto-detect public IP for P2P peer discovery if not manually set
+get_public_ip() {
+  local PROVIDERS=(
+    "http://ifconfig.me"
+    "http://api.ipify.org"
+    "http://ipecho.net/plain"
+    "http://v4.ident.me"
+  )
+  for provider in "${PROVIDERS[@]}"; do
+    local IP
+    IP=$(curl -s --max-time 10 --connect-timeout 5 "$provider")
+    if [[ $IP =~ ^[0-9]+\.[0-9]+\.[0-9]+\.[0-9]+$ ]]; then
+      echo "$IP"
+      return 0
+    fi
+  done
+  return 1
+}
+
+# If HOST_IP is not manually set (still empty), try auto-detection
+if [[ -z "$HOST_IP" ]]; then
+  if PUBLIC_IP=$(get_public_ip); then
+    echo "Auto-detected public IP: $PUBLIC_IP"
+    HOST_IP="$PUBLIC_IP"
+  else
+    echo "Could not auto-detect public IP. Peer connectivity may be limited." >&2
+  fi
+fi
+
+mkdir -p "$GETH_DATA_DIR"
 
 echo "$BASE_NODE_L2_ENGINE_AUTH_RAW" > "$BASE_NODE_L2_ENGINE_AUTH"