Cwikius-Spring/spring-security
1
0
Fork 0
mirror of https://github.com/spring-projects/spring-security.git synced 2025-05-31 17:22:13 +00:00
Code Issues Packages Projects Releases Wiki Activity

Add guard around logger.debug statement

Browse Source 
The log message involves string concatenation, the cost of which should only be incurred if debug logging is enabled
This commit is contained in:
Craig Andrews 2021-04-16 11:59:01 -04:00 committed by Josh Cummings
parent a31a855146
commit 7dc4de05b1
1 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
web/src/main/java/org/springframework/security/web/session/SimpleRedirectInvalidSessionStrategy.java
View File

@ -52,7 +52,9 @@ public final class SimpleRedirectInvalidSessionStrategy implements InvalidSessio
@Override @Override
public void onInvalidSessionDetected(HttpServletRequest request, HttpServletResponse response) throws IOException { public void onInvalidSessionDetected(HttpServletRequest request, HttpServletResponse response) throws IOException {
this.logger.debug("Starting new session (if required) and redirecting to '" + this.destinationUrl + "'"); if (this.logger.isDebugEnabled()) {
this.logger.debug("Starting new session (if required) and redirecting to '" + this.destinationUrl + "'");
}
if (this.createNewSession) { if (this.createNewSession) {
request.getSession(); request.getSession();
} }