remove unnecessary todos

Author: lonerapier

mfkdf2/src/definitions/mfkdf_derived_key/strengthening.rs

@@ -12,7 +12,6 @@ impl MFKDF2DerivedKey {
 
     let salt = general_purpose::STANDARD.decode(&self.policy.salt).unwrap();
 
-    // TODO (@lonerapier): what if the policy has a stack key?
     Argon2::new(
       argon2::Algorithm::Argon2id,
       Version::default(),

mfkdf2/src/derive/key.rs

@@ -24,7 +24,7 @@ pub fn key(
   let mut factors = factors;
   let mut new_policy = policy.clone();
 
-  for factor in &mut new_policy.factors {
+  for factor in &new_policy.factors {
     let Some(material) = factors.get_mut(factor.id.as_str()) else {
       shares_bytes.push(None);
       continue;

mfkdf2/src/setup/factors/totp.rs

@@ -33,16 +33,14 @@ pub struct TOTPOptions {
 
 impl Default for TOTPOptions {
   fn default() -> Self {
-    let now_ms = SystemTime::now().duration_since(UNIX_EPOCH).unwrap().as_millis() as u64;
-
     Self {
       id:     Some("totp".to_string()),
       secret: None,
       digits: Some(6),
       hash:   Some(HashAlgorithm::Sha1),
       issuer: Some("MFKDF".to_string()),
       label:  Some("mfkdf.com".to_string()),
-      time:   Some(now_ms), // TODO: see if this can be None
+      time:   None,
       window: Some(87600),
       step:   Some(30),
       oracle: None,