From 916d252fccbd4b793e828764e55bda8f68c888ab Mon Sep 17 00:00:00 2001 From: Dan Field Date: Thu, 16 Jan 2025 10:13:02 +0000 Subject: [PATCH] include latest changes for rangeRequests in fcrepo-storage-ocfl --- pom.xml | 2 +- .../migration/OcflSessionFactoryFactoryBean.java | 4 +++- .../handlers/ocfl/OcflObjectSessionWrapper.java | 12 ++++++++++++ .../handlers/ocfl/PlainOcflObjectSession.java | 11 +++++++++++ .../org/fcrepo/migration/pidlist/HeadOnlyIT.java | 3 ++- 5 files changed, 29 insertions(+), 3 deletions(-) diff --git a/pom.xml b/pom.xml index 06c55f52..ebfbb0ae 100644 --- a/pom.xml +++ b/pom.xml @@ -18,7 +18,7 @@ UTF-8 org.fcrepo.migration.PicocliMigrator 5.2.24.RELEASE - 6.4.0-SNAPSHOT + 6.4.0 7.1.0 4.2.0 3.1.1 diff --git a/src/main/java/org/fcrepo/migration/OcflSessionFactoryFactoryBean.java b/src/main/java/org/fcrepo/migration/OcflSessionFactoryFactoryBean.java index 6628db66..b7c58ec7 100644 --- a/src/main/java/org/fcrepo/migration/OcflSessionFactoryFactoryBean.java +++ b/src/main/java/org/fcrepo/migration/OcflSessionFactoryFactoryBean.java @@ -22,6 +22,7 @@ import com.github.benmanes.caffeine.cache.Caffeine; import io.ocfl.api.OcflConfig; import io.ocfl.api.model.DigestAlgorithm; +import io.ocfl.api.DigestAlgorithmRegistry; import io.ocfl.core.OcflRepositoryBuilder; import io.ocfl.core.extension.storage.layout.config.HashedNTupleLayoutConfig; import io.ocfl.core.path.mapper.LogicalPathMappers; @@ -93,7 +94,8 @@ public OcflSessionFactoryFactoryBean(final Path ocflRoot, final String user, final String userUri, final boolean disableChecksumValidation) { - this(ocflRoot, stagingDir, migrationType, user, userUri, DigestAlgorithm.sha512, disableChecksumValidation); + this(ocflRoot, stagingDir, migrationType, user, userUri, + DigestAlgorithmRegistry.sha512, disableChecksumValidation); } @Override diff --git a/src/main/java/org/fcrepo/migration/handlers/ocfl/OcflObjectSessionWrapper.java b/src/main/java/org/fcrepo/migration/handlers/ocfl/OcflObjectSessionWrapper.java index 0beae3bb..594bd67f 100644 --- a/src/main/java/org/fcrepo/migration/handlers/ocfl/OcflObjectSessionWrapper.java +++ b/src/main/java/org/fcrepo/migration/handlers/ocfl/OcflObjectSessionWrapper.java @@ -111,6 +111,18 @@ public ResourceContent readContent(final String resourceId, final String version return readContentTimer.record(() -> inner.readContent(resourceId, versionNumber)); } + @Override + public ResourceContent readRange(final String resourceId, final String versionNumber, + final long startPosition, final long endPosition) { + return readContentTimer.record(() -> inner.readRange(resourceId, versionNumber, + startPosition, endPosition)); + } + + @Override + public ResourceContent readRange(final String resourceId, final long startPosition, final long endPosition) { + return readContentTimer.record(() -> inner.readRange(resourceId, null, startPosition, endPosition)); + } + @Override public List listVersions(final String resourceId) { return listVersionsTimer.record(() -> inner.listVersions(resourceId)); diff --git a/src/main/java/org/fcrepo/migration/handlers/ocfl/PlainOcflObjectSession.java b/src/main/java/org/fcrepo/migration/handlers/ocfl/PlainOcflObjectSession.java index 32859116..7ece0df6 100644 --- a/src/main/java/org/fcrepo/migration/handlers/ocfl/PlainOcflObjectSession.java +++ b/src/main/java/org/fcrepo/migration/handlers/ocfl/PlainOcflObjectSession.java @@ -253,6 +253,17 @@ public ResourceContent readContent(final String resourceId, final String version throw new UnsupportedOperationException("Not implemented"); } + @Override + public ResourceContent readRange(final String resourceId, final String versionNumber, + final long startPosition, final long endPosition) { + throw new UnsupportedOperationException("Not implemented"); + } + + @Override + public ResourceContent readRange(final String resourceId, final long startPosition, final long endPosition) { + throw new UnsupportedOperationException("Not implemented"); + } + @Override public List listVersions(final String resourceId) { throw new UnsupportedOperationException("Not implemented"); diff --git a/src/test/java/org/fcrepo/migration/pidlist/HeadOnlyIT.java b/src/test/java/org/fcrepo/migration/pidlist/HeadOnlyIT.java index 9c17e8c6..1a83751b 100644 --- a/src/test/java/org/fcrepo/migration/pidlist/HeadOnlyIT.java +++ b/src/test/java/org/fcrepo/migration/pidlist/HeadOnlyIT.java @@ -10,6 +10,7 @@ import io.ocfl.api.OcflRepository; import io.ocfl.api.model.DigestAlgorithm; +import io.ocfl.api.DigestAlgorithmRegistry; import io.ocfl.api.model.FileDetails; import io.ocfl.api.model.ObjectDetails; import io.ocfl.core.OcflRepositoryBuilder; @@ -43,7 +44,7 @@ public class HeadOnlyIT { private final boolean disableChecksum = false; private final boolean disableDc = false; - private final DigestAlgorithm digestAlgorithm = DigestAlgorithm.sha512; + private final DigestAlgorithm digestAlgorithm = DigestAlgorithmRegistry.sha512; private final MigrationType migrationType = MigrationType.FEDORA_OCFL; private LegacyFSIDResolver idResolver;