phoenix/icarus_auth
1
0
Fork 0
Code Issues 4 Pull Requests Actions Packages Projects Releases 52 Wiki Activity

tsk-55: Register endpoint bug fix #72

Merged
phoenix merged 4 commits from tsk-55 into main 2025-10-20 16:05:30 +00:00
Conversation 0 Commits 4 Files Changed 4 +11 -13
4 changed files with 11 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
Cargo.lock generated
View File

@@ -748,7 +748,7 @@ dependencies = [
[[package]] [[package]]
name = "icarus_auth" name = "icarus_auth"
version = "0.6.1" version = "0.6.2"
dependencies = [ dependencies = [
"argon2", "argon2",
"axum", "axum",

2
Cargo.toml
View File

@@ -1,6 +1,6 @@
[package] [package]
name = "icarus_auth" name = "icarus_auth"
version = "0.6.1" version = "0.6.2"
edition = "2024" edition = "2024"
rust-version = "1.90" rust-version = "1.90"

12
src/callers/register.rs
View File

@@ -53,7 +53,6 @@ pub async fn register_user(
Json(payload): Json<request::Request>, Json(payload): Json<request::Request>,
) -> (StatusCode, Json<response::Response>) { ) -> (StatusCode, Json<response::Response>) {
let mut user = icarus_models::user::User { let mut user = icarus_models::user::User {
id: uuid::Uuid::nil(),
username: payload.username.clone(), username: payload.username.clone(),
password: payload.password.clone(), password: payload.password.clone(),
email: payload.email.clone(), email: payload.email.clone(),
@@ -62,19 +61,17 @@ pub async fn register_user(
lastname: payload.lastname.clone(), lastname: payload.lastname.clone(),
status: String::from("Active"), status: String::from("Active"),
email_verified: true, email_verified: true,
date_created: Some(time::OffsetDateTime::now_utc()), ..Default::default()
last_login: None,
salt_id: uuid::Uuid::nil(),
}; };
match repo::user::exists(&pool, &user.username).await { match repo::user::exists(&pool, &user.username).await {
Ok(res) => { Ok(res) => {
if res { if res {
( (
StatusCode::NOT_FOUND, StatusCode::BAD_REQUEST,
Json(response::Response { Json(response::Response {
message: String::from("Error"), message: String::from("Error"),
data: vec![user], data: Vec::new(),
}), }),
) )
} else { } else {
@@ -89,8 +86,9 @@ pub async fn register_user(
user.password = hashed_password; user.password = hashed_password;
match repo::user::insert(&pool, &user).await { match repo::user::insert(&pool, &user).await {
Ok(id) => { Ok((id, date_created)) => {
user.id = id; user.id = id;
user.date_created = date_created;
( (
StatusCode::CREATED, StatusCode::CREATED,
Json(response::Response { Json(response::Response {

8
src/repo/mod.rs
View File

@@ -94,7 +94,7 @@ pub mod user {
pub async fn insert( pub async fn insert(
pool: &sqlx::PgPool, pool: &sqlx::PgPool,
user: &icarus_models::user::User, user: &icarus_models::user::User,
) -> Result<uuid::Uuid, sqlx::Error> { ) -> Result<(uuid::Uuid, std::option::Option<time::OffsetDateTime>), sqlx::Error> {
let row = sqlx::query( let row = sqlx::query(
r#" r#"
INSERT INTO "user" (username, password, email, phone, firstname, lastname, email_verified, status, salt_id) INSERT INTO "user" (username, password, email, phone, firstname, lastname, email_verified, status, salt_id)
@@ -124,10 +124,10 @@ pub mod user {
.map_err(|_e| sqlx::Error::RowNotFound)?, .map_err(|_e| sqlx::Error::RowNotFound)?,
}; };
if !result.id.is_nil() { if result.id.is_nil() && result.date_created.is_none() {
Ok(result.id)
} else {
Err(sqlx::Error::RowNotFound) Err(sqlx::Error::RowNotFound)
} else {
Ok((result.id, result.date_created))
} }
} }
} }