]> source.dussan.org Git - vaadin-framework.git/commitdiff
#5715 server-side logging: some changes to the used log levels
authorHenri Sara <henri.sara@itmill.com>
Mon, 25 Oct 2010 13:07:24 +0000 (13:07 +0000)
committerHenri Sara <henri.sara@itmill.com>
Mon, 25 Oct 2010 13:07:24 +0000 (13:07 +0000)
svn changeset:15692/svn branch:6.5

src/com/vaadin/Application.java
src/com/vaadin/event/ListenerMethod.java
src/com/vaadin/terminal/gwt/server/AbstractApplicationPortlet.java
src/com/vaadin/terminal/gwt/server/AbstractApplicationServlet.java
src/com/vaadin/terminal/gwt/server/AbstractCommunicationManager.java
src/com/vaadin/terminal/gwt/server/ApplicationRunnerServlet.java
src/com/vaadin/terminal/gwt/server/GAEApplicationServlet.java
src/com/vaadin/terminal/gwt/server/PortletApplicationContext2.java
src/com/vaadin/terminal/gwt/widgetsetutils/ClassPathExplorer.java

index 9650146f531871b0b8dab5f17c48a7717fc89cd1..d1962f4551fa418410849759a0484b892de5831e 100644 (file)
@@ -1191,7 +1191,7 @@ public abstract class Application implements URIHandler,
         final Throwable t = event.getThrowable();
         if (t instanceof SocketException) {
             // Most likely client browser closed socket
-            logger.warning("SocketException in CommunicationManager."
+            logger.info("SocketException in CommunicationManager."
                     + " Most likely client (browser) closed socket.");
             return;
         }
@@ -1219,7 +1219,7 @@ public abstract class Application implements URIHandler,
         }
 
         // also print the error on console
