This is an automated email from the ASF dual-hosted git repository. markt pushed a commit to branch 8.5.x in repository https://gitbox.apache.org/repos/asf/tomcat.git
commit dd8c2e5f7f891b91ef38b789ca28c39c39824b6b Author: Yordanov <[email protected]> AuthorDate: Thu Jul 13 12:04:25 2023 +0200 StandardSession.java @doWriteObject: adds null-check to sessionPrincipal before checking if serializable to prevent misleading warning message logged. --- java/org/apache/catalina/session/StandardSession.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/java/org/apache/catalina/session/StandardSession.java b/java/org/apache/catalina/session/StandardSession.java index b25c999373..7a0e975424 100644 --- a/java/org/apache/catalina/session/StandardSession.java +++ b/java/org/apache/catalina/session/StandardSession.java @@ -1592,7 +1592,7 @@ public class StandardSession implements HttpSession, Session, Serializable { if (getPersistAuthentication()) { sessionAuthType = getAuthType(); sessionPrincipal = getPrincipal(); - if (!(sessionPrincipal instanceof Serializable)) { + if (sessionPrincipal != null && !(sessionPrincipal instanceof Serializable)) { sessionPrincipal = null; manager.getContext().getLogger().warn(sm.getString("standardSession.principalNotSerializable", id)); } --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
