Lower log levels (#39)

Some logs indicated problems where there are none,
so we lowered some logs from 'warn' to 'info' or
'debug' and from 'error' to 'warn'.

Author: pfeuffer

gradle/changelog/lower_log_levels.yaml

@@ -0,0 +1,2 @@
+- type: added
+  description: Lower log levels ([#39](https://github.com/scm-manager/scm-ldap-plugin/pull/39))

src/main/java/sonia/scm/auth/ldap/LdapGroupResolver.java

@@ -80,7 +80,7 @@ public Set<String> resolve(String principal) {
       try {
         return sanitizeGroupNames(resolveGroups(config, principal), config);
       } catch (LdapException ex) {
-        LOG.error("failed to resolve groups for principal: {}", principal, ex);
+        LOG.warn("failed to resolve groups for principal: {}", principal, ex);
       }
     } else {
       LOG.debug("ldap is disabled, returning empty set of groups");

src/main/java/sonia/scm/auth/ldap/LdapUtil.java

@@ -110,7 +110,7 @@ public static void close(Context context) {
       try {
         context.close();
       } catch (NamingException ex) {
-        logger.error("could not close context", ex);
+        logger.warn("could not close context", ex);
       }
     }
   }
@@ -125,7 +125,7 @@ public static void close(StartTlsResponse tls) {
       try {
         tls.close();
       } catch (IOException ex) {
-        logger.error("could not close tls response", ex);
+        logger.warn("could not close tls response", ex);
       }
     }
   }
@@ -140,7 +140,7 @@ public static void close(NamingEnumeration<?> enm) {
       try {
         enm.close();
       } catch (NamingException ex) {
-        logger.error("could not close enumeration", ex);
+        logger.warn("could not close enumeration", ex);
       }
     }
   }
@@ -168,7 +168,7 @@ public static String getAttribute(Attributes attributes, String name) {
         }
       }
     } catch (NamingException ex) {
-      logger.warn("could not fetch attribute ".concat(name), ex);
+      logger.warn("could not fetch attribute '{}'", name, ex);
     }
 
     return value;
@@ -227,10 +227,10 @@ public static int getSearchScope(String scopeString) {
       } else if (SCOPE_OBJECT.equalsIgnoreCase(scopeString)) {
         scope = SearchControls.OBJECT_SCOPE;
       } else if (logger.isWarnEnabled()) {
-        logger.warn("unknown scope {}, using subtree scope", scopeString);
+        logger.info("unknown scope {}, using subtree scope", scopeString);
       }
     } else if (logger.isWarnEnabled()) {
-      logger.warn("no search scope defined, using subtree scope");
+      logger.info("no search scope defined, using subtree scope");
     }
 
     return scope;

src/main/java/sonia/scm/auth/ldap/UserSearcher.java

@@ -57,7 +57,7 @@ Optional<SearchResult> search(String username, String... attributes) {
       if (searchResultEnm.hasMore()) {
         return Optional.of(searchResultEnm.next());
       } else {
-        logger.warn("no user with username {} found", username);
+        logger.debug("no user with username {} found", username);
         return Optional.empty();
       }
     } catch (NamingException ex) {