respond to John's comments on 5685

wicket-common/src/rack_setup.rs

@@ -335,7 +335,7 @@ impl<T: fmt::Display> fmt::Display for DisplaySlice<'_, T> {
 /// Describes the actual authentication key to use with a BGP peer.
 ///
 /// Currently, only TCP-MD5 authentication is supported.
-#[derive(Clone, Debug, Serialize, Deserialize, PartialEq, Eq, JsonSchema)]
+#[derive(Clone, Serialize, Deserialize, PartialEq, Eq, JsonSchema)]
 #[serde(tag = "kind", rename_all = "snake_case")]
 pub enum BgpAuthKey {
     /// TCP-MD5 authentication.
@@ -358,6 +358,17 @@ impl BgpAuthKey {
     }
 }
 
+// Ensure that the key is not displayed in debug output.
+impl fmt::Debug for BgpAuthKey {
+    fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
+        match self {
+            BgpAuthKey::TcpMd5 { key: _ } => {
+                f.debug_struct("TcpMd5").field("key", &"********").finish()
+            }
+        }
+    }
+}
+
 /// Describes insensitive information about a BGP authentication key.
 ///
 /// This information is considered okay to display in the UI.

wicket/src/cli/rack_setup.rs

@@ -351,8 +351,7 @@ impl SetBgpAuthKeyArgs {
         };
 
         eprintln!(
-            "\nsetting {} {} to use {} authentication \
-             (press Ctrl-C to cancel)",
+            "\nsetting {} {} to use {} authentication",
             display_count.style(styles.bold),
             keys_str(len),
             self.auth_method.style(styles.bold),