Check for sufficient free space before extracting an app
Open
Nobody has claimed this yet.
0. Needs triage
enhancement
feature: apps management
- Dominant language
- PHP
- Stars
- 36.9k
- Forks
- 5.2k
- Avg merge
- 2d 3h
- Merged PRs (30d)
- 713
Description
Apps vary in size/space needs considerably. We should check before extracting (and downloading ideally):
Maybe do a rough check for 2x archive size?
Additional comments:
- Perhaps we add a new method to
TempManagerlikecheckAvailable(X)to provide a simple way of doing this that can also be used elsewhere.
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 lib/private/Installer.php around lines 254-259, then inspect TempManager for a reusable available-space check. Confirm the required space estimate and whether downloading is in scope. Done means extraction checks available space and handles insufficient space clearly, with the behavior covered by relevant tests if available.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- php
- Domain
- backend
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100