[spring-extensions] branch maint-5 updated: JSE-31 Turn off INFO level warnings for conditional non-existing files.

Rod Widdowson rdw at steadingsoftware.com
Fri Jan 4 11:00:23 EST 2019


This is an automated email from the git hooks/post-receive script.

rdw pushed a commit to branch maint-5
in repository spring-extensions.

View the commit online:
http://git.shibboleth.net/view/?p=spring-extensions.git;a=commit;h=6093e12264afa78c7eae4dd8aea63be2c8a757b0

The following commit(s) were added to refs/heads/maint-5 by this push:
       new  6093e12   JSE-31 Turn off INFO level warnings for conditional non-existing files.
6093e12 is described below

commit 6093e12264afa78c7eae4dd8aea63be2c8a757b0
Author: Rod Widdowson <rdw at steadingsoftware.com>
AuthorDate: Wed Jan 2 15:53:05 2019 +0000

    JSE-31 Turn off INFO level warnings for conditional non-existing files.
    
    https://issues.shibboleth.net/jira/browse/JSE-31
---
 .../ext/spring/resource/ConditionalResource.java           | 14 +-------------
 1 file changed, 1 insertion(+), 13 deletions(-)

diff --git a/src/main/java/net/shibboleth/ext/spring/resource/ConditionalResource.java b/src/main/java/net/shibboleth/ext/spring/resource/ConditionalResource.java
index 371b037..24c24df 100644
--- a/src/main/java/net/shibboleth/ext/spring/resource/ConditionalResource.java
+++ b/src/main/java/net/shibboleth/ext/spring/resource/ConditionalResource.java
@@ -83,8 +83,6 @@ public class ConditionalResource extends AbstractIdentifiedInitializableComponen
         } catch (final IOException e) {
             if (log.isDebugEnabled()) {
                 log.debug("{} getInputStream failed on wrapped resource", getLogPrefix(), e);
-            } else {
-                log.info("{} getInputStream failed on wrapped resource", getLogPrefix());
             }
             return new ByteArrayInputStream(EMPTY_RESOURCE.getBytes(StandardCharsets.UTF_8));
         }
@@ -110,7 +108,7 @@ public class ConditionalResource extends AbstractIdentifiedInitializableComponen
     /** {@inheritDoc} */
     public boolean exists() {
         if (!wrappedResource.exists()) {
-            log.info("{} Wrapped resource does not exist", getLogPrefix());
+            log.debug("{} Wrapped resource does not exist", getLogPrefix());
         }
         return true;
     }
@@ -132,8 +130,6 @@ public class ConditionalResource extends AbstractIdentifiedInitializableComponen
         } catch (final IOException e) {
             if (log.isDebugEnabled()) {
                 log.debug("{} getURL failed on wrapped resource", getLogPrefix(), e);
-            } else {
-                log.info("{} getURL failed on wrapped resource", getLogPrefix());
             }
             return null;
         }
@@ -146,8 +142,6 @@ public class ConditionalResource extends AbstractIdentifiedInitializableComponen
         } catch (final IOException e) {
             if (log.isDebugEnabled()) {
                 log.debug("{} getURI failed on wrapped resource", getLogPrefix(), e);
-            } else {
-                log.info("{} getURI failed on wrapped resource", getLogPrefix());
             }
             return null;
         }
@@ -160,8 +154,6 @@ public class ConditionalResource extends AbstractIdentifiedInitializableComponen
         } catch (final IOException e) {
             if (log.isDebugEnabled()) {
                 log.debug("{} getFile failed on wrapped resource", getLogPrefix(), e);
-            } else {
-                log.info("{} getFile failed on wrapped resource", getLogPrefix());
             }
             return null;
         }
@@ -174,8 +166,6 @@ public class ConditionalResource extends AbstractIdentifiedInitializableComponen
         } catch (final IOException e) {
             if (log.isDebugEnabled()) {
                 log.debug("{} contentLength failed on wrapped resource", getLogPrefix(), e);
-            } else {
-                log.info("{} contentLength failed on wrapped resource", getLogPrefix());
             }
             return EMPTY_RESOURCE.length();
         }
@@ -188,8 +178,6 @@ public class ConditionalResource extends AbstractIdentifiedInitializableComponen
         } catch (final IOException e) {
             if (log.isDebugEnabled()) {
                 log.debug("{} lastModified failed on wrapped resource", getLogPrefix(), e);
-            } else {
-                log.info("{} lastModified failed on wrapped resource", getLogPrefix());
             }
             return 0;
         }

-- 
To stop receiving notification emails like this one, please contact
the administrator of this repository.


More information about the commits mailing list