Add details to Invalid Token error

src/error.rs

@@ -9,12 +9,24 @@ use thiserror::Error;
 pub enum Error {
     #[error("Missing TOKEN")]
     MissingToken,
-    #[error("JWT token not valid")]
-    InvalidToken,
+
+    #[error("JWT token not valid - {0}")]
+    InvalidToken(String),
+
     #[error("JWT token creation error")]
     TokenCreation,
 }
 
+impl Error {
+    pub fn status(&self) -> StatusCode {
+        match self {
+            Error::MissingToken => StatusCode::UNAUTHORIZED,
+            Error::InvalidToken(_) => StatusCode::NOT_FOUND,
+            Error::TokenCreation => StatusCode::INTERNAL_SERVER_ERROR
+        }
+    }
+}
+
 #[derive(Serialize, Debug)]
 struct ErrorResponse {
     status: String,
@@ -23,12 +35,6 @@ struct ErrorResponse {
 
 impl IntoResponse for Error {
     fn into_response(self) -> Response {
-        match self {
-            Error::MissingToken => (StatusCode::UNAUTHORIZED, self.to_string()).into_response(),
-            Error::InvalidToken => (StatusCode::NOT_FOUND, self.to_string()).into_response(),
-            Error::TokenCreation => {
-                (StatusCode::INTERNAL_SERVER_ERROR, self.to_string()).into_response()
-            }
-        }
+        (self.status(), self.to_string()).into_response()
     }
 }

src/handler.rs

@@ -38,7 +38,7 @@ pub async fn auth_with_path(Path(token): Path<String>) -> Result<String, Error>
         &jwt::DecodingKey::from_secret(CONFIG.jwt.secret.as_bytes()),
         &jwt::Validation::new(jwt::Algorithm::HS512),
     )
-    .map_err(|_| Error::InvalidToken)?;
+    .map_err(|e| Error::InvalidToken(e.to_string()))?;
 
     Ok(decoded.claims.exp.to_string())
 }
@@ -66,7 +66,7 @@ pub async fn auth_with_header(headers: HeaderMap) -> Result<String, Error> {
         .get("TOKEN")
         .ok_or(Error::MissingToken)?
         .to_str()
-        .map_err(|_| Error::InvalidToken)?;
+        .map_err(|e| Error::InvalidToken(e.to_string()))?;
     validate_jwt(token)
 }
 
@@ -79,13 +79,13 @@ pub async fn auth_with_x_forward_uri(headers: HeaderMap) -> Result<String, Error
     .get("X-Forwarded-Uri")
     .ok_or(Error::MissingToken)?
     .to_str()
-    .map_err(|_| Error::InvalidToken)?;
-    let uri = header.parse::<Uri>().map_err(|_| Error::InvalidToken)?;
+    .map_err(|e| Error::InvalidToken(e.to_string()))?;
+    let uri = header.parse::<Uri>().map_err(|e| Error::InvalidToken(e.to_string()))?;
     match uri.query_to_map().get("token") {
         Some(token) => validate_jwt(token),
         None => Err(Error::MissingToken)
     }
-    
+
 }
 
 /// Validate a JWT token
@@ -95,7 +95,7 @@ pub fn validate_jwt(token: &str) -> Result<String, Error> {
         &jwt::DecodingKey::from_secret(CONFIG.jwt.secret.as_bytes()),
         &jwt::Validation::new(jwt::Algorithm::HS512),
     )
-    .map_err(|_| Error::InvalidToken)?;
+    .map_err(|e| Error::InvalidToken(e.to_string()))?;
 
     Ok(decoded.claims.exp.to_string())
 }
@@ -147,8 +147,8 @@ mod tests {
             ("key", "value"),
             ("foo", "bar"),
             ("token", "123"),
-            
+
         ]);
         assert_eq!(hm, uri.query_to_map());
     }
-}
+}