Skip to content

Commit 9a7f9b1

Browse files
matiwinnetouMateusz Czeladka
andauthored
fix: avoid NPE when transaction is not found (#313)
Co-authored-by: Mateusz Czeladka <[email protected]>
1 parent dafd82c commit 9a7f9b1

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

api/src/main/java/org/cardanofoundation/rosetta/api/search/controller/SearchApiImpl.java

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -35,12 +35,11 @@ public class SearchApiImpl implements SearchApi {
3535
@Override
3636
public ResponseEntity<SearchTransactionsResponse> searchTransactions(
3737
SearchTransactionsRequest searchTransactionsRequest) {
38-
if(offlineMode) {
38+
if (offlineMode) {
3939
throw ExceptionFactory.notSupportedInOfflineMode();
4040
}
4141
networkService.verifyNetworkRequest(searchTransactionsRequest.getNetworkIdentifier());
4242

43-
4443
Long pageSize = Optional.ofNullable(searchTransactionsRequest.getLimit()).orElse(PAGE_SIZE);
4544
pageSize = pageSize > PAGE_SIZE ? PAGE_SIZE : pageSize;
4645
Long offset = Optional.ofNullable(searchTransactionsRequest.getOffset()).orElse(0L);
@@ -51,6 +50,6 @@ public ResponseEntity<SearchTransactionsResponse> searchTransactions(
5150

5251
return ResponseEntity.ok(searchMapper.mapToSearchTransactionsResponse(
5352
blockTransactions, nextOffset));
54-
5553
}
54+
5655
}

api/src/main/java/org/cardanofoundation/rosetta/common/exception/ApiException.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,8 @@ public class ApiException extends RuntimeException {
1212
private final Error error;
1313

1414
public ApiException(Error error) {
15-
super();
15+
super(error.getMessage() == null ? "?" : error.getMessage());
1616
this.error = error;
1717
}
18+
1819
}

0 commit comments

Comments
 (0)