keyfork: remove env_logger

Cargo.lock

@@ -709,19 +709,6 @@ version = "0.3.6"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "a357d28ed41a50f9c765dbfe56cbc04a64e53e5fc58ba79fbc34c10ef3df831f"
 
-[[package]]
-name = "env_logger"
-version = "0.10.1"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "95b3f3e67048839cb0d0781f445682a35113da7121f7c949db0e2be96a4fbece"
-dependencies = [
- "humantime",
- "is-terminal",
- "log",
- "regex",
- "termcolor",
-]
-
 [[package]]
 name = "equivalent"
 version = "1.0.1"
@@ -928,12 +915,6 @@ dependencies = [
  "digest",
 ]
 
-[[package]]
-name = "humantime"
-version = "2.1.0"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "9a3a5bfb195931eeb336b2a7b4d761daec841b97f947d34394601737a7bba5e4"
-
 [[package]]
 name = "iana-time-zone"
 version = "0.1.59"
@@ -1061,7 +1042,6 @@ version = "0.1.0"
 dependencies = [
  "card-backend-pcsc",
  "clap",
- "env_logger",
  "keyfork-derive-openpgp",
  "keyfork-derive-util",
  "keyfork-mnemonic-util",
@@ -2284,15 +2264,6 @@ dependencies = [
  "winapi",
 ]
 
-[[package]]
-name = "termcolor"
-version = "1.4.0"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "ff1bc3d3f05aff0403e8ac0d92ced918ec05b666a43f83297ccef5bea8a3d449"
-dependencies = [
- "winapi-util",
-]
-
 [[package]]
 name = "terminal_size"
 version = "0.3.0"
@@ -2636,15 +2607,6 @@ version = "0.4.0"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "ac3b87c63620426dd9b991e5ce0329eff545bccbbb34f3be09ff6fb6ab51b7b6"
 
-[[package]]
-name = "winapi-util"
-version = "0.1.6"
-source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "f29e6f9198ba0d26b4c9f07dbe6f9ed633e1f3d5b8b414090084349e46a52596"
-dependencies = [
- "winapi",
-]
-
 [[package]]
 name = "winapi-x86_64-pc-windows-gnu"
 version = "0.4.0"

keyfork/Cargo.toml

@@ -23,4 +23,3 @@ card-backend-pcsc = "0.5.0"
 openpgp-card-sequoia = "0.2.0"
 openpgp-card = "0.4.1"
 keyfork-prompt = { version = "0.1.0", path = "../keyfork-prompt" }
-env_logger = "0.10.1"

keyfork/src/cli/wizard.rs

@@ -1,13 +1,9 @@
 use super::Keyfork;
 use clap::{Parser, Subcommand};
+use std::collections::HashSet;
 
 use card_backend_pcsc::PcscBackend;
-use openpgp_card::card_do::ApplicationIdentifier;
-use openpgp_card_sequoia::{
-    state::Open,
-    types::KeyType,
-    Card,
-};
+use openpgp_card_sequoia::{state::Open, types::KeyType, Card};
 
 use keyfork_derive_openpgp::openpgp::{self, packet::UserID, types::KeyFlags, Cert};
 use keyfork_derive_util::{
@@ -49,7 +45,7 @@ fn derive_key(seed: &[u8], index: u8) -> Result<Cert> {
 // TODO: extract into crate
 /// Factory reset the current card so long as it does not match the last-used backend.
 fn factory_reset_current_card(
-    previous_backend: &mut Option<ApplicationIdentifier>,
+    seen_cards: &mut HashSet<String>,
     cert: &Cert,
 ) -> Result<()> {
     let policy = openpgp::policy::NullPolicy::new();
@@ -75,12 +71,12 @@ fn factory_reset_current_card(
     if let Some(current_backend) = PcscBackend::cards(None)?.next().transpose()? {
         let mut card = Card::<Open>::new(current_backend)?;
         let mut transaction = card.transaction()?;
-        let application_identifier = transaction.application_identifier()?;
-        if previous_backend.is_some_and(|pb| pb == application_identifier) {
+        let application_identifier = transaction.application_identifier()?.ident();
+        if seen_cards.contains(&application_identifier) {
             // we were given the same card, error
             panic!("Previously used card {application_identifier} was reused");
         } else {
-            let _ = previous_backend.insert(application_identifier);
+            seen_cards.insert(application_identifier);
         }
         transaction.factory_reset()?;
         let mut admin = transaction.to_admin_card("12345678")?;
@@ -98,15 +94,17 @@ fn generate_shard_secret(threshold: u8, max: u8, keys_per_shard: u8) -> Result<(
     let seed = keyfork_plumbing::generate_entropy_of_size(256 / 8)?;
     let mut pm = PromptManager::new(std::io::stdin(), std::io::stderr())?;
     let mut certs = vec![];
-    let mut last_card = None;
+    let mut seen_cards: HashSet<String> = HashSet::new();
 
     for index in 0..max {
         let cert = derive_key(&seed, index)?;
         for i in 0..keys_per_shard {
             pm.prompt_message(&Message::Text(format!(
-                "Please insert user {index} key {i}"
+                "Please insert key #{} for user #{}",
+                i + 1,
+                index + 1,
             )))?;
-            factory_reset_current_card(&mut last_card, &cert)?;
+            factory_reset_current_card(&mut seen_cards, &cert)?;
         }
         certs.push(cert);
     }

keyfork/src/main.rs

@@ -9,8 +9,6 @@ mod cli;
 fn main() -> ExitCode {
     let opts = cli::Keyfork::parse();
 
-    env_logger::init();
-
     if let Err(e) = opts.command .handle(&opts) {
         println!("Unable to run command: {e}");
         let mut source = e.source();