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, 54 deletions

diff --git a/src/user/routes/password/post.rs b/src/user/routes/password/post.rs
deleted file mode 100644
index 296f6cd..0000000
--- a/src/user/routes/password/post.rs
+++ /dev/null
@@ -1,54 +0,0 @@
-use axum::{
-    extract::{Json, State},
-    http::StatusCode,
-    response::{IntoResponse, Response},
-};
-
-use crate::{
-    app::App,
-    clock::RequestedAt,
-    error::Internal,
-    token::{
-        app,
-        extract::{Identity, IdentityCookie},
-    },
-    user::{Password, User},
-};
-
-pub async fn handler(
-    State(app): State<App>,
-    RequestedAt(now): RequestedAt,
-    identity: Identity,
-    cookie: IdentityCookie,
-    Json(request): Json<Request>,
-) -> Result<(IdentityCookie, Json<User>), Error> {
-    let (login, secret) = app
-        .tokens()
-        .change_password(&identity.user, &request.password, &request.to, &now)
-        .await
-        .map_err(Error)?;
-    let cookie = cookie.set(secret);
-    Ok((cookie, Json(login)))
-}
-
-#[derive(serde::Deserialize)]
-pub struct Request {
-    pub password: Password,
-    pub to: Password,
-}
-
-#[derive(Debug, thiserror::Error)]
-#[error(transparent)]
-pub struct Error(#[from] pub app::LoginError);
-
-impl IntoResponse for Error {
-    fn into_response(self) -> Response {
-        let Self(error) = self;
-        match error {
-            app::LoginError::Rejected => {
-                (StatusCode::BAD_REQUEST, "invalid name or password").into_response()
-            }
-            other => Internal::from(other).into_response(),
-        }
-    }
-}