-        logger.log(Level.SEVERE, "Terminal error:", t);
+        logger.log(Level.INFO, "Terminal error:", t);
     }
 
     /**
index 8c7aa93f81dc8674e8b039cda25a8bbad0676d40..b8ec0b1828add32a38112f6239abd1e359899a03 100644 (file)
@@ -84,7 +84,7 @@ public class ListenerMethod implements EventListener, Serializable {
             out.writeObject(name);
             out.writeObject(paramTypes);
         } catch (NotSerializableException e) {
-            logger.severe("Fatal error in serialization of the application: Class "
+            logger.warning("Error in serialization of the application: Class "
                     + object.getClass().getName()
                     + " must implement serialization.");
             throw e;
index 8f806ef69328abbfcfec6315d4f749a13cc02410..953ab0c5d48f9ab7ea770c8dd15af7bf67626b27 100644 (file)
@@ -455,7 +455,7 @@ public abstract class AbstractApplicationPortlet extends GenericPortlet
             } catch (final GeneralSecurityException e) {
                 // TODO Figure out a better way to deal with
                 // GeneralSecurityExceptions
-                logger.finest("General security exception, the security key was probably incorrect.");
+                logger.fine("General security exception, the security key was probably incorrect.");
             } catch (final Throwable e) {
                 handleServiceException(request, response, application, e);
             } finally {
@@ -659,7 +659,7 @@ public abstract class AbstractApplicationPortlet extends GenericPortlet
                 os.write(buffer, 0, bytes);
             }
         } else {
-            logger.warning("Requested resource [" + resourceID
+            logger.info("Requested resource [" + resourceID
                     + "] could not be found");
             response.setProperty(ResourceResponse.HTTP_STATUS_CODE,
                     Integer.toString(HttpServletResponse.SC_NOT_FOUND));
index 5fe94266eb7373179d210399a41283af7e069563..3e8912770bb6db0ddf4c9c1541d2b3ae30e110bb 100644 (file)
@@ -821,7 +821,7 @@ public abstract class AbstractApplicationServlet extends HttpServlet implements
                 resultPath = url.getFile();
             } catch (final Exception e) {
                 // FIXME: Handle exception
-                logger.log(Level.WARNING, "Could not find resource path "
+                logger.log(Level.INFO, "Could not find resource path "
                         + path, e);
             }
         }
@@ -1203,7 +1203,7 @@ public abstract class AbstractApplicationServlet extends HttpServlet implements
 
             if (resourceUrl == null) {
                 // cannot serve requested file
-                logger.severe("Requested resource ["
+                logger.info("Requested resource ["
                         + filename
                         + "] not found from filesystem or through class loader."
                         + " Add widgetset and/or theme JAR to your classpath or add files to WebContent/VAADIN folder.");
@@ -1661,7 +1661,7 @@ public abstract class AbstractApplicationServlet extends HttpServlet implements
         try {
             return getApplicationClass().getSimpleName();
         } catch (ClassNotFoundException e) {
-            logger.log(Level.FINER, "getApplicationCSSClassName failed", e);
+            logger.log(Level.WARNING, "getApplicationCSSClassName failed", e);
             return "unknown";
         }
     }
index 7c4a410a81aa4db6641b53a2072aa2b2713cde10..78cc4e4bae2dfbaeb697e73b0b4eb8ce167bc4a8 100644 (file)
@@ -765,7 +765,7 @@ public abstract class AbstractCommunicationManager implements
                 if (window == null) {
                     // This should not happen, no windows exists but
                     // application is still open.
-                    logger.severe("Warning, could not get window for application with request ID "
+                    logger.warning("Could not get window for application with request ID "
                             + request.getRequestID());
                     return;
                 }
@@ -1138,7 +1138,7 @@ public abstract class AbstractCommunicationManager implements
                         r.close();
                     } catch (final java.io.IOException e) {
                         // FIXME: Handle exception
-                        logger.log(Level.SEVERE, "Resource transfer failed:  "
+                        logger.log(Level.INFO, "Resource transfer failed:  "
                                 + request.getRequestID() + ".", e);
                     }
                     outWriter.print("\""
index 410fba0ea064d13e6796a263c6ccc542a5d777b4..5e660ab82cc6fdc0b67e8292a7040ca838337ded 100644 (file)
@@ -177,7 +177,7 @@ public class ApplicationRunnerServlet extends AbstractApplicationServlet {
                     } catch (Exception e2) {
                         // TODO: handle exception
                         logger.log(
-                                Level.FINER,
+                                Level.FINE,
                                 "Failed to find application class in the default package.",
                                 e2);
                     }
index 47067e5b8dc7b084c8b994da8fa88a7c16049c48..5f8e59e0567210260d04735ccfb87f004df43bd3 100644 (file)
@@ -94,7 +94,7 @@ import com.vaadin.service.ApplicationContext;
  */
 public class GAEApplicationServlet extends ApplicationServlet {
 
-    private static final Logger log = Logger
+    private static final Logger logger = Logger
             .getLogger(GAEApplicationServlet.class.getName());
 
     // memcache mutex is MUTEX_BASE + sessio id
@@ -209,7 +209,7 @@ public class GAEApplicationServlet extends ApplicationServlet {
                 try {
                     Thread.sleep(RETRY_AFTER_MILLISECONDS);
                 } catch (InterruptedException e) {
-                    log.info("Thread.sleep() interrupted while waiting for lock. Trying again. "
+                    logger.finer("Thread.sleep() interrupted while waiting for lock. Trying again. "
                             + e);
                 }
             }
@@ -252,16 +252,16 @@ public class GAEApplicationServlet extends ApplicationServlet {
             ds.put(entity);
 
         } catch (DeadlineExceededException e) {
-            log.severe("DeadlineExceeded for " + session.getId());
+            logger.warning("DeadlineExceeded for " + session.getId());
             sendDeadlineExceededNotification(request, response);
         } catch (NotSerializableException e) {
-            log.log(Level.SEVERE, "Not serializable!", e);
+            logger.log(Level.SEVERE, "Not serializable!", e);
 
             // TODO this notification is usually not shown - should we redirect
             // in some other way - can we?
             sendNotSerializableNotification(request, response);
         } catch (Exception e) {
-            log.log(Level.SEVERE,
+            logger.log(Level.WARNING,
                     "An exception occurred while servicing request.", e);
 
             sendCriticalErrorNotification(request, response);
@@ -308,12 +308,12 @@ public class GAEApplicationServlet extends ApplicationServlet {
                 session.setAttribute(WebApplicationContext.class.getName(),
                         applicationContext);
             } catch (IOException e) {
-                log.log(Level.WARNING,
+                logger.log(Level.WARNING,
                         "Could not de-serialize ApplicationContext for "
                                 + session.getId()
                                 + " A new one will be created. ", e);
             } catch (ClassNotFoundException e) {
-                log.log(Level.WARNING,
+                logger.log(Level.WARNING,
                         "Could not de-serialize ApplicationContext for "
                                 + session.getId()
                                 + " A new one will be created. ", e);
@@ -368,7 +368,7 @@ public class GAEApplicationServlet extends ApplicationServlet {
                 List<Entity> entities = pq.asList(Builder
                         .withLimit(CLEANUP_LIMIT));
                 if (entities != null) {
-                    log.info("Vaadin cleanup deleting " + entities.size()
+                    logger.info("Vaadin cleanup deleting " + entities.size()
                             + " expired Vaadin sessions.");
                     List<Key> keys = new ArrayList<Key>();
                     for (Entity e : entities) {
@@ -387,7 +387,7 @@ public class GAEApplicationServlet extends ApplicationServlet {
                 List<Entity> entities = pq.asList(Builder
                         .withLimit(CLEANUP_LIMIT));
                 if (entities != null) {
-                    log.info("Vaadin cleanup deleting " + entities.size()
+                    logger.info("Vaadin cleanup deleting " + entities.size()
                             + " expired appengine sessions.");
                     List<Key> keys = new ArrayList<Key>();
                     for (Entity e : entities) {
@@ -397,7 +397,7 @@ public class GAEApplicationServlet extends ApplicationServlet {
                 }
             }
         } catch (Exception e) {
-            log.log(Level.WARNING, "Exception while cleaning.", e);
+            logger.log(Level.WARNING, "Exception while cleaning.", e);
         }
     }
 }
index 01f2f0b589cbe334ed811f098e49a6b1853f81c1..1f5c04ebe5a25cabcbe6a20d971f690043c7506d 100644 (file)
@@ -78,7 +78,7 @@ public class PortletApplicationContext2 extends AbstractWebApplicationContext {
             } catch (final Exception e) {
                 // FIXME: Handle exception
                 logger.log(
-                        Level.FINE,
+                        Level.INFO,
                         "Cannot access base directory, possible security issue "
                                 + "with Application Server or Servlet Container",
                         e);
index abdf7bd5516ad8c30a6b7d2a8c3e0efbd296750c..f69cc468bfc398b68b15f4ff25f948f0e2878449 100644 (file)
@@ -54,8 +54,8 @@ import com.vaadin.ui.ClientWidget;
  */
 public class ClassPathExplorer {
 
-    private static Logger logger = Logger
-            .getLogger("com.vaadin.terminal.gwt.widgetsetutils");
+    private static Logger logger = Logger.getLogger(ClassPathExplorer.class
+            .getName());
 
     /**
      * File filter that only accepts directories.
@@ -197,7 +197,8 @@ public class ClassPathExplorer {
                             // should never happen as based on an existing URL,
                             // only changing end of file name/path part
                             logger.log(Level.SEVERE,
-                                    "This should never happen!", e);
+                                    "Error locating the widgetset " + classname,
+                                    e);
                         }
                     }
                 }