RenJuan/TASKPM
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Improve logging of possible exceptions checking version information

Browse source 
Avoid to print the whole stacktrace and simply show a message with the basic
exception information.

FEA: ItEr76S10NewVersionsNotification
This commit is contained in:
Manuel Rego Casasnovas 2012-01-18 07:57:44 +01:00
parent 7bfe203d07
commit f2cfd9f048
1 changed files with 7 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
libreplan-business/src/main/java/org/libreplan/business/common/VersionInformation.java
View file

@ -75,9 +75,13 @@ public class VersionInformation {
if (projectVersion != null && lastVersion != null) {
newVersionCached = !projectVersion.equals(lastVersion);
}
} catch (MalformedURLException e) {
LOG.warn("Problems generating URL to check LibrePlan version. MalformedURLException: "
+ e.getMessage());
} catch (IOException e) {
LOG.warn("Problems reading LibrePlan version from "
+ LIBREPLAN_VERSION_URL, e);
LOG.info("Could not check LibrePlan version information from "
+ LIBREPLAN_VERSION_URL + ". IOException: "
+ e.getMessage());
}
}
@ -86,7 +90,7 @@ public class VersionInformation {
String url = LIBREPLAN_VERSION_URL;
if (allowToGatherUsageStatsEnabled) {
url += "?" + LIBREPLAN_USAGE_STATS_PARAM + "=1";
url += "&version=" + projectVersion;
url += "&" + LIBREPLAN_VERSION_PARAM + "=" + projectVersion;
}
return new URL(url);
}