Skip to content

Commit 93a6bcb

Browse files
authored
fix: gracefully handle case of file resources without extension version (#1706)
1 parent 502bab9 commit 93a6bcb

File tree

1 file changed

+9
-0
lines changed

1 file changed

+9
-0
lines changed

server/src/main/java/org/eclipse/openvsx/storage/log/DownloadCountProcessor.java

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -69,6 +69,15 @@ public void persistProcessedItem(String name, String storageType, LocalDateTime
6969
public Map<Long, Integer> processDownloadCounts(String storageType, Map<String, Integer> files) {
7070
return Observation.createNotStarted("DownloadCountProcessor#processDownloadCounts", observations).observe(() -> repositories.findDownloadsByStorageTypeAndName(storageType, files.keySet()).stream()
7171
.map(fileResource -> Map.entry(fileResource, files.get(fileResource.getName().toUpperCase())))
72+
.filter(fileResource -> {
73+
var ev = fileResource.getKey().getExtension();
74+
if (ev == null) {
75+
logger.warn("no extension version found for download {}, skipping", fileResource.getKey().getName());
76+
return false;
77+
} else {
78+
return true;
79+
}
80+
})
7281
.collect(Collectors.groupingBy(
7382
e -> e.getKey().getExtension().getExtension().getId(),
7483
Collectors.summingInt(Map.Entry::getValue)

0 commit comments

Comments
 (0)