fetch salt only once

src/config.rs

@@ -8,17 +8,16 @@ use std::collections::HashMap;
 use std::env;
 use std::fs::File;
 use std::io::Read;
-use std::path::{PathBuf, Path};
+use std::path::{Path, PathBuf};
 use std::process::Command;
 
-
 #[derive(Debug, Deserialize, Serialize)]
 pub struct EnvConfig {
     credential_id: String,
     uuid: String,
     salt: String,
     mapper_name: String,
-    password_helper: String
+    password_helper: String,
 }
 
 impl Into<Config> for EnvConfig {
@@ -29,14 +28,15 @@ impl Into<Config> for EnvConfig {
             mapper_name: self.mapper_name,
             password_helper: PasswordHelper::Script(self.password_helper),
             input_salt: if PathBuf::from(&self.salt).exists() {
-                InputSalt::File { path: self.salt.into() }
+                InputSalt::File {
+                    path: self.salt.into(),
+                }
             } else {
                 InputSalt::AskPassword
+            },
         }
     }
 }
-}
-
 
 #[derive(Debug, Deserialize, Serialize)]
 pub struct Config {

src/main.rs

@@ -94,8 +94,6 @@ fn main() -> Fido2LuksResult<()> {
         ))
     };
     if args.is_empty() {
-        let salt = conf.input_salt.obtain(&conf.password_helper)?;
-        dbg!(hex::encode(&salt));
         if env.contains_key("CRYPTTAB_NAME") {
             //Indicates that this script is being run as keyscript
             let mut out = stdout();
@@ -109,7 +107,13 @@ fn main() -> Fido2LuksResult<()> {
         match args.first().map(|s| s.as_ref()).unwrap() {
             "addkey" => add_key_to_luks(&Config::load_default_location()?).map(|_| ()),
             "setup" => setup(),
-            "open" if args.get(1).map(|a| &*a == "-e").unwrap_or(false) => open(&envy::prefixed("FIDO2LUKS_").from_env::<EnvConfig>().expect("Missing env config values").into(), &secret()?),
+            "open" if args.get(1).map(|a| &*a == "-e").unwrap_or(false) => open(
+                &envy::prefixed("FIDO2LUKS_")
+                    .from_env::<EnvConfig>()
+                    .expect("Missing env config values")
+                    .into(),
+                &secret()?,
+            ),
             "open" => open(&conf, &secret()?),
             "connected" => match authenticator_connected()? {
                 false => {