echelon/i2p.i2p
1
0
Fork 0
forked from I2P_Developers/i2p.i2p
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

* NewsFetcher: Only treat correct status codes as success

Browse Source 
The last-modified date was getting updated even when
   the server returned e.g. 503, preventing routers from
   getting the latest file on a subsequent fetch.
This commit is contained in:
zzz
2014-06-10 19:52:55 +00:00
parent 4dd8a6421a
commit 1bf8fd92e4
1 changed files with 5 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
apps/routerconsole/java/src/net/i2p/router/update/NewsFetcher.java
View File

@ -89,9 +89,12 @@ class NewsFetcher extends UpdateRunner {
get.addStatusListener(this);
long start = _context.clock().now();
if (get.fetch()) {
_context.router().saveConfig(NewsHelper.PROP_LAST_CHECKED,
int status = get.getStatusCode();
if (status == 200 || status == 304) {
_context.router().saveConfig(NewsHelper.PROP_LAST_CHECKED,
Long.toString(start));
return;
return;
}
}
} catch (Throwable t) {
_log.error("Error fetching the news", t);