feat: use caching on user lookup

this needs to be deleted/expired on user update, we'll implement this when we get ways to "update" things like channels, servers and users
This commit is contained in:
Radical 2025-05-07 20:33:23 +02:00
parent 7ecc8c4270
commit 529ccd1b51

View file

@ -1,11 +1,12 @@
use actix_web::{Error, HttpRequest, HttpResponse, get, web}; use actix_web::{Error, HttpRequest, HttpResponse, get, web};
use log::error; use log::error;
use serde::Serialize; use serde::Serialize;
use tokio::sync::Mutex;
use uuid::Uuid; use uuid::Uuid;
use crate::{Data, api::v1::auth::check_access_token, utils::get_auth_header}; use crate::{Data, api::v1::auth::check_access_token, utils::get_auth_header};
#[derive(Serialize)] #[derive(Serialize, Clone)]
struct Response { struct Response {
uuid: String, uuid: String,
username: String, username: String,
@ -34,6 +35,12 @@ pub async fn res(
return Ok(error); return Ok(error);
} }
let cache_result = data.get_cache_key(uuid.to_string()).await;
if let Ok(cache_hit) = cache_result {
return Ok(HttpResponse::Ok().json(cache_hit))
}
let row = sqlx::query_as(&format!( let row = sqlx::query_as(&format!(
"SELECT username, display_name FROM users WHERE uuid = '{}'", "SELECT username, display_name FROM users WHERE uuid = '{}'",
uuid uuid
@ -48,9 +55,18 @@ pub async fn res(
let (username, display_name): (String, Option<String>) = row.unwrap(); let (username, display_name): (String, Option<String>) = row.unwrap();
Ok(HttpResponse::Ok().json(Response { let user = Response {
uuid: uuid.to_string(), uuid: uuid.to_string(),
username, username,
display_name: display_name.unwrap_or_default(), display_name: display_name.unwrap_or_default(),
})) };
let cache_result = data.set_cache_key(uuid.to_string(), user.clone(), 1800).await;
if let Err(error) = cache_result {
error!("{}", error);
return Ok(HttpResponse::InternalServerError().finish());
}
Ok(HttpResponse::Ok().json(user))
} }