Add CI clippy checks for gateway

.github/workflows/build.yml

@@ -556,6 +556,13 @@ jobs:
             cargo fmt --check
           exit $(docker wait gateway-format)
 
+      - name: Clippy check
+        run: |
+          docker run --name=gateway-clippy --net=host -v $PWD/output:/output \
+            ${{ fromJSON(needs.build.outputs.stable_tags).gateway-test }} \
+            cargo clippy --all-features --all-targets -- -D warnings
+          exit $(docker wait gateway-clippy)
+
   check_front:
     runs-on: ubuntu-latest
     needs:

gateway/Dockerfile

@@ -28,6 +28,7 @@ RUN cargo install --locked --path .
 FROM chef AS testing_env
 RUN rustup component add llvm-tools && \
     rustup component add rustfmt && \
+    rustup component add clippy && \
     cargo install --locked grcov
 COPY --from=planner /app/recipe.json recipe.json
 

gateway/actix_auth/src/auth_middleware.rs

@@ -1,19 +1,17 @@
-use std::{
-    future::{ready, Ready},
-    rc::Rc,
-};
+use actix_session::SessionExt;
+use actix_web::dev::forward_ready;
+use actix_web::dev::Service;
+use actix_web::dev::ServiceRequest;
+use actix_web::dev::ServiceResponse;
+use actix_web::dev::Transform;
+use actix_web::Error;
+use actix_web::HttpMessage;
+use std::future::ready;
+use std::future::Ready;
+use std::rc::Rc;
 
-use actix_session::{Session, SessionExt};
-use actix_web::{
-    dev::{forward_ready, Service, ServiceRequest, ServiceResponse, Transform},
-    Error, HttpMessage, HttpRequest,
-};
-
-use crate::{AuthContext, RequestAuth};
-
-pub trait AuthGate: Clone {
-    fn filter(&self, req: &HttpRequest, session: Option<Session>) -> Option<actix_web::Error>;
-}
+use crate::AuthContext;
+use crate::RequestAuth;
 
 /// The MandatoryIdentityMiddleware does not directly process requests.
 /// When actix starts, new_transform is called once per worker to create