nextcloud / nextcloud/user_migration
Bkg job don't import from temp storage
Nobody has claimed this yet.
- Dominant language
- JavaScript
- Stars
- 30
- Forks
- 10
- Avg merge
- 6h 5m
- Merged PRs (30d)
- 12
Description
The current background job import is using temporary storage when dealing with s3 primary storage or encryption.
This is because the ZipArchive can only open local files, so we need to download the file if it's not directly readable like with Local storage (using $file->getLocalFile().
To solve this, we might need to research using a different library which supports streaming the zip file somewhere for extraction.
@come-nc as discussed, for later
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start at the background job import path and inspect how ZipArchive handles files from S3 primary storage, encryption, and Local storage through getLocalFile(). Compare approaches for making the archive available locally or using a library that supports streaming extraction; done means imports work for the affected non-local storage cases.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript
- Domain
- backend, cloud
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 38/100