Merge pull request #192 from cloudflare/check-for-keys

Check for keys
cloudflare · Jun 1, 2019 · efd91f4 · efd91f4
2 parents 2d0ca35 + 28d1e98
commit efd91f4
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 0 deletions.
diff --git a/src/commands/publish/mod.rs b/src/commands/publish/mod.rs
@@ -83,6 +83,9 @@ fn publish_script(
     project: &Project,
     release: bool,
 ) -> Result<(), failure::Error> {
+    if project.account_id.is_empty() {
+        failure::bail!("You must provide an account_id in your wrangler.toml before you publish!")
+    }
     let worker_addr = format!(
         "https://api.cloudflare.com/client/v4/accounts/{}/workers/scripts/{}",
         project.account_id, project.name,

diff --git a/src/commands/publish/route.rs b/src/commands/publish/route.rs
@@ -18,6 +18,15 @@ struct RoutesResponse {
 
 impl Route {
     pub fn new(project: &Project) -> Result<Route, failure::Error> {
+        if project
+            .route
+            .clone()
+            .expect("You must provide a zone_id in your wrangler.toml before publishing!")
+            .is_empty()
+        {
+            failure::bail!("You must provide a zone_id in your wrangler.toml before publishing!");
+        }
+
         Ok(Route {
             script: Some(project.name.to_string()),
             pattern: project.route.clone().expect("⚠️ Your project config has an error, check your `wrangler.toml`: `route` must be provided.").to_string(),