true enough....

feel free to adjust....

bye
norman

Am Sonntag, 9. Oktober 2011 schrieb Eric Charles <e...@apache.org>:
> What about logging with error (getLog().error(...)).
> error will also be shown with INFO level.
>
> The HumanReadableText relate to FAILURE, so that sounds more like an
error...
>
> WDYT?
>
> Eric
>
>
> On 08/10/11 19:42, nor...@apache.org wrote:
>
> Author: norman
> Date: Sat Oct  8 17:42:39 2011
> New Revision: 1180423
>
> URL: http://svn.apache.org/viewvc?rev=1180423&view=rev
> Log:
> Log "fatal" MailboxExceptions via info level. See IMAP-343
>
> Modified:
>
james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/AbstractAuthProcessor.java
>
james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/AbstractMailboxProcessor.java
>
james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/AbstractSelectionProcessor.java
>
james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/AppendProcessor.java
>
james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/CloseProcessor.java
>
james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/CopyProcessor.java
>
james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/CreateProcessor.java
>
james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/DeleteProcessor.java
>
james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/EnableProcessor.java
>
james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/ExpungeProcessor.java
>
james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/IdleProcessor.java
>
james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/LSubProcessor.java
>
james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/ListProcessor.java
>
james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/LogoutProcessor.java
>
james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/RenameProcessor.java
>
james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/SearchProcessor.java
>
james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/StatusProcessor.java
>
james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/StoreProcessor.java
>
james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/SubscribeProcessor.java
>
james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/UnsubscribeProcessor.java
>
james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/fetch/FetchProcessor.java
>
> Modified:
james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/AbstractAuthProcessor.java
> URL:
http://svn.apache.org/viewvc/james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/AbstractAuthProcessor.java?rev=1180423&r1=1180422&r2=1180423&view=diff
>
==============================================================================
> ---
james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/AbstractAuthProcessor.java
(original)
> +++
james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/AbstractAuthProcessor.java
Sat Oct  8 17:42:39 2011
> @@ -90,7 +90,7 @@ public abstract class AbstractAuthProces
>                  }
>              }
>          } catch (MailboxException e) {
> -            session.getLog().debug("Login failed", e);
> +            session.getLog().info("Login failed", e);
>              no(command, tag, responder,
HumanReadableText.GENERIC_FAILURE_DURING_PROCESSING);
>          }
>      }
>
> Modified:
james/imap/trunk/processor/src/main/java/org/apache/james/imap/processor/AbstractMailboxProcessor.java
>
> --
> Eric
> http://about.echarles.net
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: server-dev-unsubscr...@james.apache.org
> For additional commands, e-mail: server-dev-h...@james.apache.org
>
>

Reply via email to