StringUtils.join for log

This commit is contained in:
Terry Brady
2016-02-03 15:15:26 -08:00
parent bcfe75dbef
commit 2ae5b47f2a

View File

@@ -317,7 +317,7 @@ public class ShibAuthentication implements AuthenticationMethod
if (affiliations == null) { if (affiliations == null) {
if (defaultRoles != null) if (defaultRoles != null)
affiliations = Arrays.asList(defaultRoles); affiliations = Arrays.asList(defaultRoles);
log.debug("Failed to find Shibboleth role header, '"+roleHeader+"', falling back to the default roles: '"+String.join(",", defaultRoles) +"'"); log.debug("Failed to find Shibboleth role header, '"+roleHeader+"', falling back to the default roles: '"+ StringUtils.join(defaultRoles, ",") + "'");
} else { } else {
log.debug("Found Shibboleth role header: '"+roleHeader+"' = '"+affiliations+"'"); log.debug("Found Shibboleth role header: '"+roleHeader+"' = '"+affiliations+"'");
} }
@@ -351,7 +351,7 @@ public class ShibAuthentication implements AuthenticationMethod
log.debug("Unable to find role mapping for the value, '"+affiliation+"', there should be a mapping in config/modules/authentication-shibboleth.cfg: role."+affiliation+" = <some group name>"); log.debug("Unable to find role mapping for the value, '"+affiliation+"', there should be a mapping in config/modules/authentication-shibboleth.cfg: role."+affiliation+" = <some group name>");
continue; continue;
} else { } else {
log.debug("Mapping role affiliation to DSpace group: '"+String.join(",",groupNames)+"'"); log.debug("Mapping role affiliation to DSpace group: '"+ StringUtils.join(groupNames, ",") +"'");
} }
// Add each group to the list. // Add each group to the list.
@@ -923,7 +923,7 @@ public class ShibAuthentication implements AuthenticationMethod
return; return;
} }
log.debug("Loading additional eperson metadata from: 'authentication.shib.eperson.metadata' = '"+String.join(",", mappingString) +"'"); log.debug("Loading additional eperson metadata from: 'authentication.shib.eperson.metadata' = '"+ StringUtils.join(mappingString, ",") +"'");
for (String metadataString : mappingString) { for (String metadataString : mappingString) {