Merge pull request #2465 from ljedrz/pk_further_tweaks

Forbid empty private-key args
ProvableHQ · Aug 17, 2023 · ede1870 · ede1870
2 parents da84380 + 28f1c5e
commit ede1870
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/cli/src/commands/start.rs b/cli/src/commands/start.rs
@@ -175,9 +175,13 @@ impl Start {
             bail!("Both the private key string and file path flags were specified, please pick only one");
         }
 
-        // If the private key is provided directly, don't do anything else.
-        if self.private_key.is_some() {
-            return Ok(());
+        // If the private key is provided directly, don't do anything else; make sure it's non-empty too.
+        if let Some(private_key) = &self.private_key {
+            if private_key.is_empty() {
+                bail!("The private-key argument can't be empty");
+            } else {
+                return Ok(());
+            }
         }
 
         // If a filesystem path to the private key is provided, attempt to