Fix issues flagged during review.

README.md

@@ -13,8 +13,7 @@ Krill is a Resource Public Key Infrastructure (RPKI) daemon, featuring a
 Certificate Authority (CA) and publication server, written in Rust.  If you have
 any feedback, we would love to hear from you. Don’t hesitate to [create an issue
 on Github](https://github.com/NLnetLabs/krill/issues/new) or post a message on
-our [RPKI mailing list](https://lists.nlnetlabs.nl/mailman/listinfo/rpki) or
-[Discord server](https://discord.gg/8dvKB5Ykhy). 
+our [forum](https://community.nlnetlabs.nl/c/rpki/11).
 
 For more information please refer to the [documentation](https://krill.docs.nlnetlabs.nl/en/stable/).
 

src/commons/crypto/signing/signers/pkcs11/context.rs

@@ -262,7 +262,7 @@ impl Pkcs11Context {
         })
     }
 
-    fn finalize(&mut self) -> Result<(), Pkcs11Error>{
+    fn finalize(&mut self) -> Result<(), Pkcs11Error> {
         self.logged_cryptoki_call_with_take("Finalize", |cryptoki| {
             cryptoki.finalize()
         })

src/daemon/start.rs

@@ -110,7 +110,6 @@ pub fn start_krill_daemon(
         properties_manager.init(KrillVersion::code_version())?;
     }
 
-    // XXX TODO This may need some configuration.
     let tokio = tokio::runtime::Runtime::new().map_err(|err| {
         KrillError::custom(
             format!("Failed to create Tokio runtime: {err}")

src/server/ca/child.rs

@@ -396,7 +396,7 @@ pub struct ChildCertificateUpdates {
     #[serde(skip_serializing_if = "Vec::is_empty", default)]
     pub suspended: Vec<SuspendedCert>,
 
-    /// The certificats that have been unsuspended.
+    /// The certificates that have been unsuspended.
     ///
     /// This is no longer used as of Krill 0.16.0, but kept because it is in
     /// stored state.

src/server/ca/keys.rs

@@ -79,7 +79,7 @@ impl CertifiedKey {
     /// Updates the certificate received for the key.
     pub fn set_incoming_cert(&mut self, cert: ReceivedCert) {
         self.request = None;
-        self.incoming_cert = cert
+        self.incoming_cert = cert;
     }
 
     /// Returns the certified key info for this certified key.

src/server/taproxy.rs

@@ -784,7 +784,6 @@ impl TrustAnchorProxy {
 /// a simple call to `into`.
 #[derive(Clone, Copy)]
 pub struct TrustAnchorProxyContext<'a> {
-    #[allow(dead_code)] // XXX remove!!
     tasks: &'a TaskQueue,
     signer: &'a KrillSigner,
 }

tests/auth_check.rs

@@ -12,8 +12,6 @@ mod common;
 async fn auth_check() {
     let server = common::KrillServer::start().await;
 
-    eprintln!("server is up.");
-
     // Get a client with a changed auth token.
     let client = KrillClient::new(
         server.server_uri().clone(), 
@@ -33,7 +31,6 @@ async fn auth_check() {
             )
         )
     );
-    eprintln!("back.");
 }
 
 #[tokio::test]