51l3nt51n/51l3nt51n-blog
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

refactor: Use Redirect::to instead of Redirect::temporary for all redirects.

Browse Source
This commit is contained in:
51l3nt51n
2026-03-03 16:11:48 +00:00
parent 50a3caf33c
commit 626e76adba
2 changed files with 8 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
src/handlers/admin.rs
View File

@@ -89,7 +89,7 @@ pub async fn add_user(
.await .await
.context("Failed to insert new user into database")?; .context("Failed to insert new user into database")?;
Ok(Redirect::temporary("/__dungeon").into_response()) Ok(Redirect::to("/__dungeon").into_response())
} }
pub async fn delete_user( pub async fn delete_user(
@@ -111,7 +111,7 @@ pub async fn delete_user(
.await .await
.context(format!("Failed to delete user with id {}", id))?; .context(format!("Failed to delete user with id {}", id))?;
Ok(Redirect::temporary("/__dungeon").into_response()) Ok(Redirect::to("/__dungeon").into_response())
} }
#[derive(Deserialize)] #[derive(Deserialize)]
@@ -138,5 +138,5 @@ pub async fn change_password(
.await .await
.context(format!("Failed to update password for user with id {}", id))?; .context(format!("Failed to update password for user with id {}", id))?;
Ok(Redirect::temporary("/__dungeon").into_response()) Ok(Redirect::to("/__dungeon").into_response())
} }

10
src/handlers/auth.rs
View File

@@ -45,7 +45,7 @@ pub async fn setup_form(State(state): State<Arc<AppState>>) -> Result<Response,
.context("Failed to check user count for setup form")?; .context("Failed to check user count for setup form")?;
if count.0 > 0 { if count.0 > 0 {
return Ok(Redirect::temporary("/__dungeon/login").into_response()); return Ok(Redirect::to("/__dungeon/login").into_response());
} }
Ok(HtmlTemplate(SetupTemplate { error: None }).into_response()) Ok(HtmlTemplate(SetupTemplate { error: None }).into_response())
@@ -67,7 +67,7 @@ pub async fn setup_submit(
.context("Failed to check user count for setup submit")?; .context("Failed to check user count for setup submit")?;
if count.0 > 0 { if count.0 > 0 {
return Ok(Redirect::temporary("/__dungeon/login").into_response()); return Ok(Redirect::to("/__dungeon/login").into_response());
} }
let hashed = hash(&payload.password, DEFAULT_COST).context("Failed to hash password")?; let hashed = hash(&payload.password, DEFAULT_COST).context("Failed to hash password")?;
@@ -80,7 +80,7 @@ pub async fn setup_submit(
.await .await
.context("Failed to insert initial admin user")?; .context("Failed to insert initial admin user")?;
Ok(Redirect::temporary("/__dungeon/login").into_response()) Ok(Redirect::to("/__dungeon/login").into_response())
} }
pub async fn login_form() -> impl IntoResponse { pub async fn login_form() -> impl IntoResponse {
@@ -119,7 +119,7 @@ pub async fn login_submit(
let mut cookie = Cookie::new("dungeon_session", session_id); let mut cookie = Cookie::new("dungeon_session", session_id);
cookie.set_path("/"); cookie.set_path("/");
return Ok((cookie_jar.add(cookie), Redirect::temporary("/__dungeon")).into_response()); return Ok((cookie_jar.add(cookie), Redirect::to("/__dungeon")).into_response());
} }
} }
@@ -147,7 +147,7 @@ pub async fn logout_submit(
Ok(( Ok((
cookie_jar.remove(remove_cookie), cookie_jar.remove(remove_cookie),
Redirect::temporary("/__dungeon/login"), Redirect::to("/__dungeon/login"),
) )
.into_response()) .into_response())
} }