refactor(backend): replace String with &str in fn params
This commit is contained in:
@@ -22,7 +22,7 @@ async fn login(
|
||||
cookies: Cookies,
|
||||
Json(payload): Json<LoginPayload>,
|
||||
) -> Result<StatusCode, LoftError> {
|
||||
let user = user_repository.find_by_username(payload.username).await?;
|
||||
let user = user_repository.find_by_username(&payload.username).await?;
|
||||
// TODO: replace unwrap with ?
|
||||
let parsed_hash = PasswordHash::new(&user.password_hash).unwrap();
|
||||
if Argon2::default()
|
||||
@@ -31,8 +31,8 @@ async fn login(
|
||||
{
|
||||
let expires_at = chrono::Utc::now() + chrono::Duration::days(1);
|
||||
let cookie = create_cookie();
|
||||
let auth_token = cookie.value().to_string();
|
||||
cookies.add(cookie);
|
||||
let auth_token = cookie.value();
|
||||
cookies.add(cookie.clone());
|
||||
user_repository
|
||||
.create_session(user.id, auth_token, expires_at)
|
||||
.await?;
|
||||
@@ -50,7 +50,7 @@ async fn logout(
|
||||
let auth_token = cookies.get(AUTH_TOKEN).map(|c| c.value().to_string());
|
||||
|
||||
if let Some(auth_token) = auth_token {
|
||||
user_repository.delete_session(auth_token.clone()).await?;
|
||||
user_repository.delete_session(&auth_token).await?;
|
||||
cookies.remove(Cookie::build(AUTH_TOKEN).path("/").build());
|
||||
}
|
||||
|
||||
@@ -62,7 +62,7 @@ async fn register(
|
||||
Json(payload): Json<LoginPayload>,
|
||||
) -> Result<StatusCode, LoftError> {
|
||||
if user_repository
|
||||
.find_by_username(payload.username.clone())
|
||||
.find_by_username(&payload.username)
|
||||
.await
|
||||
.is_ok()
|
||||
{
|
||||
@@ -78,7 +78,7 @@ async fn register(
|
||||
.to_string();
|
||||
|
||||
user_repository
|
||||
.create_user(payload.username, password_hash)
|
||||
.create_user(&payload.username, &password_hash)
|
||||
.await?;
|
||||
|
||||
Ok(StatusCode::CREATED)
|
||||
|
||||
Reference in New Issue
Block a user