You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@geode.apache.org by kl...@apache.org on 2016/03/04 20:33:59 UTC
incubator-geode git commit: Remove debug log statements
Repository: incubator-geode
Updated Branches:
refs/heads/feature/GEODE-949-2 f00184c04 -> d74644f16
Remove debug log statements
Project: http://git-wip-us.apache.org/repos/asf/incubator-geode/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-geode/commit/d74644f1
Tree: http://git-wip-us.apache.org/repos/asf/incubator-geode/tree/d74644f1
Diff: http://git-wip-us.apache.org/repos/asf/incubator-geode/diff/d74644f1
Branch: refs/heads/feature/GEODE-949-2
Commit: d74644f16f373eaf1df6885ca62435e2372d82b1
Parents: f00184c
Author: Kirk Lund <kl...@apache.org>
Authored: Fri Mar 4 11:00:59 2016 -0800
Committer: Kirk Lund <kl...@apache.org>
Committed: Fri Mar 4 11:00:59 2016 -0800
----------------------------------------------------------------------
geode-core/src/test/java/security/AuthzCredentialGenerator.java | 3 ---
geode-core/src/test/java/security/SSLCredentialGenerator.java | 2 --
.../src/test/java/templates/security/LdapUserAuthenticator.java | 1 -
geode-core/src/test/java/templates/security/PKCSAuthInit.java | 1 -
.../src/test/java/templates/security/PKCSAuthenticator.java | 2 --
.../src/test/java/templates/security/XmlErrorHandler.java | 5 +----
6 files changed, 1 insertion(+), 13 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/d74644f1/geode-core/src/test/java/security/AuthzCredentialGenerator.java
----------------------------------------------------------------------
diff --git a/geode-core/src/test/java/security/AuthzCredentialGenerator.java b/geode-core/src/test/java/security/AuthzCredentialGenerator.java
index 7ad35d5..fdd10b0 100755
--- a/geode-core/src/test/java/security/AuthzCredentialGenerator.java
+++ b/geode-core/src/test/java/security/AuthzCredentialGenerator.java
@@ -258,7 +258,6 @@ public abstract class AuthzCredentialGenerator {
this.sysProps = init();
}
catch (IllegalArgumentException ex) {
- logger.error("KIRK:SECURITY: ex={}", ex);
return false;
}
return true;
@@ -326,7 +325,6 @@ public abstract class AuthzCredentialGenerator {
return this.cGen.getValidCredentials(principal);
}
catch (IllegalArgumentException ex) {
- logger.error("KIRK:SECURITY: ex={}", ex);
}
}
return null;
@@ -369,7 +367,6 @@ public abstract class AuthzCredentialGenerator {
return this.cGen.getValidCredentials(principal);
}
catch (IllegalArgumentException ex) {
- logger.error("KIRK:SECURITY: ex={}", ex);
}
}
return null;
http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/d74644f1/geode-core/src/test/java/security/SSLCredentialGenerator.java
----------------------------------------------------------------------
diff --git a/geode-core/src/test/java/security/SSLCredentialGenerator.java b/geode-core/src/test/java/security/SSLCredentialGenerator.java
index ae7695f..e547630 100755
--- a/geode-core/src/test/java/security/SSLCredentialGenerator.java
+++ b/geode-core/src/test/java/security/SSLCredentialGenerator.java
@@ -51,7 +51,6 @@ public class SSLCredentialGenerator extends CredentialGenerator {
return props;
}
catch (IOException ex) {
- logger.error("KIRK:SECURITY: ex={}", ex);
throw new AuthenticationFailedException(
"SSL: Exception while opening the key store: " + ex.getMessage(), ex);
}
@@ -68,7 +67,6 @@ public class SSLCredentialGenerator extends CredentialGenerator {
return props;
}
catch (IOException ex) {
- logger.error("KIRK:SECURITY: ex={}", ex);
throw new AuthenticationFailedException(
"SSL: Exception while opening the key store: " + ex.getMessage(), ex);
}
http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/d74644f1/geode-core/src/test/java/templates/security/LdapUserAuthenticator.java
----------------------------------------------------------------------
diff --git a/geode-core/src/test/java/templates/security/LdapUserAuthenticator.java b/geode-core/src/test/java/templates/security/LdapUserAuthenticator.java
index 59fcfaa..49059c3 100755
--- a/geode-core/src/test/java/templates/security/LdapUserAuthenticator.java
+++ b/geode-core/src/test/java/templates/security/LdapUserAuthenticator.java
@@ -105,7 +105,6 @@ public class LdapUserAuthenticator implements Authenticator {
ctx.close();
}
catch (Exception e) {
- logger.error("KIRK:SECURITY: e={}", e);
throw new AuthenticationFailedException(
"LdapUserAuthenticator: Failure with provided username, password "
+ "combination for user name: " + userName, e);
http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/d74644f1/geode-core/src/test/java/templates/security/PKCSAuthInit.java
----------------------------------------------------------------------
diff --git a/geode-core/src/test/java/templates/security/PKCSAuthInit.java b/geode-core/src/test/java/templates/security/PKCSAuthInit.java
index e61feee..f4004f3 100755
--- a/geode-core/src/test/java/templates/security/PKCSAuthInit.java
+++ b/geode-core/src/test/java/templates/security/PKCSAuthInit.java
@@ -125,7 +125,6 @@ public class PKCSAuthInit implements AuthInitialize {
}
}
catch (Exception ex) {
- logger.error("KIRK:SECURITY: ex={}", ex);
throw new AuthenticationFailedException(
"PKCSAuthInit: Exception while getting credentials: " + ex, ex);
}
http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/d74644f1/geode-core/src/test/java/templates/security/PKCSAuthenticator.java
----------------------------------------------------------------------
diff --git a/geode-core/src/test/java/templates/security/PKCSAuthenticator.java b/geode-core/src/test/java/templates/security/PKCSAuthenticator.java
index 7e0b3d1..7af7312 100755
--- a/geode-core/src/test/java/templates/security/PKCSAuthenticator.java
+++ b/geode-core/src/test/java/templates/security/PKCSAuthenticator.java
@@ -86,7 +86,6 @@ public class PKCSAuthenticator implements Authenticator {
}
}
catch (Exception e) {
- logger.error("KIRK:SECURITY: e={}", e);
throw new AuthenticationFailedException(
"Exception while getting public keys: " + e.getMessage(), e);
}
@@ -157,7 +156,6 @@ public class PKCSAuthenticator implements Authenticator {
return new PKCSPrincipal(alias);
}
catch (Exception ex) {
- logger.error("KIRK:SECURITY: ex={}", ex);
throw getException(ex.toString(), ex);
}
}
http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/d74644f1/geode-core/src/test/java/templates/security/XmlErrorHandler.java
----------------------------------------------------------------------
diff --git a/geode-core/src/test/java/templates/security/XmlErrorHandler.java b/geode-core/src/test/java/templates/security/XmlErrorHandler.java
index ab2bf38..1326548 100755
--- a/geode-core/src/test/java/templates/security/XmlErrorHandler.java
+++ b/geode-core/src/test/java/templates/security/XmlErrorHandler.java
@@ -52,7 +52,6 @@ public class XmlErrorHandler implements ErrorHandler {
* where the exception occurred.
*/
public void error(SAXParseException exception) throws SAXException {
- logger.error("KIRK:SECURITY: exception={}", exception);
throw new SAXParseException("Error while parsing XML at line "
+ exception.getLineNumber() + " column " + exception.getColumnNumber()
+ ": " + exception.getMessage(), null, exception);
@@ -63,7 +62,6 @@ public class XmlErrorHandler implements ErrorHandler {
* where the exception occurred.
*/
public void fatalError(SAXParseException exception) throws SAXException {
- logger.error("KIRK:SECURITY: exception={}", exception);
throw new SAXParseException("Fatal error while parsing XML at line "
+ exception.getLineNumber() + " column " + exception.getColumnNumber()
+ ": " + exception.getMessage(), null, exception);
@@ -74,10 +72,9 @@ public class XmlErrorHandler implements ErrorHandler {
* filename and the position of exception in the file.
*/
public void warning(SAXParseException exception) throws SAXException {
- logger.error("KIRK:SECURITY: exception={}", exception);
this.logWriter.warning("Warning while parsing XML [" + this.xmlFileName
+ "] at line " + exception.getLineNumber() + " column "
- + exception.getColumnNumber() + ": " + exception.getMessage());
+ + exception.getColumnNumber() + ": " + exception.getMessage(), exception);
}