feat: modernized host/port setting for perseus serve

Added `--host`/`--port` and renamed environment variables to
`PERSEUS_HOST` and `PERSEUS_PORT`. Note that the originals are still
supported in deployment binaries.

Closes #107.

Author: arctic-hen7

examples/basic/.perseus/server/src/main.rs

@@ -68,9 +68,16 @@ fn get_standalone_and_act() -> bool {
 
 /// Gets the host and port to serve on.
 fn get_host_and_port() -> (String, u16) {
-    let host = env::var("HOST").unwrap_or_else(|_| "127.0.0.1".to_string());
-    let port = env::var("PORT")
-        .unwrap_or_else(|_| "8080".to_string())
+    // We have to use two sets of environment variables until v0.4.0
+    // TODO Remove the old environment variables in v0.4.0
+    let host_old = env::var("HOST");
+    let port_old = env::var("PORT");
+    let host = env::var("PERSEUS_HOST");
+    let port = env::var("PERSEUS_PORT");
+
+    let host = host.unwrap_or_else(|_| host_old.unwrap_or_else(|_| "127.0.0.1".to_string()));
+    let port = port
+        .unwrap_or_else(|_| port_old.unwrap_or_else(|_| "8080".to_string()))
         .parse::<u16>()
         .expect("Port must be a number.");
 

packages/perseus-cli/src/deploy.rs

@@ -35,6 +35,9 @@ fn deploy_full(dir: PathBuf, output: String, integration: Integration) -> Result
             standalone: true,
             integration,
             watch: false,
+            // These have no impact if `no_run` is `true` (which it is), so we can use the defaults here
+            host: "127.0.0.1".to_string(),
+            port: 8080,
         },
     )?;
     if serve_exit_code != 0 {

packages/perseus-cli/src/parse.rs

@@ -105,6 +105,12 @@ pub struct ServeOpts {
     /// Watch the files in your working directory for changes (exluding `target/` and `.perseus/`)
     #[clap(short, long)]
     pub watch: bool,
+    /// Where to host your exported app
+    #[clap(long, default_value = "127.0.0.1")]
+    pub host: String,
+    /// The port to host your exported app on
+    #[clap(long, default_value = "8080")]
+    pub port: u16,
 }
 /// Removes `.perseus/` entirely for updates or to fix corruptions
 #[derive(Parser)]
@@ -162,4 +168,10 @@ pub struct SnoopServeOpts {
     /// The server integration to use
     #[clap(short, long, default_value = "warp")]
     pub integration: Integration,
+    /// Where to host your exported app
+    #[clap(long, default_value = "127.0.0.1")]
+    pub host: String,
+    /// The port to host your exported app on
+    #[clap(long, default_value = "8080")]
+    pub port: u16,
 }

packages/perseus-cli/src/serve.rs

@@ -163,15 +163,15 @@ fn run_server(
             cmd: server_exec_path,
             source: err,
         })?;
-    // Figure out what host/port the app will be live on
-    let host = env::var("HOST").unwrap_or_else(|_| "localhost".to_string());
-    let port = env::var("PORT")
+    // Figure out what host/port the app will be live on (these have been set by the system)
+    let host = env::var("PERSEUS_HOST").unwrap_or_else(|_| "localhost".to_string());
+    let port = env::var("PERSEUS_PORT")
         .unwrap_or_else(|_| "8080".to_string())
         .parse::<u16>()
         .map_err(|err| ExecutionError::PortNotNumber { source: err })?;
     // Give the user a nice informational message
     println!(
-        "  {} {} Your app is now live on <http://{host}:{port}>! To change this, re-run this command with different settings of the HOST/PORT environment variables.",
+        "  {} {} Your app is now live on <http://{host}:{port}>! To change this, re-run this command with different settings for `--host` and `--port`.",
         style(format!("[{}/{}]", num_steps, num_steps)).bold().dim(),
         SERVING,
         host=host,
@@ -199,6 +199,10 @@ fn run_server(
 /// Builds the subcrates to get a directory that we can serve and then serves it. If possible, this will return the path to the server
 /// executable so that it can be used in deployment.
 pub fn serve(dir: PathBuf, opts: ServeOpts) -> Result<(i32, Option<String>), ExecutionError> {
+    // Set the environment variables for the host and port
+    env::set_var("PERSEUS_HOST", opts.host);
+    env::set_var("PERSEUS_PORT", opts.port.to_string());
+
     let spinners = MultiProgress::new();
     let did_build = !opts.no_build;
     let should_run = !opts.no_run;

packages/perseus-cli/src/snoop.rs

@@ -66,6 +66,10 @@ pub fn snoop_wasm_build(dir: PathBuf, opts: SnoopWasmOpts) -> Result<i32, Execut
 
 /// Runs the commands to run the server directly so the user can see detailed logs.
 pub fn snoop_server(dir: PathBuf, opts: SnoopServeOpts) -> Result<i32, ExecutionError> {
+    // Set the environment variables for the host and port
+    env::set_var("PERSEUS_HOST", opts.host);
+    env::set_var("PERSEUS_PORT", opts.port.to_string());
+
     let target = dir.join(".perseus/server");
     run_cmd_directly(
         format!(