Spring/spring-ldap
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Fixed debug log format strings

Browse Source 
There were a few '%s$1' instead of '%1$s', which caused "$1" to be written to logs.
This commit is contained in:
mfijas
2017-10-21 17:50:36 +02:00
committed by Josh Cummings
parent 02dd0731fd
commit aead910d64
1 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
core/src/main/java/org/springframework/ldap/core/LdapTemplate.java
View File

@@ -1686,7 +1686,7 @@ public class LdapTemplate implements LdapOperations, InitializingBean {
@Override
public <T> T findByDn(Name dn, final Class<T> clazz) {
if (LOG.isDebugEnabled()) {
LOG.debug(String.format("Reading Entry at - %s$1", dn));
LOG.debug(String.format("Reading Entry at - %1$s", dn));
}
// Make sure the class is OK before doing the lookup
@@ -1703,7 +1703,7 @@ public class LdapTemplate implements LdapOperations, InitializingBean {
throw new OdmException(String.format("Entry %1$s does not have the required objectclasses ", dn));
}
if (LOG.isDebugEnabled()) {
LOG.debug(String.format("Found entry - %s$1", result));
LOG.debug(String.format("Found entry - %1$s", result));
}
return result;
@@ -1717,7 +1717,7 @@ public class LdapTemplate implements LdapOperations, InitializingBean {
Assert.notNull(entry, "Entry must not be null");
if (LOG.isDebugEnabled()) {
LOG.debug(String.format("Creating entry - %s$1", entry));
LOG.debug(String.format("Creating entry - %1$s", entry));
}
Name id = odm.getId(entry);
@@ -1741,7 +1741,7 @@ public class LdapTemplate implements LdapOperations, InitializingBean {
public void update(Object entry) {
Assert.notNull(entry, "Entry must not be null");
if (LOG.isDebugEnabled()) {
LOG.debug(String.format("Updating entry - %s$1", entry));
LOG.debug(String.format("Updating entry - %1$s", entry));
}
Name originalId = odm.getId(entry);
@@ -1786,7 +1786,7 @@ public class LdapTemplate implements LdapOperations, InitializingBean {
public void delete(Object entry) {
Assert.notNull(entry, "Entry must not be null");
if (LOG.isDebugEnabled()) {
LOG.debug(String.format("Deleting %s$1", entry));
LOG.debug(String.format("Deleting %1$s", entry));
}
Name id = odm.getId(entry);