Skip to content

Commit 268d023

Browse files
Vladimir Kotalahornace
andcommitted
Update plugins/src/main/java/opengrok/auth/plugin/LdapAttrPlugin.java
accept review comment Co-authored-by: Adam Hornáček <[email protected]>
1 parent e008682 commit 268d023

File tree

1 file changed

+2
-3
lines changed

1 file changed

+2
-3
lines changed

plugins/src/main/java/opengrok/auth/plugin/LdapAttrPlugin.java

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -128,9 +128,8 @@ String getSessionAllowedAttrName() {
128128
public void fillSession(HttpServletRequest req, User user) {
129129
updateSession(req, false);
130130

131-
LdapUser ldapUser;
132-
if ((ldapUser = (LdapUser) req.getSession().
133-
getAttribute(LdapUserPlugin.getSessionAttrName(ldapUserInstance))) == null) {
131+
LdapUser ldapUser = (LdapUser) req.getSession(). getAttribute(LdapUserPlugin.getSessionAttrName(ldapUserInstance));
132+
if (ldapUser == null) {
134133
LOGGER.log(Level.WARNING, "cannot get {0} attribute from {1}",
135134
new Object[]{LdapUserPlugin.SESSION_ATTR, user});
136135
return;

0 commit comments

Comments
 (0)