Bump base32

Cargo.toml

@@ -28,7 +28,7 @@ rsa_stoken = ["stoken"]
 [dependencies]
 rust-crypto = "^0"
 byteorder = "^1.4"
-base32 = "^0"
+base32 = "^0.5"
 toml = "^0"
 serde = { version = "^1.0", features = ["derive"] }
 structopt = "^0"

src/config.rs

@@ -7,7 +7,7 @@ use crate::totp::TokenAlgorithm;
 use crate::{TotpConfigError, TotpResult};
 use serde::{self, Deserialize, Serialize};
 
-#[derive(Debug, Deserialize, Serialize, Clone)]
+#[derive(Debug, Deserialize, Serialize, Clone, Default)]
 pub struct Config {
     totp: HashMap<String, TotpOptions>,
 }
@@ -73,14 +73,6 @@ impl TotpOptions {
     }
 }
 
-impl Default for Config {
-    fn default() -> Self {
-        Config {
-            totp: HashMap::new(),
-        }
-    }
-}
-
 impl Config {
     pub fn lookup(&self, name: &str) -> TotpResult<&TotpOptions> {
         Ok(self

src/lib.rs

@@ -72,7 +72,7 @@ pub fn add_totp_secret<P: AsRef<Path>>(
     name: &str,
     secret: String,
 ) -> TotpResult<()> {
-    base32::decode(base32::Alphabet::RFC4648 { padding: false }, &secret)
+    base32::decode(base32::Alphabet::Rfc4648 { padding: false }, &secret)
         .expect("Invalid base32 OTP secret");
 
     add_secret(&config, config_dir, name, secret, TokenAlgorithm::TotpSha1).map(|_| ())
@@ -107,7 +107,7 @@ pub fn add_secret<P: AsRef<Path>>(
     secret: String,
     algorithm: TokenAlgorithm,
 ) -> TotpResult<Config> {
-    let totp_options = secrets::store_secret(&name, &secret, algorithm)?;
+    let totp_options = secrets::store_secret(name, &secret, algorithm)?;
     let mut config: Config = config.clone();
     config.insert(name.to_string(), totp_options);
     let string = toml::to_string(&config)?;

src/otpcli/main.rs

@@ -30,7 +30,7 @@ fn main() -> Result<(), Box<dyn Error>> {
             Ok(())
         }
         Command::AddSecret { name, secret } => {
-            otp::add_totp_secret(config, config_dir, &name, secret.replace(" ", ""))?;
+            otp::add_totp_secret(config, config_dir, &name, secret.replace(' ', ""))?;
             Ok(())
         }
         #[cfg(feature = "ras_stoken")]

src/totp.rs

@@ -12,7 +12,7 @@ use serde::{Deserialize, Serialize};
 
 use super::secrets;
 
-static ALPHABET: base32::Alphabet = base32::Alphabet::RFC4648 { padding: false };
+static ALPHABET: base32::Alphabet = base32::Alphabet::Rfc4648 { padding: false };
 
 /// [RFC6238-timestep]: https://tools.ietf.org/html/rfc6238#section-5.2
 /// [RFC6238 recommended][RFC6238-timestep] time step duration of 30 seconds.
@@ -29,6 +29,7 @@ pub enum TokenAlgorithm {
 
 impl Copy for TokenAlgorithm {}
 
+#[allow(dead_code)]
 trait AsDigest {
     fn as_digest(&self) -> Box<dyn Digest>;
 }
@@ -63,13 +64,13 @@ impl AsDigest for TokenAlgorithm {
 ///
 /// ```
 pub fn standard_totp(name: &str, options: &TotpOptions) -> TotpResult<String> {
-    let secret = secrets::get_secret(name, &options)?;
+    let secret = secrets::get_secret(name, options)?;
     generate_sha1_code(secret)
 }
 
 /// Cleans a base32 secret by removing spaces and making sure it's upper-cased.
 pub fn clean_secret(secret: &str) -> String {
-    secret.replace(" ", "").to_uppercase()
+    secret.replace(' ', "").to_uppercase()
 }
 
 /// Generate a SHA1 TOTP code
@@ -91,7 +92,7 @@ pub fn generate_sha1_code(secret: String) -> TotpResult<String> {
         .duration_since(SystemTime::UNIX_EPOCH)
         .expect("Can't get time since UNIX_EPOCH?");
 
-    let clean_secret = secret.replace(" ", "").to_uppercase();
+    let clean_secret = secret.replace(' ', "").to_uppercase();
     let secret = base32::decode(ALPHABET, &clean_secret)
         .ok_or(TotpError("Failed to decode secret from base32"))?;