chore(*): remove or update obsolete TODOs

blah-types/src/identity.rs

@@ -89,7 +89,6 @@ impl UserIdentityDesc {
     }
 }
 
-// TODO: JWS or alike?
 #[derive(Debug, Clone, PartialEq, Eq, Serialize, Deserialize)]
 #[serde(tag = "typ", rename = "user_act_key")]
 pub struct UserActKeyDesc {

blahd/Cargo.toml

@@ -12,7 +12,7 @@ anyhow = "1"
 axum = { version = "0.7", features = ["ws"] }
 axum-extra = "0.9"
 clap = { version = "4", features = ["derive"] }
-data-encoding = "2" # TODO: Also use it for hex?
+data-encoding = "2"
 ed25519-dalek = "2"
 futures-util = "0.3"
 hex = { version = "0.4", features = ["serde"] }

blahd/config.example.toml

@@ -24,7 +24,8 @@ address = "localhost:8080"
 
 # Use systemd socket activation mechanism to get listener fd from envvars.
 # See also sd_listen_fds(3) and systemd.socket(5).
-# NB. Currently only TCP sockets are supported. UNIX domain socket is TODO.
+# NB. Currently only TCP sockets are supported. UNIX domain socket support is
+# blocked by axum 0.5, see https://github.com/tokio-rs/axum/pull/2941
 #systemd = true
 
 [server]

blahd/src/feed.rs

@@ -176,7 +176,6 @@ impl fmt::Display for AtomFeed {
                 r#"  <link rel="next" type="application/atom+xml" href="{esc_next_url}"/>"#
             )?;
         }
-        // TODO: HTML links.
 
         for msg in msgs {
             let content = &msg.msg.signee.payload.rich_text;

blahd/tests/webapi.rs

@@ -417,7 +417,7 @@ fn server() -> Server {
     server_with(db, &CONFIG)
 }
 
-// TODO: Testing config is hard to build because it does have a `Default` impl.
+// TODO: Testing config is hard to build because it lacks a `Default` impl.
 #[track_caller]
 fn server_with(db: Database, config: &dyn Fn(u16) -> String) -> Server {
     // Use std's to avoid async, since we need no name resolution.
@@ -1215,7 +1215,6 @@ async fn register_flow(server: Server) {
     req.server_url = BASE_URL.parse().unwrap();
 
     // Trailing dot in id_url.
-    // TODO: Rule this out in `IdUrl` parser?
     register_fast(&req)
         .await
         .expect_api_err(StatusCode::FORBIDDEN, "disabled");

contrib/module.nix

@@ -31,7 +31,7 @@ let
   settingsType = types.submodule {
     freeformType = toml.type;
 
-    # TODO: Auto-generate these options? Now only required options are documented.
+    # TODO: Auto-generate these options?
     options = { };
   };