opendev/gerrit
Code Issues Proposed changes

ChangeApiImpl: Don't catch bare Exception when getting past assignees

Browse Source 
Change-Id: Iaf506c9a813a452eb7ad0639d4e8cdf225fb8284
This commit is contained in:
David Pursehouse
2017-03-29 21:20:54 +09:00
parent 483baef151
commit 811b72685b
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
gerrit-server/src/main/java/com/google/gerrit/server/api/changes/ChangeApiImpl.java
View File

@@ -491,7 +491,7 @@ class ChangeApiImpl implements ChangeApi {
public List<AccountInfo> getPastAssignees() throws RestApiException { public List<AccountInfo> getPastAssignees() throws RestApiException {
try { try {
return getPastAssignees.apply(change).value(); return getPastAssignees.apply(change).value();
} catch (Exception e) { } catch (OrmException e) {
throw new RestApiException("Cannot get past assignees", e); throw new RestApiException("Cannot get past assignees", e);
} }
} }