409/axum-valid
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

refactor Arguments

Browse Source
This commit is contained in:
gengteng
2023-09-29 10:45:39 +08:00
parent e4f54a5cb9
commit 21d7ce5e4e
2 changed files with 37 additions and 49 deletions
Download Patch File Download Diff File Expand all files Collapse all files

57
src/lib.rs
View File

@@ -122,16 +122,6 @@ impl<E, A> ValidEx<E, A> {
}
}
/// `ValidationArguments` configures the response returned when validation fails.
///
/// By providing a ValidationArguments to the Valid extractor, you can customize
/// the HTTP status code and response body returned on validation failure.
///
#[derive(Debug, Copy, Clone, Default)]
pub struct ValidationContext<Arguments> {
arguments: Arguments,
}
fn response_builder(ve: ValidationErrors) -> Response {
#[cfg(feature = "into_json")]
{
@@ -143,30 +133,17 @@ fn response_builder(ve: ValidationErrors) -> Response {
}
}
impl ValidationContext<()> {
/// Creates a new `ValidationArguments`.
pub fn with_arguments<Arguments>(self, arguments: Arguments) -> ValidationContext<Arguments> {
ValidationContext { arguments }
}
}
impl<Arguments> ValidationContext<Arguments> {
/// Creates a `ValidationArguments` with arguments
pub fn new(arguments: Arguments) -> Self {
Self { arguments }
}
}
/// # Arguments
/// `Arguments` provides the validation arguments for the data type `T`.
///
/// * `T`: The data type to validate using arguments
/// This trait has an associated type `T` which represents the data type to
/// validate. `T` must implement the `ValidateArgs` trait which defines the
/// validation logic.
///
pub trait Arguments<'a, T>
where
T: ValidateArgs<'a>,
{
/// Get arguments from `self`
fn get(&'a self) -> T::Args;
pub trait Arguments<'a> {
/// The data type to validate using this arguments
type T: ValidateArgs<'a>;
/// This method gets the arguments required by `ValidateArgs::validate_args`
fn get(&'a self) -> <<Self as Arguments<'a>>::T as ValidateArgs<'a>>::Args;
}
/// `ValidRejection` is returned when the `Valid` extractor fails.
@@ -276,15 +253,17 @@ impl<State, Body, Extractor, Args> FromRequest<State, Body> for ValidEx<Extracto
where
State: Send + Sync,
Body: Send + Sync + 'static,
Args: Send + Sync + for<'a> Arguments<'a, <Extractor as HasValidateArgs<'a>>::ValidateArgs>,
Args: Send
+ Sync
+ FromRef<State>
+ for<'a> Arguments<'a, T = <Extractor as HasValidateArgs<'a>>::ValidateArgs>,
Extractor: for<'v> HasValidateArgs<'v> + FromRequest<State, Body>,
for<'v> <Extractor as HasValidateArgs<'v>>::ValidateArgs: ValidateArgs<'v>,
ValidationContext<Args>: FromRef<State>,
{
type Rejection = ValidRejection<<Extractor as FromRequest<State, Body>>::Rejection>;
async fn from_request(req: Request<Body>, state: &State) -> Result<Self, Self::Rejection> {
let ValidationContext { arguments }: ValidationContext<Args> = FromRef::from_ref(state);
let arguments: Args = FromRef::from_ref(state);
let inner = Extractor::from_request(req, state)
.await
.map_err(ValidRejection::Inner)?;
@@ -298,15 +277,17 @@ where
impl<State, Extractor, Args> FromRequestParts<State> for ValidEx<Extractor, Args>
where
State: Send + Sync,
Args: Send + Sync + for<'a> Arguments<'a, <Extractor as HasValidateArgs<'a>>::ValidateArgs>,
Args: Send
+ Sync
+ FromRef<State>
+ for<'a> Arguments<'a, T = <Extractor as HasValidateArgs<'a>>::ValidateArgs>,
Extractor: for<'v> HasValidateArgs<'v> + FromRequestParts<State>,
for<'v> <Extractor as HasValidateArgs<'v>>::ValidateArgs: ValidateArgs<'v>,
ValidationContext<Args>: FromRef<State>,
{
type Rejection = ValidRejection<<Extractor as FromRequestParts<State>>::Rejection>;
async fn from_request_parts(parts: &mut Parts, state: &State) -> Result<Self, Self::Rejection> {
let ValidationContext { arguments }: ValidationContext<Args> = FromRef::from_ref(state);
let arguments: Args = FromRef::from_ref(state);
let inner = Extractor::from_request_parts(parts, state)
.await
.map_err(ValidRejection::Inner)?;

29
src/test.rs
View File

@@ -1,6 +1,6 @@
use crate::test::extra_typed_path::TypedPathParamExValidationArguments;
use crate::tests::{ValidTest, ValidTestParameter};
use crate::{Arguments, HasValidate, Valid, ValidEx, ValidationContext, VALIDATION_ERROR_STATUS};
use crate::{Arguments, HasValidate, Valid, ValidEx, VALIDATION_ERROR_STATUS};
use axum::extract::{FromRef, Path, Query};
use axum::routing::{get, post};
use axum::{Form, Json, Router};
@@ -11,6 +11,7 @@ use serde::{Deserialize, Serialize};
use std::any::type_name;
use std::net::SocketAddr;
use std::ops::{Deref, RangeInclusive};
use std::sync::Arc;
use validator::{Validate, ValidateArgs, ValidationError};
#[derive(Clone, Deserialize, Serialize, Validate, Eq, PartialEq)]
@@ -56,18 +57,24 @@ fn validate_v1(v: &str, args: &RangeInclusive<usize>) -> Result<(), ValidationEr
}
#[derive(Debug, Clone)]
pub struct ParametersExValidationArguments {
struct ParametersExValidationArgumentsInner {
v0_range: RangeInclusive<i32>,
v1_length_range: RangeInclusive<usize>,
}
impl<'a> Arguments<'a, ParametersEx> for ParametersExValidationArguments {
#[derive(Debug, Clone, Default)]
pub struct ParametersExValidationArguments {
inner: Arc<ParametersExValidationArgumentsInner>,
}
impl<'a> Arguments<'a> for ParametersExValidationArguments {
type T = ParametersEx;
fn get(&'a self) -> <ParametersEx as ValidateArgs<'a>>::Args {
(&self.v0_range, &self.v1_length_range)
(&self.inner.v0_range, &self.inner.v1_length_range)
}
}
impl Default for ParametersExValidationArguments {
impl Default for ParametersExValidationArgumentsInner {
fn default() -> Self {
Self {
v0_range: 5..=10,
@@ -110,16 +117,15 @@ impl HasValidate for Parameters {
#[derive(Debug, Clone, FromRef)]
struct MyState {
param_validation_ctx: ValidationContext<ParametersExValidationArguments>,
typed_path_validation_ctx: ValidationContext<TypedPathParamExValidationArguments>,
param_validation_ctx: ParametersExValidationArguments,
typed_path_validation_ctx: TypedPathParamExValidationArguments,
}
#[tokio::test]
async fn test_main() -> anyhow::Result<()> {
let state = MyState {
param_validation_ctx: ValidationContext::<ParametersExValidationArguments>::default(),
typed_path_validation_ctx:
ValidationContext::<TypedPathParamExValidationArguments>::default(),
param_validation_ctx: ParametersExValidationArguments::default(),
typed_path_validation_ctx: TypedPathParamExValidationArguments::default(),
};
let router = Router::new()
@@ -1068,7 +1074,8 @@ mod extra_typed_path {
}
}
impl<'a> Arguments<'a, TypedPathParamEx> for TypedPathParamExValidationArguments {
impl<'a> Arguments<'a> for TypedPathParamExValidationArguments {
type T = TypedPathParamEx;
fn get(&'a self) -> <TypedPathParamEx as ValidateArgs<'a>>::Args {
(&self.v0_range, &self.v1_length_range)
}