feat: support OIDC id_token
- generate JWT id_token in token exchange - store optional nonce in authorization object - switch to RS256 algorithm for JWT signature - add JWKs endpoint to provide OIDC clients with public keys
This commit is contained in:
parent
ca84a0f99f
commit
d982f2af0d
29 changed files with 255 additions and 82 deletions
1
.env
1
.env
|
|
@ -1 +0,0 @@
|
||||||
APP_JWT_SECRET=bc1996ea-5464-424a-9a38-5604f2bc865a
|
|
||||||
BIN
.swp
Normal file
BIN
.swp
Normal file
Binary file not shown.
49
Cargo.lock
generated
49
Cargo.lock
generated
|
|
@ -349,6 +349,12 @@ dependencies = [
|
||||||
"windows-targets 0.52.6",
|
"windows-targets 0.52.6",
|
||||||
]
|
]
|
||||||
|
|
||||||
|
[[package]]
|
||||||
|
name = "base64"
|
||||||
|
version = "0.13.1"
|
||||||
|
source = "registry+https://github.com/rust-lang/crates.io-index"
|
||||||
|
checksum = "9e1b586273c5702936fe7b7d6896644d8be71e6314cfe09d3167c95f712589e8"
|
||||||
|
|
||||||
[[package]]
|
[[package]]
|
||||||
name = "base64"
|
name = "base64"
|
||||||
version = "0.21.7"
|
version = "0.21.7"
|
||||||
|
|
@ -966,11 +972,13 @@ dependencies = [
|
||||||
"chrono",
|
"chrono",
|
||||||
"env_logger",
|
"env_logger",
|
||||||
"fully_pub",
|
"fully_pub",
|
||||||
|
"jsonwebkey-convert",
|
||||||
"jsonwebtoken",
|
"jsonwebtoken",
|
||||||
"kernel",
|
"kernel",
|
||||||
"log",
|
"log",
|
||||||
"minijinja",
|
"minijinja",
|
||||||
"minijinja-embed",
|
"minijinja-embed",
|
||||||
|
"pem 3.0.4",
|
||||||
"serde",
|
"serde",
|
||||||
"serde_json",
|
"serde_json",
|
||||||
"serde_urlencoded",
|
"serde_urlencoded",
|
||||||
|
|
@ -1238,6 +1246,20 @@ dependencies = [
|
||||||
"wasm-bindgen",
|
"wasm-bindgen",
|
||||||
]
|
]
|
||||||
|
|
||||||
|
[[package]]
|
||||||
|
name = "jsonwebkey-convert"
|
||||||
|
version = "0.3.0"
|
||||||
|
dependencies = [
|
||||||
|
"base64 0.13.1",
|
||||||
|
"lazy_static",
|
||||||
|
"num-bigint",
|
||||||
|
"pem 0.8.3",
|
||||||
|
"serde",
|
||||||
|
"serde_json",
|
||||||
|
"simple_asn1 0.5.4",
|
||||||
|
"thiserror 1.0.69",
|
||||||
|
]
|
||||||
|
|
||||||
[[package]]
|
[[package]]
|
||||||
name = "jsonwebtoken"
|
name = "jsonwebtoken"
|
||||||
version = "9.3.0"
|
version = "9.3.0"
|
||||||
|
|
@ -1246,11 +1268,11 @@ checksum = "b9ae10193d25051e74945f1ea2d0b42e03cc3b890f7e4cc5faa44997d808193f"
|
||||||
dependencies = [
|
dependencies = [
|
||||||
"base64 0.21.7",
|
"base64 0.21.7",
|
||||||
"js-sys",
|
"js-sys",
|
||||||
"pem",
|
"pem 3.0.4",
|
||||||
"ring",
|
"ring",
|
||||||
"serde",
|
"serde",
|
||||||
"serde_json",
|
"serde_json",
|
||||||
"simple_asn1",
|
"simple_asn1 0.6.2",
|
||||||
]
|
]
|
||||||
|
|
||||||
[[package]]
|
[[package]]
|
||||||
|
|
@ -1561,6 +1583,17 @@ version = "1.0.15"
|
||||||
source = "registry+https://github.com/rust-lang/crates.io-index"
|
source = "registry+https://github.com/rust-lang/crates.io-index"
|
||||||
checksum = "57c0d7b74b563b49d38dae00a0c37d4d6de9b432382b2892f0574ddcae73fd0a"
|
checksum = "57c0d7b74b563b49d38dae00a0c37d4d6de9b432382b2892f0574ddcae73fd0a"
|
||||||
|
|
||||||
|
[[package]]
|
||||||
|
name = "pem"
|
||||||
|
version = "0.8.3"
|
||||||
|
source = "registry+https://github.com/rust-lang/crates.io-index"
|
||||||
|
checksum = "fd56cbd21fea48d0c440b41cd69c589faacade08c992d9a54e471b79d0fd13eb"
|
||||||
|
dependencies = [
|
||||||
|
"base64 0.13.1",
|
||||||
|
"once_cell",
|
||||||
|
"regex",
|
||||||
|
]
|
||||||
|
|
||||||
[[package]]
|
[[package]]
|
||||||
name = "pem"
|
name = "pem"
|
||||||
version = "3.0.4"
|
version = "3.0.4"
|
||||||
|
|
@ -1921,6 +1954,18 @@ dependencies = [
|
||||||
"rand_core",
|
"rand_core",
|
||||||
]
|
]
|
||||||
|
|
||||||
|
[[package]]
|
||||||
|
name = "simple_asn1"
|
||||||
|
version = "0.5.4"
|
||||||
|
source = "registry+https://github.com/rust-lang/crates.io-index"
|
||||||
|
checksum = "8eb4ea60fb301dc81dfc113df680571045d375ab7345d171c5dc7d7e13107a80"
|
||||||
|
dependencies = [
|
||||||
|
"chrono",
|
||||||
|
"num-bigint",
|
||||||
|
"num-traits",
|
||||||
|
"thiserror 1.0.69",
|
||||||
|
]
|
||||||
|
|
||||||
[[package]]
|
[[package]]
|
||||||
name = "simple_asn1"
|
name = "simple_asn1"
|
||||||
version = "0.6.2"
|
version = "0.6.2"
|
||||||
|
|
|
||||||
5
TODO.md
5
TODO.md
|
|
@ -1,5 +1,8 @@
|
||||||
# TODO
|
# TODO
|
||||||
|
|
||||||
|
- [ ] better OIDC support
|
||||||
|
- [ ] better support of `profile` `openid` `email` `roles` scopes
|
||||||
|
|
||||||
- [ ] i18n strings in the http website.
|
- [ ] i18n strings in the http website.
|
||||||
|
|
||||||
- [ ] Instance customization support
|
- [ ] Instance customization support
|
||||||
|
|
@ -50,3 +53,5 @@
|
||||||
- [X] basic docker setup
|
- [X] basic docker setup
|
||||||
- [ ] make `docker stop` working (handle SIGTERM/SIGINT)
|
- [ ] make `docker stop` working (handle SIGTERM/SIGINT)
|
||||||
- [ ] implement docker secrets. https://docs.docker.com/engine/swarm/secrets/
|
- [ ] implement docker secrets. https://docs.docker.com/engine/swarm/secrets/
|
||||||
|
|
||||||
|
- [ ] Find a minimal OpenID client implementation like Listmonk but a little bit more mature
|
||||||
|
|
|
||||||
|
|
@ -1,3 +1,5 @@
|
||||||
|
signing_key = "tmp/secrets/signing.key"
|
||||||
|
|
||||||
[instance]
|
[instance]
|
||||||
base_uri = "https://auth.fictive.org"
|
base_uri = "https://auth.fictive.org"
|
||||||
name = "Fictive's auth"
|
name = "Fictive's auth"
|
||||||
|
|
|
||||||
|
|
@ -43,6 +43,15 @@ argh = { workspace = true }
|
||||||
sqlx = { workspace = true }
|
sqlx = { workspace = true }
|
||||||
uuid = { workspace = true }
|
uuid = { workspace = true }
|
||||||
url = { workspace = true }
|
url = { workspace = true }
|
||||||
|
pem = "3.0.4"
|
||||||
|
|
||||||
|
# For now, we test if it's viable, and later we will fork it to fix the build (cf. issue
|
||||||
|
# https://github.com/informationsea/jsonwebkey-rs#1 )
|
||||||
|
[dependencies.jsonwebkey-convert]
|
||||||
|
path = "/home/mbess/workspace/foss/rust_libs/jsonwebkey-rs/jsonwebkey-convert"
|
||||||
|
features = ["simple_asn1", "pem"]
|
||||||
|
|
||||||
|
pem = "3.0.4"
|
||||||
|
|
||||||
[build-dependencies]
|
[build-dependencies]
|
||||||
minijinja-embed = "2.3.1"
|
minijinja-embed = "2.3.1"
|
||||||
|
|
|
||||||
|
|
@ -6,7 +6,7 @@ use serde::{Deserialize, Serialize};
|
||||||
|
|
||||||
use kernel::{models::authorization::Authorization, repositories::users::get_user_by_id};
|
use kernel::{models::authorization::Authorization, repositories::users::get_user_by_id};
|
||||||
use crate::{
|
use crate::{
|
||||||
services::{app_session::AppClientSession, session::create_token}, token_claims::{AppUserTokenClaims, OAuth2AccessTokenClaims, OIDCIdTokenClaims}, AppState
|
services::{app_session::AppClientSession, session::create_token}, token_claims::{OAuth2AccessTokenClaims, OIDCIdTokenClaims}, AppState
|
||||||
};
|
};
|
||||||
|
|
||||||
const AUTHORIZATION_CODE_TTL_SECONDS: i64 = 120;
|
const AUTHORIZATION_CODE_TTL_SECONDS: i64 = 120;
|
||||||
|
|
@ -81,22 +81,25 @@ pub async fn get_access_token(
|
||||||
|
|
||||||
// 3.1. Generate JWT for OAuth2 client user session
|
// 3.1. Generate JWT for OAuth2 client user session
|
||||||
let access_token_jwt = create_token(
|
let access_token_jwt = create_token(
|
||||||
|
&app_state.config,
|
||||||
&app_state.secrets,
|
&app_state.secrets,
|
||||||
OAuth2AccessTokenClaims::new(
|
OAuth2AccessTokenClaims::new(
|
||||||
&app_state.config,
|
&app_state.config,
|
||||||
&app_client_session.client_id,
|
|
||||||
&user,
|
&user,
|
||||||
authorization.scopes.to_vec()
|
authorization.scopes.to_vec()
|
||||||
)
|
)
|
||||||
);
|
);
|
||||||
// 3.2. Generate id_token for OIDC client
|
// 3.2. Generate id_token for OIDC client
|
||||||
|
let id_token_claims = OIDCIdTokenClaims::new(
|
||||||
|
&app_state.config,
|
||||||
|
&app_client_session.client_id,
|
||||||
|
user.clone(),
|
||||||
|
authorization.nonce.clone()
|
||||||
|
);
|
||||||
let id_token_jwt = create_token(
|
let id_token_jwt = create_token(
|
||||||
|
&app_state.config,
|
||||||
&app_state.secrets,
|
&app_state.secrets,
|
||||||
OIDCIdTokenClaims::new(
|
id_token_claims
|
||||||
&app_state.config,
|
|
||||||
&app_client_session.client_id,
|
|
||||||
&user
|
|
||||||
)
|
|
||||||
);
|
);
|
||||||
// 4. return JWT
|
// 4. return JWT
|
||||||
let access_token_res = AccessTokenResponse {
|
let access_token_res = AccessTokenResponse {
|
||||||
|
|
|
||||||
45
lib/http_server/src/controllers/api/openid/keys.rs
Normal file
45
lib/http_server/src/controllers/api/openid/keys.rs
Normal file
|
|
@ -0,0 +1,45 @@
|
||||||
|
use jsonwebkey_convert::RSAPublicKey;
|
||||||
|
use jsonwebkey_convert::der::FromPem;
|
||||||
|
|
||||||
|
use axum::{extract::State, response::IntoResponse, Json};
|
||||||
|
use fully_pub::fully_pub;
|
||||||
|
use serde::Serialize;
|
||||||
|
|
||||||
|
use crate::AppState;
|
||||||
|
|
||||||
|
// /// JSON Web Key
|
||||||
|
// /// @See https://www.rfc-editor.org/rfc/rfc7517.html
|
||||||
|
// #[derive(Serialize)]
|
||||||
|
// #[fully_pub]
|
||||||
|
// struct RsaJWK {
|
||||||
|
// #[serde(rename = "use")]
|
||||||
|
// utilisation: String,
|
||||||
|
// alg: String,
|
||||||
|
// kid: String,
|
||||||
|
// #[serde(rename = "modulus")]
|
||||||
|
// modulus: String,
|
||||||
|
// exp: String
|
||||||
|
// }
|
||||||
|
|
||||||
|
/// JSON Web Key set
|
||||||
|
/// @See https://www.rfc-editor.org/rfc/rfc7517.html
|
||||||
|
#[derive(Serialize)]
|
||||||
|
#[fully_pub]
|
||||||
|
struct JWKs {
|
||||||
|
keys: Vec<RSAPublicKey>
|
||||||
|
}
|
||||||
|
|
||||||
|
pub async fn get_signing_public_keys(
|
||||||
|
State(app_state): State<AppState>,
|
||||||
|
) -> impl IntoResponse {
|
||||||
|
let pem_data = app_state.secrets.signing_keypair.0;
|
||||||
|
|
||||||
|
// extract modulus and exp number from ASN.1 encoded PCKS 1 package
|
||||||
|
let rsa_jwk = RSAPublicKey::from_pem(pem_data)
|
||||||
|
.expect("Expected to decode PEM public key");
|
||||||
|
dbg!(&rsa_jwk);
|
||||||
|
|
||||||
|
Json(JWKs {
|
||||||
|
keys: vec![rsa_jwk]
|
||||||
|
}).into_response()
|
||||||
|
}
|
||||||
|
|
@ -1 +1,2 @@
|
||||||
pub mod well_known;
|
pub mod well_known;
|
||||||
|
pub mod keys;
|
||||||
|
|
|
||||||
|
|
@ -18,7 +18,8 @@ struct WellKnownOpenIdConfiguration {
|
||||||
scopes_supported: Vec<String>,
|
scopes_supported: Vec<String>,
|
||||||
response_types_supported: Vec<String>,
|
response_types_supported: Vec<String>,
|
||||||
token_endpoint_auth_methods_supported: Vec<String>,
|
token_endpoint_auth_methods_supported: Vec<String>,
|
||||||
id_token_signing_alg_values_supported: Vec<String>
|
id_token_signing_alg_values_supported: Vec<String>,
|
||||||
|
jwks_uri: String
|
||||||
}
|
}
|
||||||
|
|
||||||
pub async fn get_well_known_openid_configuration(
|
pub async fn get_well_known_openid_configuration(
|
||||||
|
|
@ -33,7 +34,8 @@ pub async fn get_well_known_openid_configuration(
|
||||||
scopes_supported: AuthorizationScope::iter().map(|v| v.to_string()).collect(),
|
scopes_supported: AuthorizationScope::iter().map(|v| v.to_string()).collect(),
|
||||||
response_types_supported: vec!["code".into()],
|
response_types_supported: vec!["code".into()],
|
||||||
token_endpoint_auth_methods_supported: vec!["client_secret_basic".into()],
|
token_endpoint_auth_methods_supported: vec!["client_secret_basic".into()],
|
||||||
id_token_signing_alg_values_supported: vec!["HS256".into()],
|
id_token_signing_alg_values_supported: vec!["RS256".into()],
|
||||||
|
jwks_uri: format!("{}/.well-known/jwks", base_url)
|
||||||
// jwks_uri:
|
// jwks_uri:
|
||||||
// subject_types_supported
|
// subject_types_supported
|
||||||
})
|
})
|
||||||
|
|
|
||||||
|
|
@ -2,7 +2,7 @@ use axum::{extract::State, response::IntoResponse, Extension, Json};
|
||||||
use fully_pub::fully_pub;
|
use fully_pub::fully_pub;
|
||||||
use serde::Serialize;
|
use serde::Serialize;
|
||||||
|
|
||||||
use crate::{token_claims::AppUserTokenClaims, AppState};
|
use crate::{token_claims::OAuth2AccessTokenClaims, AppState};
|
||||||
use kernel::models::user::User;
|
use kernel::models::user::User;
|
||||||
|
|
||||||
#[derive(Serialize)]
|
#[derive(Serialize)]
|
||||||
|
|
@ -18,11 +18,11 @@ struct ReadUserBasicExtract {
|
||||||
|
|
||||||
pub async fn read_user_basic(
|
pub async fn read_user_basic(
|
||||||
State(app_state): State<AppState>,
|
State(app_state): State<AppState>,
|
||||||
Extension(token_claims): Extension<AppUserTokenClaims>,
|
Extension(token_claims): Extension<OAuth2AccessTokenClaims>,
|
||||||
) -> impl IntoResponse {
|
) -> impl IntoResponse {
|
||||||
// 1. This handler require app user authentification (JWT)
|
// 1. This handler require app user authentification (JWT)
|
||||||
let user_res = sqlx::query_as::<_, User>("SELECT * FROM users WHERE id = $1")
|
let user_res = sqlx::query_as::<_, User>("SELECT * FROM users WHERE id = $1")
|
||||||
.bind(&token_claims.user_id)
|
.bind(&token_claims.sub)
|
||||||
.fetch_one(&app_state.db.0)
|
.fetch_one(&app_state.db.0)
|
||||||
.await
|
.await
|
||||||
.expect("To get user from claim");
|
.expect("To get user from claim");
|
||||||
|
|
|
||||||
|
|
@ -7,9 +7,7 @@ use serde::{Deserialize, Serialize};
|
||||||
use url::Url;
|
use url::Url;
|
||||||
use uuid::Uuid;
|
use uuid::Uuid;
|
||||||
|
|
||||||
use kernel::{
|
use kernel::models::{authorization::Authorization, config::AppAuthorizeFlow};
|
||||||
models::{authorization::Authorization, config::AppAuthorizeFlow}
|
|
||||||
};
|
|
||||||
use utils::get_random_alphanumerical;
|
use utils::get_random_alphanumerical;
|
||||||
use crate::{
|
use crate::{
|
||||||
renderer::TemplateRenderer, services::oauth2::{parse_scope, verify_redirect_uri}, token_claims::UserTokenClaims, AppState
|
renderer::TemplateRenderer, services::oauth2::{parse_scope, verify_redirect_uri}, token_claims::UserTokenClaims, AppState
|
||||||
|
|
@ -25,6 +23,7 @@ struct AuthorizationParams {
|
||||||
redirect_uri: String,
|
redirect_uri: String,
|
||||||
/// An opaque value used by the client to maintain state between the request and callback
|
/// An opaque value used by the client to maintain state between the request and callback
|
||||||
state: String,
|
state: String,
|
||||||
|
nonce: Option<String>
|
||||||
}
|
}
|
||||||
|
|
||||||
fn redirect_to_client(
|
fn redirect_to_client(
|
||||||
|
|
@ -34,7 +33,7 @@ fn redirect_to_client(
|
||||||
let target_url = format!("{}?code={}&state={}",
|
let target_url = format!("{}?code={}&state={}",
|
||||||
authorization_params.redirect_uri,
|
authorization_params.redirect_uri,
|
||||||
authorization_code,
|
authorization_code,
|
||||||
authorization_params.state,
|
authorization_params.state
|
||||||
);
|
);
|
||||||
debug!("Redirecting to {}", target_url);
|
debug!("Redirecting to {}", target_url);
|
||||||
|
|
||||||
|
|
@ -117,9 +116,10 @@ pub async fn authorize_form(
|
||||||
// Create new auth code
|
// Create new auth code
|
||||||
let authorization_code = get_random_alphanumerical(32);
|
let authorization_code = get_random_alphanumerical(32);
|
||||||
// Update last used timestamp for this authorization
|
// Update last used timestamp for this authorization
|
||||||
let _result = sqlx::query("UPDATE authorizations SET code = $2, last_used_at = $3 WHERE id = $1")
|
let _result = sqlx::query("UPDATE authorizations SET code = $2, nonce = $3, last_used_at = $4 WHERE id = $1")
|
||||||
.bind(existing_authorization.id)
|
.bind(existing_authorization.id)
|
||||||
.bind(authorization_code.clone())
|
.bind(authorization_code.clone())
|
||||||
|
.bind(authorization_params.nonce.clone())
|
||||||
.bind(Utc::now().to_rfc3339_opts(SecondsFormat::Millis, true))
|
.bind(Utc::now().to_rfc3339_opts(SecondsFormat::Millis, true))
|
||||||
.execute(&app_state.db.0)
|
.execute(&app_state.db.0)
|
||||||
.await.unwrap();
|
.await.unwrap();
|
||||||
|
|
@ -173,6 +173,7 @@ pub async fn perform_authorize(
|
||||||
Extension(token_claims): Extension<UserTokenClaims>,
|
Extension(token_claims): Extension<UserTokenClaims>,
|
||||||
Form(authorize_form): Form<AuthorizationParams>
|
Form(authorize_form): Form<AuthorizationParams>
|
||||||
) -> impl IntoResponse {
|
) -> impl IntoResponse {
|
||||||
|
dbg!(&authorize_form);
|
||||||
// 1. Get the app details
|
// 1. Get the app details
|
||||||
let app = match app_state.config.applications
|
let app = match app_state.config.applications
|
||||||
.iter()
|
.iter()
|
||||||
|
|
@ -204,6 +205,7 @@ pub async fn perform_authorize(
|
||||||
client_id: app.client_id.clone(),
|
client_id: app.client_id.clone(),
|
||||||
scopes: sqlx::types::Json(scopes),
|
scopes: sqlx::types::Json(scopes),
|
||||||
code: authorization_code.clone(),
|
code: authorization_code.clone(),
|
||||||
|
nonce: authorize_form.nonce.clone(),
|
||||||
last_used_at: Some(Utc::now()),
|
last_used_at: Some(Utc::now()),
|
||||||
created_at: Utc::now(),
|
created_at: Utc::now(),
|
||||||
};
|
};
|
||||||
|
|
@ -211,14 +213,15 @@ pub async fn perform_authorize(
|
||||||
// 3. Save authorization in DB with state
|
// 3. Save authorization in DB with state
|
||||||
let res = sqlx::query("
|
let res = sqlx::query("
|
||||||
INSERT INTO authorizations
|
INSERT INTO authorizations
|
||||||
(id, user_id, client_id, scopes, code, last_used_at, created_at)
|
(id, user_id, client_id, scopes, code, nonce, last_used_at, created_at)
|
||||||
VALUES ($1, $2, $3, $4, $5, $6, $7)
|
VALUES ($1, $2, $3, $4, $5, $6, $7, $8)
|
||||||
")
|
")
|
||||||
.bind(authorization.id.clone())
|
.bind(authorization.id.clone())
|
||||||
.bind(authorization.user_id)
|
.bind(authorization.user_id)
|
||||||
.bind(authorization.client_id)
|
.bind(authorization.client_id)
|
||||||
.bind(authorization.scopes)
|
.bind(authorization.scopes)
|
||||||
.bind(authorization.code)
|
.bind(authorization.code)
|
||||||
|
.bind(authorization.nonce)
|
||||||
.bind(authorization.last_used_at.map(|x| x.to_rfc3339_opts(SecondsFormat::Millis, true)))
|
.bind(authorization.last_used_at.map(|x| x.to_rfc3339_opts(SecondsFormat::Millis, true)))
|
||||||
.bind(authorization.created_at.to_rfc3339_opts(SecondsFormat::Millis, true))
|
.bind(authorization.created_at.to_rfc3339_opts(SecondsFormat::Millis, true))
|
||||||
.execute(&app_state.db.0)
|
.execute(&app_state.db.0)
|
||||||
|
|
|
||||||
|
|
@ -91,8 +91,8 @@ pub async fn perform_login(
|
||||||
.await.unwrap();
|
.await.unwrap();
|
||||||
|
|
||||||
let jwt_max_age = Duration::days(15);
|
let jwt_max_age = Duration::days(15);
|
||||||
let claims = UserTokenClaims::new(&user.id, jwt_max_age);
|
let claims = UserTokenClaims::new(&app_state.config, &user.id, jwt_max_age);
|
||||||
let jwt = create_token(&app_state.secrets, claims);
|
let jwt = create_token(&app_state.config, &app_state.secrets, claims);
|
||||||
|
|
||||||
// TODO: handle keep_session boolean from form and specify cookie max age only if this setting
|
// TODO: handle keep_session boolean from form and specify cookie max age only if this setting
|
||||||
// is true
|
// is true
|
||||||
|
|
|
||||||
|
|
@ -9,7 +9,7 @@ use utils::parse_basic_auth;
|
||||||
|
|
||||||
use crate::{
|
use crate::{
|
||||||
services::{app_session::AppClientSession, session::verify_token},
|
services::{app_session::AppClientSession, session::verify_token},
|
||||||
token_claims::AppUserTokenClaims,
|
token_claims::OAuth2AccessTokenClaims,
|
||||||
AppState
|
AppState
|
||||||
};
|
};
|
||||||
|
|
||||||
|
|
@ -102,14 +102,16 @@ pub async fn enforce_jwt_auth_middleware(
|
||||||
);
|
);
|
||||||
}
|
}
|
||||||
};
|
};
|
||||||
let token_claims: AppUserTokenClaims = match verify_token(&app_state.secrets, jwt) {
|
let token_claims: OAuth2AccessTokenClaims =
|
||||||
Ok(val) => val,
|
match verify_token(&app_state.config, &app_state.secrets, jwt) {
|
||||||
Err(_e) => {
|
Ok(val) => val,
|
||||||
return Err(
|
Err(_e) => {
|
||||||
(StatusCode::UNAUTHORIZED, Html("Unauthorized: The provided JWT is invalid."))
|
dbg!(_e);
|
||||||
);
|
return Err(
|
||||||
}
|
(StatusCode::UNAUTHORIZED, Html("Unauthorized: The provided JWT is invalid."))
|
||||||
};
|
);
|
||||||
|
}
|
||||||
|
};
|
||||||
req.extensions_mut().insert(token_claims);
|
req.extensions_mut().insert(token_claims);
|
||||||
Ok(next.run(req).await)
|
Ok(next.run(req).await)
|
||||||
}
|
}
|
||||||
|
|
|
||||||
|
|
@ -28,18 +28,20 @@ pub async fn auth_middleware(
|
||||||
return Ok(next.run(req).await)
|
return Ok(next.run(req).await)
|
||||||
}
|
}
|
||||||
};
|
};
|
||||||
let token_claims: UserTokenClaims = match verify_token(&app_state.secrets, jwt) {
|
let token_claims: UserTokenClaims =
|
||||||
Ok(val) => val,
|
match verify_token(&app_state.config, &app_state.secrets, jwt) {
|
||||||
Err(_e) => {
|
Ok(val) => val,
|
||||||
// UserWebGUI: delete invalid JWT cookie
|
Err(_e) => {
|
||||||
return Err(
|
dbg!(&_e);
|
||||||
(
|
// UserWebGUI: delete invalid JWT cookie
|
||||||
cookies.remove(WEB_GUI_JWT_COOKIE_NAME),
|
return Err(
|
||||||
Redirect::to(&original_uri.to_string())
|
(
|
||||||
)
|
cookies.remove(WEB_GUI_JWT_COOKIE_NAME),
|
||||||
);
|
Redirect::to(&original_uri.to_string())
|
||||||
}
|
)
|
||||||
};
|
);
|
||||||
|
}
|
||||||
|
};
|
||||||
req.extensions_mut().insert(token_claims);
|
req.extensions_mut().insert(token_claims);
|
||||||
Ok(next.run(req).await)
|
Ok(next.run(req).await)
|
||||||
}
|
}
|
||||||
|
|
|
||||||
|
|
@ -51,7 +51,8 @@ pub fn build_router(server_config: &ServerConfig, app_state: AppState) -> Router
|
||||||
.route("/api/user-assets/:asset_id", get(api::public_assets::get_user_asset));
|
.route("/api/user-assets/:asset_id", get(api::public_assets::get_user_asset));
|
||||||
|
|
||||||
let well_known_routes = Router::new()
|
let well_known_routes = Router::new()
|
||||||
.route("/.well-known/openid-configuration", get(api::openid::well_known::get_well_known_openid_configuration));
|
.route("/.well-known/openid-configuration", get(api::openid::well_known::get_well_known_openid_configuration))
|
||||||
|
.route("/.well-known/jwks", get(api::openid::keys::get_signing_public_keys));
|
||||||
|
|
||||||
Router::new()
|
Router::new()
|
||||||
.merge(public_routes)
|
.merge(public_routes)
|
||||||
|
|
|
||||||
|
|
@ -1,24 +1,37 @@
|
||||||
use anyhow::Result;
|
use anyhow::Result;
|
||||||
use serde::{de::DeserializeOwned, Serialize};
|
use serde::{de::DeserializeOwned, Serialize};
|
||||||
use jsonwebtoken::{encode, decode, Header, Algorithm, Validation, EncodingKey, DecodingKey};
|
use jsonwebtoken::{encode, decode, Header, Algorithm, Validation, EncodingKey, DecodingKey};
|
||||||
use kernel::context::AppSecrets;
|
use kernel::{context::AppSecrets, models::config::Config};
|
||||||
|
|
||||||
|
|
||||||
pub fn create_token<T: Serialize>(secrets: &AppSecrets, claims: T) -> String {
|
pub fn create_token<T: Serialize>(
|
||||||
|
_config: &Config,
|
||||||
|
secrets: &AppSecrets,
|
||||||
|
claims: T
|
||||||
|
) -> String {
|
||||||
let token = encode(
|
let token = encode(
|
||||||
&Header::default(),
|
&Header::new(Algorithm::RS256),
|
||||||
&claims,
|
&claims,
|
||||||
&EncodingKey::from_secret(secrets.jwt_secret.as_bytes())
|
&EncodingKey::from_rsa_pem(&secrets.signing_keypair.1)
|
||||||
|
.expect("To build encoding key from signing key.")
|
||||||
).expect("Create token");
|
).expect("Create token");
|
||||||
|
|
||||||
token
|
token
|
||||||
}
|
}
|
||||||
|
|
||||||
pub fn verify_token<T: DeserializeOwned>(secrets: &AppSecrets, jwt: &str) -> Result<T> {
|
pub fn verify_token<T: DeserializeOwned>(
|
||||||
|
config: &Config,
|
||||||
|
secrets: &AppSecrets,
|
||||||
|
jwt: &str
|
||||||
|
) -> Result<T> {
|
||||||
|
let mut validation = Validation::new(Algorithm::RS256);
|
||||||
|
validation.set_issuer(&[config.instance.base_uri.clone()]);
|
||||||
|
validation.set_audience(&[config.instance.base_uri.clone()]);
|
||||||
let token_data = decode::<T>(
|
let token_data = decode::<T>(
|
||||||
jwt,
|
jwt,
|
||||||
&DecodingKey::from_secret(secrets.jwt_secret.as_bytes()),
|
&DecodingKey::from_rsa_pem(&secrets.signing_keypair.0)
|
||||||
&Validation::new(Algorithm::HS256)
|
.expect("To build decoding key from signing key."),
|
||||||
|
&validation
|
||||||
)?;
|
)?;
|
||||||
|
|
||||||
Ok(token_data.claims)
|
Ok(token_data.claims)
|
||||||
|
|
|
||||||
|
|
@ -12,16 +12,19 @@ struct UserTokenClaims {
|
||||||
/// token expiration
|
/// token expiration
|
||||||
exp: u64,
|
exp: u64,
|
||||||
/// token issuer
|
/// token issuer
|
||||||
iss: String
|
iss: String,
|
||||||
// TODO: add roles
|
// TODO: add roles
|
||||||
|
/// token audience
|
||||||
|
aud: String
|
||||||
}
|
}
|
||||||
|
|
||||||
impl UserTokenClaims {
|
impl UserTokenClaims {
|
||||||
pub fn new(user_id: &str, max_age: Duration) -> Self {
|
pub fn new(config: &Config, user_id: &str, max_age: Duration) -> Self {
|
||||||
UserTokenClaims {
|
UserTokenClaims {
|
||||||
sub: user_id.into(),
|
sub: user_id.into(),
|
||||||
exp: get_current_timestamp() + max_age.whole_seconds() as u64,
|
exp: get_current_timestamp() + max_age.whole_seconds() as u64,
|
||||||
iss: "Minauthator".into()
|
iss: config.instance.base_uri.clone(),
|
||||||
|
aud: config.instance.base_uri.clone(),
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
@ -33,7 +36,7 @@ impl UserTokenClaims {
|
||||||
struct OAuth2AccessTokenClaims {
|
struct OAuth2AccessTokenClaims {
|
||||||
/// Token issuer (URI to the issuer)
|
/// Token issuer (URI to the issuer)
|
||||||
iss: String,
|
iss: String,
|
||||||
/// Audiance (client_id)
|
/// Audiance (In this case, the audiance is equal to the issuer)
|
||||||
aud: String,
|
aud: String,
|
||||||
/// End-user id assigned by the issuer (user_id)
|
/// End-user id assigned by the issuer (user_id)
|
||||||
sub: String,
|
sub: String,
|
||||||
|
|
@ -44,10 +47,10 @@ struct OAuth2AccessTokenClaims {
|
||||||
}
|
}
|
||||||
|
|
||||||
impl OAuth2AccessTokenClaims {
|
impl OAuth2AccessTokenClaims {
|
||||||
pub fn new(config: &Config, client_id: &str, user: &User, scopes: Vec<AuthorizationScope>) -> Self {
|
pub fn new(config: &Config, user: &User, scopes: Vec<AuthorizationScope>) -> Self {
|
||||||
OAuth2AccessTokenClaims {
|
OAuth2AccessTokenClaims {
|
||||||
iss: config.instance.base_uri.clone(),
|
iss: config.instance.base_uri.clone(),
|
||||||
aud: client_id.into(),
|
aud: config.instance.base_uri.clone(),
|
||||||
sub: user.id.clone(),
|
sub: user.id.clone(),
|
||||||
exp: get_current_timestamp() + 86_000,
|
exp: get_current_timestamp() + 86_000,
|
||||||
scopes,
|
scopes,
|
||||||
|
|
@ -73,20 +76,27 @@ struct OIDCIdTokenClaims {
|
||||||
name: Option<String>,
|
name: Option<String>,
|
||||||
email: Option<String>,
|
email: Option<String>,
|
||||||
preferred_username: Option<String>,
|
preferred_username: Option<String>,
|
||||||
roles: Vec<String>
|
roles: Vec<String>,
|
||||||
|
nonce: Option<String>
|
||||||
}
|
}
|
||||||
|
|
||||||
impl OIDCIdTokenClaims {
|
impl OIDCIdTokenClaims {
|
||||||
pub fn new(config: &Config, client_id: &str, user: &User) -> Self {
|
pub fn new(
|
||||||
|
config: &Config,
|
||||||
|
client_id: &str,
|
||||||
|
user: User,
|
||||||
|
nonce: Option<String>
|
||||||
|
) -> Self {
|
||||||
OIDCIdTokenClaims {
|
OIDCIdTokenClaims {
|
||||||
iss: config.instance.base_uri.clone(),
|
iss: config.instance.base_uri.clone(),
|
||||||
aud: client_id.into(),
|
aud: client_id.into(),
|
||||||
sub: user.id.clone(),
|
sub: user.id,
|
||||||
exp: get_current_timestamp() + 86_000,
|
exp: get_current_timestamp() + 86_000,
|
||||||
email: user.email.clone(),
|
email: user.email,
|
||||||
name: user.full_name.clone(),
|
name: user.full_name,
|
||||||
preferred_username: Some(user.handle.clone()),
|
preferred_username: Some(user.handle),
|
||||||
roles: user.roles.0.clone()
|
roles: user.roles.0,
|
||||||
|
nonce
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
|
||||||
|
|
@ -1,4 +1,5 @@
|
||||||
pub const DEFAULT_DB_PATH: &str = "/var/lib/minauthator/minauthator.db";
|
pub const DEFAULT_DB_PATH: &str = "/var/lib/minauthator/minauthator.db";
|
||||||
pub const DEFAULT_ASSETS_PATH: &str = "/usr/local/lib/minauthator/assets";
|
pub const DEFAULT_ASSETS_PATH: &str = "/usr/local/lib/minauthator/assets";
|
||||||
pub const DEFAULT_CONFIG_PATH: &str = "/etc/minauthator/config.toml";
|
pub const DEFAULT_CONFIG_PATH: &str = "/etc/minauthator/config.toml";
|
||||||
|
pub const DEFAULT_SIGNING_KEY_PATH: &str = "/etc/minauthator/secrets/jwt.key.pem";
|
||||||
|
|
||||||
|
|
|
||||||
|
|
@ -1,11 +1,13 @@
|
||||||
use std::{env, fs};
|
use std::{env, fs, path::Path};
|
||||||
use anyhow::{Result, Context, anyhow};
|
use anyhow::{Result, Context, anyhow};
|
||||||
use fully_pub::fully_pub;
|
use fully_pub::fully_pub;
|
||||||
|
|
||||||
use log::info;
|
use log::info;
|
||||||
use sqlx::{Pool, Sqlite};
|
|
||||||
use crate::{
|
use crate::{
|
||||||
consts::{DEFAULT_CONFIG_PATH, DEFAULT_DB_PATH}, database::prepare_database, models::config::Config, repositories::storage::Storage
|
consts::{DEFAULT_CONFIG_PATH, DEFAULT_DB_PATH, DEFAULT_SIGNING_KEY_PATH},
|
||||||
|
database::prepare_database,
|
||||||
|
models::config::Config,
|
||||||
|
repositories::storage::Storage
|
||||||
};
|
};
|
||||||
|
|
||||||
/// get server config
|
/// get server config
|
||||||
|
|
@ -26,9 +28,18 @@ struct StartKernelConfig {
|
||||||
#[derive(Debug, Clone)]
|
#[derive(Debug, Clone)]
|
||||||
#[fully_pub]
|
#[fully_pub]
|
||||||
struct AppSecrets {
|
struct AppSecrets {
|
||||||
jwt_secret: String
|
/// RSA keypair (public, private) used to signed the JWT issued by minauthator in PEM conainer format
|
||||||
|
signing_keypair: (Vec<u8>, Vec<u8>)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
#[derive(Debug, Clone)]
|
||||||
|
#[fully_pub]
|
||||||
|
struct ComputedConfig {
|
||||||
|
signing_public_key: Vec<u8>,
|
||||||
|
signing_private_key: Vec<u8>
|
||||||
|
}
|
||||||
|
|
||||||
|
|
||||||
#[derive(Debug, Clone)]
|
#[derive(Debug, Clone)]
|
||||||
#[fully_pub]
|
#[fully_pub]
|
||||||
struct KernelContext {
|
struct KernelContext {
|
||||||
|
|
@ -37,6 +48,19 @@ struct KernelContext {
|
||||||
storage: Storage
|
storage: Storage
|
||||||
}
|
}
|
||||||
|
|
||||||
|
fn get_signing_keypair(key_path: &str) -> Result<(Vec<u8>, Vec<u8>)> {
|
||||||
|
let key_path = Path::new(key_path);
|
||||||
|
let pub_key_path = key_path.with_extension("pub");
|
||||||
|
let private_key: Vec<u8> = fs::read_to_string(&key_path)
|
||||||
|
.context(format!("Failed to read private key from path {:?}.", key_path))?
|
||||||
|
.as_bytes().to_vec();
|
||||||
|
let public_key: Vec<u8> = fs::read_to_string(&pub_key_path)
|
||||||
|
.context(format!("Failed to read public key from path {:?}.", pub_key_path))?
|
||||||
|
.as_bytes().to_vec();
|
||||||
|
|
||||||
|
Ok((public_key, private_key))
|
||||||
|
}
|
||||||
|
|
||||||
pub async fn get_kernel_context(start_config: StartKernelConfig) -> Result<KernelContext> {
|
pub async fn get_kernel_context(start_config: StartKernelConfig) -> Result<KernelContext> {
|
||||||
env_logger::init();
|
env_logger::init();
|
||||||
let _ = dotenvy::dotenv();
|
let _ = dotenvy::dotenv();
|
||||||
|
|
@ -50,10 +74,13 @@ pub async fn get_kernel_context(start_config: StartKernelConfig) -> Result<Kerne
|
||||||
let config: Config = get_config(config_path)
|
let config: Config = get_config(config_path)
|
||||||
.expect("Cannot get config.");
|
.expect("Cannot get config.");
|
||||||
|
|
||||||
|
let signing_key_path = config.signing_key.clone().unwrap_or(DEFAULT_SIGNING_KEY_PATH.to_string());
|
||||||
|
|
||||||
|
// optionally load dotenv file
|
||||||
let _ = dotenvy::dotenv();
|
let _ = dotenvy::dotenv();
|
||||||
|
|
||||||
let secrets = AppSecrets {
|
let secrets = AppSecrets {
|
||||||
jwt_secret: env::var("APP_JWT_SECRET")
|
signing_keypair: get_signing_keypair(&signing_key_path)?
|
||||||
.context("Expected APP_JWT_SECRET environment variable to exists.")?
|
|
||||||
};
|
};
|
||||||
|
|
||||||
Ok(KernelContext {
|
Ok(KernelContext {
|
||||||
|
|
|
||||||
|
|
@ -25,6 +25,8 @@ struct Authorization {
|
||||||
client_id: String,
|
client_id: String,
|
||||||
scopes: Json<Vec<AuthorizationScope>>,
|
scopes: Json<Vec<AuthorizationScope>>,
|
||||||
|
|
||||||
|
nonce: Option<String>,
|
||||||
|
|
||||||
/// defined in https://datatracker.ietf.org/doc/html/rfc6749#section-4.1.2
|
/// defined in https://datatracker.ietf.org/doc/html/rfc6749#section-4.1.2
|
||||||
code: String,
|
code: String,
|
||||||
last_used_at: Option<DateTime<Utc>>,
|
last_used_at: Option<DateTime<Utc>>,
|
||||||
|
|
|
||||||
|
|
@ -66,11 +66,6 @@ struct Role {
|
||||||
struct Config {
|
struct Config {
|
||||||
instance: InstanceConfig,
|
instance: InstanceConfig,
|
||||||
applications: Vec<Application>,
|
applications: Vec<Application>,
|
||||||
roles: Vec<Role>
|
roles: Vec<Role>,
|
||||||
}
|
signing_key: Option<String>
|
||||||
|
|
||||||
#[derive(Debug, Clone)]
|
|
||||||
#[fully_pub]
|
|
||||||
struct AppSecrets {
|
|
||||||
jwt_secret: String
|
|
||||||
}
|
}
|
||||||
|
|
|
||||||
|
|
@ -14,7 +14,7 @@ enum UserStatus {
|
||||||
Active
|
Active
|
||||||
}
|
}
|
||||||
|
|
||||||
#[derive(sqlx::FromRow, Deserialize, Serialize, Debug)]
|
#[derive(sqlx::FromRow, Deserialize, Serialize, Debug, Clone)]
|
||||||
#[fully_pub]
|
#[fully_pub]
|
||||||
struct User {
|
struct User {
|
||||||
/// uuid
|
/// uuid
|
||||||
|
|
|
||||||
|
|
@ -33,6 +33,7 @@ CREATE TABLE authorizations (
|
||||||
client_id TEXT NOT NULL,
|
client_id TEXT NOT NULL,
|
||||||
scopes TEXT, -- json array of app scope (permissions)
|
scopes TEXT, -- json array of app scope (permissions)
|
||||||
code TEXT,
|
code TEXT,
|
||||||
|
nonce TEXT, -- code used to associate client session to id_token
|
||||||
|
|
||||||
last_used_at DATETIME,
|
last_used_at DATETIME,
|
||||||
created_at DATETIME NOT NULL
|
created_at DATETIME NOT NULL
|
||||||
|
|
|
||||||
|
|
@ -1,3 +1,5 @@
|
||||||
|
signing_key = "tmp/secrets/signing.key"
|
||||||
|
|
||||||
[instance]
|
[instance]
|
||||||
base_uri = "http://localhost:8086"
|
base_uri = "http://localhost:8086"
|
||||||
name = "Example org"
|
name = "Example org"
|
||||||
|
|
|
||||||
|
|
@ -1,6 +1,6 @@
|
||||||
POST {{ base_url }}/login
|
POST {{ base_url }}/login
|
||||||
[FormParams]
|
[FormParams]
|
||||||
login: root
|
login: john.doe
|
||||||
password: root
|
password: root
|
||||||
HTTP 303
|
HTTP 303
|
||||||
[Captures]
|
[Captures]
|
||||||
|
|
|
||||||
|
|
@ -1,3 +1,5 @@
|
||||||
|
signing_key = "tmp/secrets/signing.key"
|
||||||
|
|
||||||
[instance]
|
[instance]
|
||||||
base_uri = "http://localhost:8086"
|
base_uri = "http://localhost:8086"
|
||||||
name = "Example org"
|
name = "Example org"
|
||||||
|
|
|
||||||
|
|
@ -27,7 +27,7 @@ handle: root
|
||||||
email: root@johndoe.net
|
email: root@johndoe.net
|
||||||
full_name: John Doe
|
full_name: John Doe
|
||||||
website: https://johndoe.net
|
website: https://johndoe.net
|
||||||
picture: file,john_doe_profile_pic.jpg; image/jpeg
|
avatar: file,john_doe_profile_pic.jpg; image/jpeg
|
||||||
HTTP 200
|
HTTP 200
|
||||||
|
|
||||||
GET {{ base_url }}/me/authorizations
|
GET {{ base_url }}/me/authorizations
|
||||||
|
|
|
||||||
|
|
@ -1,3 +1,4 @@
|
||||||
|
signing_key = "tmp/secrets/signing.key"
|
||||||
applications = []
|
applications = []
|
||||||
|
|
||||||
roles = []
|
roles = []
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue