Fixed error logging

pull/210/head
Justin Richer 2012-09-10 16:50:16 -04:00
parent 7aec0dfe80
commit 920b2a59ba
17 changed files with 29 additions and 29 deletions

View File

@ -142,9 +142,9 @@ public class HmacSigner extends AbstractJwtSigner implements InitializingBean {
mac.update(signatureBase.getBytes("UTF-8")); mac.update(signatureBase.getBytes("UTF-8"));
} catch (GeneralSecurityException e) { } catch (GeneralSecurityException e) {
logger.error("GeneralSecurityException in HmacSigner.java: " + e.getStackTrace()); logger.error("GeneralSecurityException in HmacSigner.java: ", e);
} catch (UnsupportedEncodingException e) { } catch (UnsupportedEncodingException e) {
logger.error("UnsupportedEncodingException in HmacSigner.java: " + e.getStackTrace()); logger.error("UnsupportedEncodingException in HmacSigner.java: ", e);
} }
byte[] sigBytes = mac.doFinal(); byte[] sigBytes = mac.doFinal();

View File

@ -184,9 +184,9 @@ public class RsaSigner extends AbstractJwtSigner implements InitializingBean {
sig = (new String(Base64.encodeBase64URLSafe(sigBytes))).replace("=", ""); sig = (new String(Base64.encodeBase64URLSafe(sigBytes))).replace("=", "");
} catch (GeneralSecurityException e) { } catch (GeneralSecurityException e) {
logger.error("GeneralSecurityException in RsaSigner.java: " + e.getStackTrace()); logger.error("GeneralSecurityException in RsaSigner.java: ", e);
} catch (UnsupportedEncodingException e) { } catch (UnsupportedEncodingException e) {
logger.error("UnsupportedEncodingException in RsaSigner.java: " + e.getStackTrace()); logger.error("UnsupportedEncodingException in RsaSigner.java: ", e);
} }
return sig; return sig;
@ -281,9 +281,9 @@ public class RsaSigner extends AbstractJwtSigner implements InitializingBean {
signer.update(signingInput.getBytes("UTF-8")); signer.update(signingInput.getBytes("UTF-8"));
value = signer.verify(Base64.decodeBase64(s64)); value = signer.verify(Base64.decodeBase64(s64));
} catch (GeneralSecurityException e) { } catch (GeneralSecurityException e) {
logger.error("GeneralSecurityException in RsaSigner.java: " + e.getStackTrace()); logger.error("GeneralSecurityException in RsaSigner.java: ", e);
} catch (UnsupportedEncodingException e) { } catch (UnsupportedEncodingException e) {
logger.error("UnsupportedEncodingException in RsaSigner.java: " + e.getStackTrace()); logger.error("UnsupportedEncodingException in RsaSigner.java: ", e);
} }
return value; return value;

View File

@ -65,9 +65,9 @@ public class KeyFetcher {
X509Certificate cert = (X509Certificate) factory.generateCertificate(x509Stream); X509Certificate cert = (X509Certificate) factory.generateCertificate(x509Stream);
key = cert.getPublicKey(); key = cert.getPublicKey();
} catch (HttpClientErrorException e) { } catch (HttpClientErrorException e) {
logger.error("HttpClientErrorException in KeyFetcher.java: " + e.getStackTrace()); logger.error("HttpClientErrorException in KeyFetcher.java: ", e);
} catch (CertificateException e) { } catch (CertificateException e) {
logger.error("CertificateException in KeyFetcher.java: " + e.getStackTrace()); logger.error("CertificateException in KeyFetcher.java: ", e);
} }
return key; return key;
@ -97,11 +97,11 @@ public class KeyFetcher {
} }
} catch (HttpClientErrorException e) { } catch (HttpClientErrorException e) {
logger.error("HttpClientErrorException in KeyFetcher.java: " + e.getStackTrace()); logger.error("HttpClientErrorException in KeyFetcher.java: ", e);
} catch (NoSuchAlgorithmException e) { } catch (NoSuchAlgorithmException e) {
logger.error("NoSuchAlgorithmException in KeyFetcher.java: " + e.getStackTrace()); logger.error("NoSuchAlgorithmException in KeyFetcher.java: ", e);
} catch (InvalidKeySpecException e) { } catch (InvalidKeySpecException e) {
logger.error("InvalidKeySpecException in KeyFetcher.java: " + e.getStackTrace()); logger.error("InvalidKeySpecException in KeyFetcher.java: ", e);
} }
return pub; return pub;
} }

View File

@ -122,7 +122,7 @@ public class JwkKeyListView extends AbstractView {
} catch (IOException e) { } catch (IOException e) {
logger.error("IOException in JwkKeyListView.java: " + e.getStackTrace()); logger.error("IOException in JwkKeyListView.java: ", e);
} }

View File

@ -94,15 +94,15 @@ public class X509CertificateView extends AbstractView {
X509Certificate cert = v3CertGen.generate(rsaSigner.getPrivateKey()); X509Certificate cert = v3CertGen.generate(rsaSigner.getPrivateKey());
pemWriter.writeObject(cert); pemWriter.writeObject(cert);
} catch (CertificateEncodingException e) { } catch (CertificateEncodingException e) {
logger.error("CertificateEncodingException in X509CertificateView.java: " + e.getStackTrace()); logger.error("CertificateEncodingException in X509CertificateView.java: ", e);
} catch (InvalidKeyException e) { } catch (InvalidKeyException e) {
logger.error("InvalidKeyException in X509CertificateView.java: " + e.getStackTrace()); logger.error("InvalidKeyException in X509CertificateView.java: ", e);
} catch (IllegalStateException e) { } catch (IllegalStateException e) {
logger.error("IllegalStateException in X509CertificateView.java" + e.getStackTrace()); logger.error("IllegalStateException in X509CertificateView.java", e);
} catch (NoSuchAlgorithmException e) { } catch (NoSuchAlgorithmException e) {
logger.error("NoSuchAlgorithmException in X509CertificateView.java" + e.getStackTrace()); logger.error("NoSuchAlgorithmException in X509CertificateView.java", e);
} catch (SignatureException e) { } catch (SignatureException e) {
logger.error("SignatureException in X509CertificateView.java" + e.getStackTrace()); logger.error("SignatureException in X509CertificateView.java", e);
} finally { } finally {
pemWriter.flush(); pemWriter.flush();
writer.flush(); writer.flush();

View File

@ -118,7 +118,7 @@ public class TokenIntrospection extends AbstractView {
} catch (IOException e) { } catch (IOException e) {
logger.error("IOException occurred in TokenIntrospection.java: " + e.getStackTrace()); logger.error("IOException occurred in TokenIntrospection.java: ", e);
} }

View File

@ -52,7 +52,7 @@ public class ClientAssociateView extends AbstractView {
} catch (IOException e) { } catch (IOException e) {
logger.error("IOException " + e.getStackTrace()); logger.error("IOException ", e);
} }

View File

@ -47,7 +47,7 @@ public class ClientUpdateView extends AbstractView {
} catch (IOException e) { } catch (IOException e) {
logger.error("IOException " + e.getStackTrace()); logger.error("IOException ", e);
} }

View File

@ -68,7 +68,7 @@ public class ExceptionAsJSONView extends AbstractView {
} catch (IOException e) { } catch (IOException e) {
logger.error("IOException in ExceptionAsJSONView.java: " + e.getStackTrace()); logger.error("IOException in ExceptionAsJSONView.java: ", e);
} }
} }

View File

@ -66,7 +66,7 @@ public class JSONClientView extends AbstractView {
} catch (IOException e) { } catch (IOException e) {
logger.error("IOException in JSONClientView.java: " + e.getStackTrace()); logger.error("IOException in JSONClientView.java: ", e);
} }
} }

View File

@ -79,7 +79,7 @@ public class JSONUserInfoView extends AbstractView {
} catch (IOException e) { } catch (IOException e) {
logger.error("IOException in JSONUserInfoView.java: " + e.getStackTrace()); logger.error("IOException in JSONUserInfoView.java: ", e);
} }

View File

@ -61,7 +61,7 @@ public class JsonEntityView extends AbstractView {
} catch (IOException e) { } catch (IOException e) {
logger.error("IOException in JsonEntityView.java: " + e.getStackTrace()); logger.error("IOException in JsonEntityView.java: ", e);
} }
} }

View File

@ -80,7 +80,7 @@ public class POCOUserInfoView extends AbstractView {
} catch (IOException e) { } catch (IOException e) {
logger.error("IOException in POCOUserInfoView.java: " + e.getStackTrace()); logger.error("IOException in POCOUserInfoView.java: ", e);
} }

View File

@ -61,7 +61,7 @@ public class StatsSummary extends AbstractView {
} catch (IOException e) { } catch (IOException e) {
logger.error("IOException in JSONClientView.java: " + e.getStackTrace()); logger.error("IOException in JSONClientView.java: ", e);
} }

View File

@ -76,7 +76,7 @@ public class JsonOpenIdConfigurationView extends AbstractView {
} catch (IOException e) { } catch (IOException e) {
logger.error("IOException in JsonOpenIdConfigurationView.java: " + e.getStackTrace()); logger.error("IOException in JsonOpenIdConfigurationView.java: ", e);
} }

View File

@ -86,7 +86,7 @@ public class SwdResponse extends AbstractView {
} catch (IOException e) { } catch (IOException e) {
logger.error("IOException in SwdResponse.java: " + e.getStackTrace()); logger.error("IOException in SwdResponse.java: ", e);
} }

View File

@ -98,7 +98,7 @@ public class XrdJsonResponse extends AbstractView {
} catch (IOException e) { } catch (IOException e) {
logger.error("IOException in XrdJsonResponse.java: " + e.getStackTrace()); logger.error("IOException in XrdJsonResponse.java: ", e);
} }
} }