fix(api): use custom path extractor

Author: MasterPtato

packages/common/api-builder/src/extract.rs

@@ -2,7 +2,7 @@ use anyhow::anyhow;
 use axum::{
 	extract::{
 		Request,
-		rejection::{ExtensionRejection, JsonRejection},
+		rejection::{ExtensionRejection, JsonRejection, PathRejection},
 		{FromRequest, FromRequestParts},
 	},
 	response::IntoResponse,
@@ -102,7 +102,36 @@ where
 			.map(|ext| Extension(ext.0))
 			.map_err(|err| {
 				ExtractorError(
-					anyhow!("developer error: extension error: {}", err.body_text()).into(),
+					ApiBadRequest {
+						reason: err.body_text(),
+					}
+					.build()
+					.into(),
+				)
+			})
+	}
+}
+
+pub struct Path<T>(pub T);
+
+impl<S, T> FromRequestParts<S> for Path<T>
+where
+	axum::extract::Path<T>: FromRequestParts<S, Rejection = PathRejection>,
+	S: Send + Sync,
+{
+	type Rejection = ExtractorError;
+
+	async fn from_request_parts(parts: &mut Parts, state: &S) -> Result<Self, Self::Rejection> {
+		axum::extract::Path::<T>::from_request_parts(parts, state)
+			.await
+			.map(|ext| Path(ext.0))
+			.map_err(|err| {
+				ExtractorError(
+					ApiBadRequest {
+						reason: err.body_text(),
+					}
+					.build()
+					.into(),
 				)
 			})
 	}

packages/common/api-builder/src/wrappers.rs

@@ -1,7 +1,6 @@
 use anyhow::Result;
 use axum::{
 	body::Bytes,
-	extract::Path,
 	response::IntoResponse,
 	routing::{
 		delete as axum_delete, get as axum_get, patch as axum_patch, post as axum_post,
@@ -14,7 +13,7 @@ use std::future::Future;
 use crate::{
 	context::ApiCtx,
 	error_response::ApiError,
-	extract::{Extension, Json, Query},
+	extract::{Extension, Json, Path, Query},
 };
 
 /// Macro to generate wrapper functions for HTTP methods

packages/core/api-public/src/actors/delete.rs

@@ -1,12 +1,11 @@
 use anyhow::Result;
 use axum::{
-	extract::Path,
 	http::HeaderMap,
 	response::{IntoResponse, Response},
 };
 use rivet_api_builder::{
 	ApiError,
-	extract::{Extension, Json, Query},
+	extract::{Extension, Json, Path, Query},
 };
 use rivet_api_util::request_remote_datacenter_raw;
 use rivet_util::Id;

packages/core/api-public/src/runner_configs.rs

@@ -1,12 +1,11 @@
 use anyhow::Result;
 use axum::{
-	extract::Path,
 	http::HeaderMap,
 	response::{IntoResponse, Response},
 };
 use rivet_api_builder::{
 	ApiError,
-	extract::{Extension, Json, Query},
+	extract::{Extension, Json, Path, Query},
 };
 
 use rivet_api_peer::runner_configs::*;

packages/core/api-public/src/ui.rs

@@ -1,9 +1,9 @@
 use axum::{
-	extract::Path,
 	http::{StatusCode, header},
 	response::{IntoResponse, Response},
 };
 use include_dir::{Dir, include_dir};
+use rivet_api_builder::extract::Path;
 
 static UI_DIR: Dir<'_> = include_dir!("$OUT_DIR/ui");