Skip to content

Commit 155cfc9

Browse files
committed
fixup for immutable Map returned by S3 SDK client for object metadata
1 parent 814a39b commit 155cfc9

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

src/main/java/io/cryostat/events/S3TemplateService.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,7 @@
2323
import java.nio.file.Path;
2424
import java.text.ParseException;
2525
import java.util.ArrayList;
26+
import java.util.HashMap;
2627
import java.util.List;
2728
import java.util.Map;
2829
import java.util.Objects;
@@ -249,7 +250,7 @@ private Template convertObject(S3Object object)
249250
break;
250251
case METADATA:
251252
var headReq = HeadObjectRequest.builder().bucket(bucket).key(object.key()).build();
252-
var meta = storage.headObject(headReq).metadata();
253+
var meta = new HashMap<>(storage.headObject(headReq).metadata());
253254
label = Objects.requireNonNull(meta.get(META_KEY_NAME));
254255
description = Objects.requireNonNull(meta.get(META_KEY_DESCRIPTION));
255256
provider = Objects.requireNonNull(meta.get(META_KEY_PROVIDER));

src/main/java/io/cryostat/recordings/RecordingHelper.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1039,7 +1039,7 @@ public Optional<Metadata> getArchivedRecordingMetadata(String storageKey) {
10391039
if (!resp.hasMetadata()) {
10401040
return Optional.empty();
10411041
}
1042-
return Optional.of(new Metadata(resp.metadata()));
1042+
return Optional.of(new Metadata(new HashMap<>(resp.metadata())));
10431043
case BUCKET:
10441044
return metadataService.get().read(storageKey);
10451045
default:

0 commit comments

Comments
 (0)