feat: Add storage provider for remote files#674
Open
mesemus wants to merge 1 commit intoinveniosoftware:masterfrom
Open
feat: Add storage provider for remote files#674mesemus wants to merge 1 commit intoinveniosoftware:masterfrom
mesemus wants to merge 1 commit intoinveniosoftware:masterfrom
Conversation
* Implement storage factory for remote files to support even low-level operations such as checksums for remote files * fix: Corrects checksum computation for remote files * fix: Fixes 500 on files-archive endpoint when remote files are present. Ensures remote files are included in the ZIP archive returned by the files-archive endpoint
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
This PR introduces a storage provider for remote files and fixes related issues in the file handling workflow.
The implementation adds a storage factory capable of opening/accessing remote files on the record level. This makes remote files behave exactly the same as local files and fixes issues related to checksum calculation and archive generation when remote files are present - #672.
Note
This PR depends on inveniosoftware/invenio-files-rest#348
Checklist
Ticks in all boxes and 🟢 on all GitHub actions status checks are required to merge:
Frontend
Reminder
By using GitHub, you have already agreed to the GitHub’s Terms of Service including that: