src/cli/edit_command.rs

@@ -75,7 +75,7 @@ pub struct Edit {
 
 impl LprsCommand for Edit {
     fn run(self, mut vault_manager: Vaults) -> LprsResult<()> {
-        let vault = match utils::vault_by_index_or_name(&self.location, &mut vault_manager.vaults) {
+        let vault = match utils::vault_by_index_or_name(self.location, &mut vault_manager.vaults) {
             Ok((_, v)) => v,
             Err(err) => {
                 if self.force {

src/cli/get_command.rs

@@ -112,7 +112,7 @@ pub struct Get {
 impl LprsCommand for Get {
     fn run(self, mut vault_manager: Vaults) -> LprsResult<()> {
         let (index, vault) =
-            utils::vault_by_index_or_name(&self.location, &mut vault_manager.vaults)?;
+            utils::vault_by_index_or_name(self.location, &mut vault_manager.vaults)?;
 
         if let Some(field) = self.field {
             if field == VaultGetField::Index {

src/utils.rs

@@ -19,11 +19,7 @@ use std::num::NonZeroUsize;
 use std::{fs, path::PathBuf};
 
 use either::Either;
-use inquire::{
-    validator::{StringValidator, Validation},
-    Password,
-    PasswordDisplayMode,
-};
+use inquire::{validator::Validation, Password, PasswordDisplayMode};
 use passwords::{analyzer, scorer};
 #[cfg(feature = "update-notify")]
 use reqwest::blocking::Client as BlockingClient;
@@ -51,26 +47,6 @@ pub fn local_project_file(filename: &str) -> LprsResult<PathBuf> {
     Ok(local_dir.join(filename))
 }
 
-/// Ask the user for a secret in the stdin
-///
-/// ## Errors
-/// - If can't read the user input
-fn secret_prompt(
-    prompt_message: &str,
-    confirmation: bool,
-    validators: Option<Vec<Box<dyn StringValidator>>>,
-) -> LprsResult<String> {
-    Password {
-        validators: validators.unwrap_or_default(),
-        enable_confirmation: confirmation,
-        ..Password::new(prompt_message)
-            .with_formatter(&|p| "*".repeat(p.chars().count()))
-            .with_display_mode(PasswordDisplayMode::Masked)
-    }
-    .prompt()
-    .map_err(LprsError::Inquire)
-}
-
 /// Returns the user secret if any
 ///
 /// - If the `secret` is `None` will return `None`
@@ -90,7 +66,15 @@ pub fn user_secret(
         Some(Some(p)) => Some(p),
         Some(None) => {
             log::debug!("User didn't provide a secret, prompting it");
-            Some(secret_prompt(prompt_message, confirmation, None)?)
+            Some(
+                Password {
+                    enable_confirmation: confirmation,
+                    ..Password::new(prompt_message)
+                        .with_formatter(&|p| "*".repeat(p.chars().count()))
+                        .with_display_mode(PasswordDisplayMode::Masked)
+                }
+                .prompt()?,
+            )
         }
     })
 }
@@ -135,12 +119,21 @@ pub fn password_validator(password: &str) -> Result<Validation, inquire::CustomU
 ///
 /// Return's the password as 32 bytes after hash it (256 bit)
 pub fn master_password_prompt(is_new_vaults_file: bool) -> LprsResult<[u8; 32]> {
-    secret_prompt(
-        "Master Password:",
-        is_new_vaults_file,
-        Some(vec![Box::new(password_validator)]),
-    )
+    inquire::Password {
+        message: "Master Password:",
+        enable_confirmation: is_new_vaults_file,
+        validators: if is_new_vaults_file {
+            vec![Box::new(password_validator)]
+        } else {
+            vec![]
+        },
+        ..inquire::Password::new("")
+    }
+    .with_formatter(&|p| "*".repeat(p.chars().count()))
+    .with_display_mode(PasswordDisplayMode::Masked)
+    .prompt()
     .map(|p| sha2::Sha256::digest(p).into())
+    .map_err(Into::into)
 }
 
 /// Retuns the current lprs version from `crates.io`
@@ -232,7 +225,7 @@ pub fn prompt_custom(
         if let Some(value) = value {
             new_fields.push((key, value));
         } else {
-            let value = secret_prompt(&format!("Value of `{key}`:"), false, None)?;
+            let value = inquire::Text::new(&format!("Value of `{key}`:")).prompt()?;
             new_fields.push((key, value));
         }
     }
@@ -244,17 +237,16 @@ pub fn prompt_custom(
 ///
 /// ## Errors
 /// - If there is no vault with the given index or name
-pub fn vault_by_index_or_name<'v>(
-    location: &Either<NonZeroUsize, String>,
-    vaults: &'v mut [Vault],
-) -> LprsResult<(usize, &'v mut Vault)> {
+pub fn vault_by_index_or_name(
+    location: Either<NonZeroUsize, String>,
+    vaults: &mut [Vault],
+) -> LprsResult<(usize, &mut Vault)> {
     let idx = location
-        .as_ref()
         .map_right(|name| {
             vaults
                 .iter()
                 .enumerate()
-                .find_map(|(idx, v)| (&v.name == name).then_some(idx))
+                .find_map(|(idx, v)| (v.name == name).then_some(idx))
                 .ok_or_else(|| {
                     LprsError::Other(format!("There is no vault with the given name `{name}`"))
                 })
@@ -265,10 +257,7 @@ pub fn vault_by_index_or_name<'v>(
     Ok((
         idx,
         vaults.get_mut(idx).ok_or_else(|| {
-            LprsError::Other(format!(
-                "There is no vault with the given index `{}`",
-                idx + 1
-            ))
+            LprsError::Other(format!("There is no vault with the given index `{idx}`"))
         })?,
     ))
 }

src/vault/mod.rs

@@ -44,7 +44,7 @@ pub enum Format {
 }
 
 /// The vault struct
-#[derive(Clone, Debug, Deserialize, Serialize, Parser, Eq, PartialEq)]
+#[derive(Clone, Debug, Deserialize, Serialize, Parser)]
 pub struct Vault {
     /// The name of the vault
     pub name:          String,