Handlers are _named operations_, which can be exposed via routes.

Each domain module that exposes handlers does so through a `handlers` child module, ideally as a top-level symbol that can be plugged directly into Axum's `MethodRouter`. Modules could make exceptions to this - kill the doctrinaire inside yourself, after all - but none of the API modules that actually exist need such exceptions, and consistency is useful.

The related details of request types, URL types, response types, errors, &c &c are then organized into modules under `handlers`, along with their respective tests.

1 files changed, 0 insertions, 52 deletions

diff --git a/src/setup/routes/post.rs b/src/setup/routes/post.rs
deleted file mode 100644
index 0ff5d69..0000000
--- a/src/setup/routes/post.rs
+++ /dev/null
@@ -1,52 +0,0 @@
-use axum::{
-    extract::{Json, State},
-    http::StatusCode,
-    response::{IntoResponse, Response},
-};
-
-use crate::{
-    app::App,
-    clock::RequestedAt,
-    error::Internal,
-    name::Name,
-    setup::app,
-    token::extract::IdentityCookie,
-    user::{Password, User},
-};
-
-pub async fn handler(
-    State(app): State<App>,
-    RequestedAt(setup_at): RequestedAt,
-    identity: IdentityCookie,
-    Json(request): Json<Request>,
-) -> Result<(IdentityCookie, Json<User>), Error> {
-    let (user, secret) = app
-        .setup()
-        .initial(&request.name, &request.password, &setup_at)
-        .await
-        .map_err(Error)?;
-    let identity = identity.set(secret);
-    Ok((identity, Json(user)))
-}
-
-#[derive(serde::Deserialize)]
-pub struct Request {
-    pub name: Name,
-    pub password: Password,
-}
-
-#[derive(Debug)]
-pub struct Error(pub app::Error);
-
-impl IntoResponse for Error {
-    fn into_response(self) -> Response {
-        let Self(error) = self;
-        match error {
-            app::Error::InvalidName(_) => {
-                (StatusCode::BAD_REQUEST, error.to_string()).into_response()
-            }
-            app::Error::SetupCompleted => (StatusCode::CONFLICT, error.to_string()).into_response(),
-            other => Internal::from(other).into_response(),
-        }
-    }
-}