From 19bf83f50540c40f256543bf24ede38993475f49 Mon Sep 17 00:00:00 2001 From: Iouri Goussev Date: Wed, 25 Nov 2020 12:17:40 -0500 Subject: [PATCH] Logger is not passed required parameter We should either not log the token and remove %s or log the token. Original pull request: gh-599. --- .../vault/authentication/LifecycleAwareSessionManager.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/spring-vault-core/src/main/java/org/springframework/vault/authentication/LifecycleAwareSessionManager.java b/spring-vault-core/src/main/java/org/springframework/vault/authentication/LifecycleAwareSessionManager.java index e1a8c06e..f91e7806 100644 --- a/spring-vault-core/src/main/java/org/springframework/vault/authentication/LifecycleAwareSessionManager.java +++ b/spring-vault-core/src/main/java/org/springframework/vault/authentication/LifecycleAwareSessionManager.java @@ -177,7 +177,7 @@ public class LifecycleAwareSessionManager extends LifecycleAwareSessionManagerSu dispatch(new AfterLoginTokenRevocationEvent(token)); } catch (RuntimeException e) { - this.logger.warn("Cannot revoke VaultToken: %s", e); + this.logger.warn(String.format("Cannot revoke VaultToken: %s", token.getToken()), e); dispatch(new LoginTokenRevocationFailedEvent(token, e)); } }