Change default socket addresses to localhost

Author: Dzejkop

README.md

@@ -18,12 +18,13 @@ cargo run -- [OPTIONS]
 - `--builder-url <URL>`: URL of the builder execution engine (required)
 - `--builder-jwt-token <TOKEN>`: JWT token for builder authentication (required)
 - `--builder-jwt-path <PATH>`: Path to the builder JWT secret file (required if `--builder-jwt-token` is not provided)
-- `--rpc-host <HOST>`: Host to run the server on (default: 0.0.0.0)
+- `--rpc-host <HOST>`: Host to run the server on (default: 127.0.0.1)
 - `--rpc-port <PORT>`: Port to run the server on (default: 8081)
 - `--tracing`: Enable tracing (default: false)
 - `--log-level <LEVEL>`: Log level (default: info)
 - `--log-format <FORMAT>`: Log format (default: text)
 - `--metrics`: Enable metrics (default: false)
+- `--metrics-host <METRICS_HOST>`: Host to run the metrics server on (default: 127.0.0.1)
 - `--no-boost-sync`: Disables using the proposer to sync the builder node (default: true)
 - `--debug-host <HOST>`: Host to run the server on (default: 127.0.0.1)
 - `--debug-server-port <PORT>`: Port to run the debug server on (default: 5555)

src/cli.rs

@@ -40,7 +40,7 @@ pub struct Args {
     pub no_boost_sync: bool,
 
     /// Host to run the server on
-    #[arg(long, env, default_value = "0.0.0.0")]
+    #[arg(long, env, default_value = "127.0.0.1")]
     pub rpc_host: String,
 
     /// Port to run the server on
@@ -56,7 +56,7 @@ pub struct Args {
     pub metrics: bool,
 
     /// Host to run the metrics server on
-    #[arg(long, env, default_value = "0.0.0.0")]
+    #[arg(long, env, default_value = "127.0.0.1")]
     pub metrics_host: String,
 
     /// Port to run the metrics server on

src/client/rpc.rs

@@ -395,7 +395,7 @@ pub mod tests {
 
     use super::*;
 
-    const AUTH_ADDR: &str = "0.0.0.0";
+    const AUTH_ADDR: &str = "127.0.0.1";
     const SECRET: &str = "f79ae8046bc11c9927afe911db7143c51a806c4a537cc08e0d37140b0192f430";
 
     pub fn get_available_port() -> u16 {

src/health.rs

@@ -101,7 +101,7 @@ mod tests {
                 + 'static,
         {
             {
-                let listener = TcpListener::bind("0.0.0.0:0").await?;
+                let listener = TcpListener::bind("127.0.0.1:0").await?;
                 let addr = listener.local_addr()?;
 
                 let handle = tokio::spawn(async move {

src/proxy.rs

@@ -206,7 +206,7 @@ mod tests {
                 // None,
             ));
 
-            let temp_listener = TcpListener::bind("0.0.0.0:0").await?;
+            let temp_listener = TcpListener::bind("127.0.0.1:0").await?;
             let server_addr = temp_listener.local_addr()?;
             drop(temp_listener);
 
@@ -247,7 +247,7 @@ mod tests {
 
     impl MockHttpServer {
         async fn serve() -> eyre::Result<Self> {
-            let listener = TcpListener::bind("0.0.0.0:0").await?;
+            let listener = TcpListener::bind("127.0.0.1:0").await?;
             let addr = listener.local_addr()?;
             let requests = Arc::new(Mutex::new(vec![]));
 

src/server.rs

@@ -728,12 +728,12 @@ mod tests {
     use std::sync::Arc;
     use tokio::time::sleep;
 
-    const HOST: &str = "0.0.0.0";
+    const HOST: &str = "127.0.0.1";
     const L2_PORT: u16 = 8545;
     const L2_ADDR: &str = "127.0.0.1:8545";
     const BUILDER_PORT: u16 = 8544;
     const BUILDER_ADDR: &str = "127.0.0.1:8544";
-    const SERVER_ADDR: &str = "0.0.0.0:8556";
+    const SERVER_ADDR: &str = "127.0.0.1:8556";
 
     #[derive(Debug, Clone)]
     pub struct MockEngineServer {
@@ -851,7 +851,7 @@ mod tests {
 
             let server = Server::builder()
                 .set_http_middleware(http_middleware)
-                .build("0.0.0.0:8556".parse::<SocketAddr>().unwrap())
+                .build("127.0.0.1:8556".parse::<SocketAddr>().unwrap())
                 .await
                 .unwrap()
                 .start(module);