mart-w/kanidm
1
0
Fork 0
mirror of https://github.com/kanidm/kanidm.git synced 2025-02-23 04:27:02 +01:00
Code Issues Actions 16 Packages Projects Releases Wiki Activity

Fix update intent ttl parameters (#2540)

Browse source
This commit is contained in:
Firstyear 2024-02-16 17:02:36 +10:00 committed by GitHub
parent faec47d13f
commit 816fde766f
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 5 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
libs/client/src/lib.rs
View file

@ -1665,7 +1665,7 @@ impl KanidmClient {
) -> Result<CUIntentToken, ClientError> {
if let Some(ttl) = ttl {
self.perform_get_request(&format!(
"/v1/person/{}/_credential/_update_intent?ttl={}",
"/v1/person/{}/_credential/_update_intent/{}",
id, ttl
))
.await

9
server/core/src/https/v1.rs
View file

@ -1,6 +1,6 @@
//! The V1 API things!
use axum::extract::{Path, Query, State};
use axum::extract::{Path, State};
use axum::http::{HeaderMap, HeaderValue};
use axum::middleware::from_fn;
use axum::response::{IntoResponse, Response};
@ -1108,7 +1108,7 @@ pub async fn person_id_credential_update_get(
#[utoipa::path(
get,
path = "/v1/person/{id}/_credential/_update_intent/?ttl={ttl}",
path = "/v1/person/{id}/_credential/_update_intent/{ttl}",
params(
("ttl" = u64, description="The new TTL for the credential?")
),
@ -1125,8 +1125,7 @@ pub async fn person_id_credential_update_intent_ttl_get(
State(state): State<ServerState>,
Extension(kopid): Extension<KOpId>,
VerifiedClientInformation(client_auth_info): VerifiedClientInformation,
Path(id): Path<String>,
Query(ttl): Query<u64>,
Path((id, ttl)): Path<(String, u64)>,
) -> Result<Json<CUIntentToken>, WebError> {
state
.qe_w_ref
@ -2937,7 +2936,7 @@ pub(crate) fn route_setup(state: ServerState) -> Router<ServerState> {
get(person_id_credential_update_get),
)
.route(
"/v1/person/:id/_credential/_update_intent/:ttl", // TODO: I'm pretty sure this route is wrong, because we match the query not the path
"/v1/person/:id/_credential/_update_intent/:ttl",
get(person_id_credential_update_intent_ttl_get),
)
.route(