Remove Clone requirement from Args

eisberg-labs · Mar 29, 2022 · e02048c · e02048c
1 parent 168cf37
commit e02048c
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 7 deletions.
diff --git a/Cargo.toml b/Cargo.toml
@@ -1,6 +1,6 @@
 [package]
 name = "actix-permissions"
-version = "1.0.0"
+version = "1.0.1"
 edition = "2018"
 authors = ["Ana Bujan <ana@eisberg-labs.com>"]
 readme = "README.md"

diff --git a/src/lib.rs b/src/lib.rs
@@ -46,9 +46,9 @@ pub fn check_with_custom_deny<F, Args, P1, P1Args>(
 ) -> Route
 where
     F: Handler<Args>,
-    Args: FromRequest + 'static + Clone,
+    Args: FromRequest + 'static,
     P1: Permission<P1Args>,
-    P1Args: FromRequest + 'static + Clone,
+    P1Args: FromRequest + 'static,
     F::Output: Responder,
 {
     route.service(fn_factory(move || {

diff --git a/src/tests/test_service.rs b/src/tests/test_service.rs
@@ -51,10 +51,6 @@ mod tests {
 
     #[actix_web::test]
     async fn test_deny_all_default_handler() {
-        fn custom_deny_handler(_req: HttpRequest) -> HttpResponse {
-            HttpResponse::new(StatusCode::FORBIDDEN)
-        }
-
         async fn deny_all(_req: HttpRequest) -> actix_web::Result<bool> {
             Ok(false)
         }