admin: update log messages for admin backends

The admin backend is only the first step in granting access to an admin
endpoint. The admin backend validates the admin token. Next the role
extracted from that token is checked.

As such, change the logging in the admin backends to talk about the
admin token validation passing/failing rather than access being granted
to an endpoint.

Signed-off-by: Tobin Feldman-Fitzthum <[email protected]>

Author: fitzthum

kbs/src/admin/allow_all.rs

@@ -13,7 +13,7 @@ pub struct InsecureAllowAllBackend {}
 
 impl AdminBackend for InsecureAllowAllBackend {
     fn validate_admin_token(&self, _request: &HttpRequest) -> Result<String> {
-        warn!("Allow All admin backend is set. Anyone can access admin APIs");
+        warn!("Allow All admin backend is set. All admin tokens are valid.");
         Ok("Anonymous".to_string())
     }
 }

kbs/src/admin/simple.rs

@@ -66,13 +66,16 @@ impl AdminBackend for SimpleAdminBackend {
                 .verify_token::<NoCustomClaims>(token, Some(VerificationOptions::default()));
             match res {
                 Ok(_claims) => {
-                    info!("Admin access granted for {}", persona.id);
+                    info!("Admin token validated (simple) for persona: {}", persona.id);
 
                     // Return the first matching persona
                     return Ok(persona.id.clone());
                 }
                 Err(e) => {
-                    info!("Access not granted for {} due to: \n{}", persona.id, e);
+                    info!(
+                        "Admin token not validated for {} due to: \n{}",
+                        persona.id, e
+                    );
                 }
             }
         }