Make subscription failures visible.

--HG--
extra : rebase_source : 9e7232b821665d9a6342c928d0b17ac3bdd9e6e7
This commit is contained in:
Risto Kankkunen
2013-05-20 22:14:56 +03:00
parent 9fc5cee6fe
commit 8f9e8fd2ea
2 changed files with 12 additions and 3 deletions

View File

@@ -224,9 +224,15 @@ public class FeedREST extends AbstractResourceREST {
FeedCategory category = CategoryREST.ALL.equals(req.getCategoryId()) ? null
: feedCategoryDAO.findById(Long.valueOf(req.getCategoryId()));
FeedInfo info = (FeedInfo) fetchFeed(url).getEntity();
feedSubscriptionService.subscribe(getUser(), info.getUrl(),
try {
feedSubscriptionService.subscribe(getUser(), info.getUrl(),
req.getTitle(), category);
} catch (Exception e) {
log.info("Failed to subscribe to URL {}: {}", url, e.getMessage());
return Response.status(Status.SERVICE_UNAVAILABLE).entity(
"Failed to subscribe to URL " + url + ": " + e.getMessage()
).build();
}
return Response.ok(Status.OK).build();
}