Merge pull request #1477 from cloudflare/josh/dev-toml

Added dev args to wrangler.toml

src/main.rs

@@ -751,17 +751,31 @@ fn run() -> Result<(), failure::Error> {
         commands::preview(target, user, options, verbose)?;
     } else if let Some(matches) = matches.subcommand_matches("dev") {
         log::info!("Starting dev server");
-        let port: Option<u16> = matches
-            .value_of("port")
-            .map(|p| p.parse().expect("--port expects a number"));
-        let host = matches.value_of("host");
-        let ip = matches.value_of("ip");
+
         let config_path = Path::new(
             matches
                 .value_of("config")
                 .unwrap_or(commands::DEFAULT_CONFIG_PATH),
         );
         let manifest = settings::toml::Manifest::new(config_path)?;
+
+        let host: Option<&str> = matches.value_of("host");
+        let mut ip: Option<&str> = matches.value_of("ip");
+        let mut port: Option<u16> = matches
+            .value_of("port")
+            .map(|p| p.parse().expect("--port expects a number"));
+
+        // Check if arg not given but present in wrangler.toml
+        if let Some(d) = &manifest.dev {
+            if ip.is_none() && d.ip.is_some() {
+                ip = d.ip.as_deref();
+            }
+
+            if port.is_none() && d.port.is_some() {
+                port = d.port;
+            }
+        }
+
         let env = matches.value_of("env");
         let deploy_config = manifest.deploy_config(env)?;
         is_preview = true;

src/settings/toml/dev.rs

@@ -0,0 +1,8 @@
+use serde::{Deserialize, Serialize};
+
+#[derive(Clone, Debug, Deserialize, PartialEq, Serialize)]
+#[serde(deny_unknown_fields)]
+pub struct Dev {
+    pub ip: Option<String>,
+    pub port: Option<u16>,
+}

src/settings/toml/manifest.rs

@@ -11,6 +11,7 @@ use serde_with::rust::string_empty_as_none;
 
 use crate::commands::{validate_worker_name, DEFAULT_CONFIG_PATH};
 use crate::settings::toml::deploy_config::{DeployConfig, RouteConfig};
+use crate::settings::toml::dev::Dev;
 use crate::settings::toml::environment::Environment;
 use crate::settings::toml::kv_namespace::{ConfigKvNamespace, KvNamespace};
 use crate::settings::toml::site::Site;
@@ -37,6 +38,7 @@ pub struct Manifest {
     // TODO: maybe one day, serde toml support will allow us to serialize sites
     // as a TOML inline table (this would prevent confusion with environments too!)
     pub site: Option<Site>,
+    pub dev: Option<Dev>,
     #[serde(alias = "kv-namespaces")]
     pub kv_namespaces: Option<Vec<ConfigKvNamespace>>,
     pub env: Option<HashMap<String, Environment>>,

src/settings/toml/mod.rs

@@ -1,4 +1,5 @@
 mod deploy_config;
+mod dev;
 mod environment;
 mod kv_namespace;
 mod manifest;