Fix authentication middleware to return errors

GPeaky · Dec 11, 2023 · c8d358d · c8d358d
1 parent 8c3366d
commit c8d358d
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/src/middlewares/admin.rs b/src/middlewares/admin.rs
@@ -17,6 +17,7 @@ impl<S> Middleware<S> for Admin {
     }
 }
 
+// TODO: Fix this fucking shit and return errors
 impl<S, Err> Service<web::WebRequest<Err>> for AdminMiddleware<S>
 where
     S: Service<web::WebRequest<Err>, Response = web::WebResponse, Error = web::Error>,

diff --git a/src/middlewares/authenticated.rs b/src/middlewares/authenticated.rs
@@ -27,6 +27,7 @@ pub struct AuthenticationMiddleware<S> {
     service: S,
 }
 
+// TODO: Fix this fucking shit and return errors
 impl<S, Err> Service<web::WebRequest<Err>> for AuthenticationMiddleware<S>
 where
     S: Service<web::WebRequest<Err>, Response = web::WebResponse, Error = web::Error>,
@@ -74,9 +75,6 @@ where
             ctx.call(&self.service, req).await
         };
 
-        Box::pin(async move {
-            let res = fut.await?;
-            Ok(res)
-        })
+        Box::pin(fut)
     }
 }