From 2f1324c015f6c1470c32b61f1d82234c4e06f9ae Mon Sep 17 00:00:00 2001 From: Christian Schwarz Date: Mon, 27 Mar 2023 18:31:20 +0200 Subject: [PATCH] fix: use parse_request_param Co-authored-by: Dmitry Rodionov --- pageserver/src/http/routes.rs | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/pageserver/src/http/routes.rs b/pageserver/src/http/routes.rs index 375f9090ca..c43416d0a1 100644 --- a/pageserver/src/http/routes.rs +++ b/pageserver/src/http/routes.rs @@ -1189,9 +1189,7 @@ pub fn make_router( /// Testing helper to transition a tenant to [`crate::tenant::TenantState::Broken`]. #[cfg(feature = "testing")] async fn handle_tenant_break(r: Request) -> Result, ApiError> { - use std::str::FromStr; - let tenant_id = get_request_param(&r, "tenant_id")?; - let tenant_id = TenantId::from_str(tenant_id).map_err(|e| ApiError::BadRequest(e.into()))?; + let tenant_id: TenantId = parse_request_param(&request, "tenant_id")?; let tenant = crate::tenant::mgr::get_tenant(tenant_id, true) .await