platform: less noisy logging of update checker network errors appcode/140.2006 clion/140.2005 phpstorm/140.2004
authorRoman Shevchenko <roman.shevchenko@jetbrains.com>
Wed, 28 Jan 2015 11:51:51 +0000 (14:51 +0300)
committerRoman Shevchenko <roman.shevchenko@jetbrains.com>
Wed, 28 Jan 2015 11:52:31 +0000 (14:52 +0300)
platform/platform-impl/src/com/intellij/openapi/updateSettings/impl/UpdateChecker.java
platform/platform-resources-en/src/messages/IdeBundle.properties

index c26e223259d62263bb38bf3637eac5402171b301..6bb4d1116278610322f6720236ad336667d4795d 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright 2000-2014 JetBrains s.r.o.
+ * Copyright 2000-2015 JetBrains s.r.o.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -55,7 +55,6 @@ import org.jetbrains.annotations.Nullable;
 import javax.swing.event.HyperlinkEvent;
 import java.io.File;
 import java.io.IOException;
-import java.io.InterruptedIOException;
 import java.net.URISyntaxException;
 import java.net.URL;
 import java.util.*;
@@ -161,8 +160,9 @@ public final class UpdateChecker {
     }
     else if (result.getState() == UpdateStrategy.State.CONNECTION_ERROR) {
       Exception e = result.getError();
-      if (e != null) LOG.info(e);
-      showErrorMessage(manualCheck, IdeBundle.message(e instanceof InterruptedIOException ? "updates.timeout.error" : "updates.error.connection.failed"));
+      if (e != null) LOG.debug(e);
+      String cause = e != null ? e.getMessage() : "internal error";
+      showErrorMessage(manualCheck, IdeBundle.message("updates.error.connection.failed", cause));
       return;
     }
 
@@ -306,12 +306,12 @@ public final class UpdateChecker {
         }
       }
       catch (IOException e) {
+        LOG.debug(e);
         if (host != null) {
-          LOG.warn("failed to load plugin descriptions from " + host, e);
+          LOG.info("failed to load plugin descriptions from " + host + ": " + e.getMessage());
         }
         else {
-          LOG.warn(e);
-          showErrorMessage(manualCheck, e.getMessage());
+          showErrorMessage(manualCheck, IdeBundle.message("updates.error.connection.failed", e.getMessage()));
         }
       }
     }
@@ -363,6 +363,7 @@ public final class UpdateChecker {
   }
 
   private static void showErrorMessage(boolean showDialog, final String message) {
+    LOG.info(message);
     if (showDialog) {
       UIUtil.invokeLaterIfNeeded(new Runnable() {
         @Override
@@ -371,9 +372,6 @@ public final class UpdateChecker {
         }
       });
     }
-    else {
-      LOG.info(message);
-    }
   }
 
   @Contract("null -> false")
index ddce3a91c9bf0e8272a0677f56427b6df39d7f37..105cd16d548241dbd5ea100a7518e5607975bbbb 100644 (file)
@@ -899,7 +899,7 @@ updates.checking.progress=Checking for updates
 updates.checking.platform=Checking for platform update
 updates.checking.plugins=Checking for plugins update
 updates.error.connection.title=Connection Error
-updates.error.connection.failed=Connection failed. Please check your network connection and try again.
+updates.error.connection.failed=Connection failed ({0}). Please check network connection and try again.
 updates.error.plugin.description.failed=Failed to load plugin descriptions from: \"{0}\"
 update.notifications.title=Platform and Plugin Updates
 update.notifications.group=Platform and Plugin Updates
@@ -913,7 +913,6 @@ updates.no.updates.message=You already have the latest version of {0} installed.
 updates.plugins.ready.header=<html><b>Plugins from configured hosts are ready to update.</b><br> \
   Uncheck plugins you do not want to update.</html>
 updates.configure.label=To configure automatic update settings, see the <b><a href=\"updates\">Updates</a></b> dialog of your IDE {0}.
-updates.timeout.error=Connection timed out
 updates.incompatible.plugins.found={0,choice,1#Plugin|2#Plugins} incompatible with the new build found:{0,choice,1# |2#<br/>} {1}
 updates.download.and.install.button=&Download Patch and Shutdown
 updates.download.and.restart.button=Up&date and Restart