Compare commits
2 commits
d615f1392e
...
60f0219e85
Author | SHA1 | Date | |
---|---|---|---|
60f0219e85 | |||
38aab46534 |
6 changed files with 42 additions and 17 deletions
|
@ -11,7 +11,7 @@ use crate::{
|
|||
error::Error,
|
||||
schema::*,
|
||||
utils::{
|
||||
PASSWORD_REGEX, generate_access_token, generate_refresh_token, refresh_token_cookie,
|
||||
PASSWORD_REGEX, generate_access_token, generate_refresh_token, new_refresh_token_cookie,
|
||||
user_uuid_from_identifier,
|
||||
},
|
||||
};
|
||||
|
@ -89,6 +89,6 @@ pub async fn response(
|
|||
.await?;
|
||||
|
||||
Ok(HttpResponse::Ok()
|
||||
.cookie(refresh_token_cookie(refresh_token))
|
||||
.cookie(new_refresh_token_cookie(refresh_token))
|
||||
.json(Response { access_token }))
|
||||
}
|
||||
|
|
31
src/api/v1/auth/logout.rs
Normal file
31
src/api/v1/auth/logout.rs
Normal file
|
@ -0,0 +1,31 @@
|
|||
use actix_web::{HttpRequest, HttpResponse, post, web};
|
||||
use diesel::{ExpressionMethods, delete};
|
||||
use diesel_async::RunQueryDsl;
|
||||
|
||||
use crate::{
|
||||
Data,
|
||||
error::Error,
|
||||
schema::refresh_tokens::{self, dsl},
|
||||
};
|
||||
|
||||
// TODO: Should maybe be a delete request?
|
||||
#[post("/logout")]
|
||||
pub async fn res(
|
||||
req: HttpRequest,
|
||||
data: web::Data<Data>,
|
||||
) -> Result<HttpResponse, Error> {
|
||||
let mut refresh_token_cookie = req.cookie("refresh_token").ok_or(Error::Unauthorized("request has no refresh token".to_string()))?;
|
||||
|
||||
let refresh_token = String::from(refresh_token_cookie.value());
|
||||
|
||||
let mut conn = data.pool.get().await?;
|
||||
|
||||
delete(refresh_tokens::table)
|
||||
.filter(dsl::token.eq(refresh_token))
|
||||
.execute(&mut conn)
|
||||
.await?;
|
||||
|
||||
refresh_token_cookie.make_removal();
|
||||
|
||||
Ok(HttpResponse::Ok().cookie(refresh_token_cookie).finish())
|
||||
}
|
|
@ -9,6 +9,7 @@ use uuid::Uuid;
|
|||
use crate::{Conn, error::Error, schema::access_tokens::dsl};
|
||||
|
||||
mod login;
|
||||
mod logout;
|
||||
mod refresh;
|
||||
mod register;
|
||||
mod reset_password;
|
||||
|
@ -24,6 +25,7 @@ pub fn web() -> Scope {
|
|||
web::scope("/auth")
|
||||
.service(register::res)
|
||||
.service(login::response)
|
||||
.service(logout::res)
|
||||
.service(refresh::res)
|
||||
.service(revoke::res)
|
||||
.service(verify_email::get)
|
||||
|
|
|
@ -11,20 +11,16 @@ use crate::{
|
|||
access_tokens::{self, dsl},
|
||||
refresh_tokens::{self, dsl as rdsl},
|
||||
},
|
||||
utils::{generate_access_token, generate_refresh_token, refresh_token_cookie},
|
||||
utils::{generate_access_token, generate_refresh_token, new_refresh_token_cookie},
|
||||
};
|
||||
|
||||
use super::Response;
|
||||
|
||||
#[post("/refresh")]
|
||||
pub async fn res(req: HttpRequest, data: web::Data<Data>) -> Result<HttpResponse, Error> {
|
||||
let recv_refresh_token_cookie = req.cookie("refresh_token");
|
||||
let mut refresh_token_cookie = req.cookie("refresh_token").ok_or(Error::Unauthorized("request has no refresh token".to_string()))?;
|
||||
|
||||
if recv_refresh_token_cookie.is_none() {
|
||||
return Ok(HttpResponse::Unauthorized().finish());
|
||||
}
|
||||
|
||||
let mut refresh_token = String::from(recv_refresh_token_cookie.unwrap().value());
|
||||
let mut refresh_token = String::from(refresh_token_cookie.value());
|
||||
|
||||
let current_time = SystemTime::now().duration_since(UNIX_EPOCH)?.as_secs() as i64;
|
||||
|
||||
|
@ -47,8 +43,6 @@ pub async fn res(req: HttpRequest, data: web::Data<Data>) -> Result<HttpResponse
|
|||
error!("{}", error);
|
||||
}
|
||||
|
||||
let mut refresh_token_cookie = refresh_token_cookie(refresh_token);
|
||||
|
||||
refresh_token_cookie.make_removal();
|
||||
|
||||
return Ok(HttpResponse::Unauthorized()
|
||||
|
@ -91,12 +85,10 @@ pub async fn res(req: HttpRequest, data: web::Data<Data>) -> Result<HttpResponse
|
|||
.await?;
|
||||
|
||||
return Ok(HttpResponse::Ok()
|
||||
.cookie(refresh_token_cookie(refresh_token))
|
||||
.cookie(new_refresh_token_cookie(refresh_token))
|
||||
.json(Response { access_token }));
|
||||
}
|
||||
|
||||
let mut refresh_token_cookie = refresh_token_cookie(refresh_token);
|
||||
|
||||
refresh_token_cookie.make_removal();
|
||||
|
||||
Ok(HttpResponse::Unauthorized()
|
||||
|
|
|
@ -21,7 +21,7 @@ use crate::{
|
|||
},
|
||||
utils::{
|
||||
EMAIL_REGEX, PASSWORD_REGEX, USERNAME_REGEX, generate_access_token, generate_refresh_token,
|
||||
refresh_token_cookie,
|
||||
new_refresh_token_cookie,
|
||||
},
|
||||
};
|
||||
|
||||
|
@ -146,7 +146,7 @@ pub async fn res(
|
|||
.await?;
|
||||
|
||||
return Ok(HttpResponse::Ok()
|
||||
.cookie(refresh_token_cookie(refresh_token))
|
||||
.cookie(new_refresh_token_cookie(refresh_token))
|
||||
.json(Response { access_token }));
|
||||
}
|
||||
|
||||
|
|
|
@ -100,7 +100,7 @@ pub fn get_ws_protocol_header(headers: &HeaderMap) -> Result<&str, Error> {
|
|||
Ok(auth_value.unwrap())
|
||||
}
|
||||
|
||||
pub fn refresh_token_cookie(refresh_token: String) -> Cookie<'static> {
|
||||
pub fn new_refresh_token_cookie(refresh_token: String) -> Cookie<'static> {
|
||||
Cookie::build("refresh_token", refresh_token)
|
||||
.http_only(true)
|
||||
.secure(true)
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue