chore: code polishing

Author: eigmax

sdk/src/network/prover.rs

@@ -33,14 +33,15 @@ pub struct NetworkProver {
 
 impl NetworkProver {
     pub async fn new(client_config: &ClientCfg) -> anyhow::Result<NetworkProver> {
-        let ca_cert_path = client_config.ca_cert_path.to_owned().expect("CA_CERT_PATH must be set");
-        let cert_path = client_config.cert_path.to_owned().expect("CERT_PATH must be set");
-        let key_path = client_config.key_path.to_owned().expect("KEY_PATH must be set");
-        let ssl_config = if ca_cert_path.is_empty() {
+        let ssl_config = if client_config.ca_cert_path.as_ref().is_none() {
             None
         } else {
-            let (ca_cert, identity) =
-                get_cert_and_identity(ca_cert_path, cert_path, key_path).await?;
+            let (ca_cert, identity) = get_cert_and_identity(
+                client_config.ca_cert_path.as_ref().expect("CA_CERT_PATH not set"),
+                client_config.cert_path.as_ref().expect("CERT_PATH not set"),
+                client_config.key_path.as_ref().expect("KEY_PATH not set"),
+            )
+            .await?;
             Some(Config { ca_cert, identity })
         };
         let endpoint_para = client_config.endpoint.to_owned().expect("ENDPOINT must be set");
@@ -228,13 +229,13 @@ impl Prover for NetworkProver {
 }
 
 async fn get_cert_and_identity(
-    ca_cert_path: String,
-    cert_path: String,
-    key_path: String,
+    ca_cert_path: &str,
+    cert_path: &str,
+    key_path: &str,
 ) -> anyhow::Result<(Option<Certificate>, Option<Identity>)> {
-    let ca_cert_path = Path::new(&ca_cert_path);
-    let cert_path = Path::new(&cert_path);
-    let key_path = Path::new(&key_path);
+    let ca_cert_path = Path::new(ca_cert_path);
+    let cert_path = Path::new(cert_path);
+    let key_path = Path::new(key_path);
     // if !ca_cert_path.is_file() || !cert_path.is_file() || !key_path.is_file() {
     //     bail!("both ca_cert_path, cert_path and key_path should be valid file")
     // }

sdk/src/prover.rs

@@ -32,10 +32,7 @@ impl ClientCfg {
         let snark_setup =
             env::var("SNARK_SETUP").ok().and_then(|seg| seg.parse::<bool>().ok()).unwrap_or(false);
 
-        let guest_input = match env::var("ARGS") {
-            Ok(input) => Some(input),
-            _ => None,
-        };
+        let guest_input = env::var("ARGS").ok();
         let elf_path = env::var("ELF_PATH").expect("ELF not found");
         println!("{}", elf_path);
         let proof_results_path =