Fix backup disk driver configuration for S3 support#18710
Fix backup disk driver configuration for S3 support#18710snipe merged 1 commit intogrokability:masterfrom
Conversation
- Fix the backup disk in config/filesystems.php to use a dedicated BACKUP_FILESYSTEM_DRIVER env var instead of PRIVATE_FILESYSTEM_DISK
- Add AWS credential fields to the backup disk config so S3 backups work
- Use BACKUP_FILESYSTEM_ROOT with safe default (storage_path('app')) for local driver
- Document BACKUP_FILESYSTEM_DRIVER and BACKUP_FILESYSTEM_ROOT in .env.example
Fixes grokability#14057
|
Hi there - thanks for this! Can you please re-target this PR to point to the You don't usually need to close and re-open. After you create a pull request, you can modify the base branch so that the changes in the pull request are compared against a different branch. By changing the base branch of your original pull request rather than opening a new one with the correct base branch, you’ll be able to keep valuable work and discussion. If you change that branch and you see bunch of someone else's commits in the "files changed" tab of this PR, you might need to rebase. (Leaving those changes in can make it kinda confusing to see this PR actually changes.) To RebaseAssuming you started with this repo as the origin, and added yourself as a named remote:
otherwise Thanks! |
1 similar comment
|
Hi there - thanks for this! Can you please re-target this PR to point to the You don't usually need to close and re-open. After you create a pull request, you can modify the base branch so that the changes in the pull request are compared against a different branch. By changing the base branch of your original pull request rather than opening a new one with the correct base branch, you’ll be able to keep valuable work and discussion. If you change that branch and you see bunch of someone else's commits in the "files changed" tab of this PR, you might need to rebase. (Leaving those changes in can make it kinda confusing to see this PR actually changes.) To RebaseAssuming you started with this repo as the origin, and added yourself as a named remote:
otherwise Thanks! |
|
Gentle ping :) |
Up to standards ✅🟢 Issues
|
| Metric | Results |
|---|---|
| Complexity | 0 |
TIP This summary will be updated as you push new changes. Give us feedback

Fixes #14057
Summary
backupdisk inconfig/filesystems.phpto use a dedicatedBACKUP_FILESYSTEM_DRIVERenv var instead ofPRIVATE_FILESYSTEM_DISKBACKUP_FILESYSTEM_ROOTwith safe default (storage_path('app')) for backward compatibilityBACKUP_FILESYSTEM_DRIVERandBACKUP_FILESYSTEM_ROOTin.env.exampleProblem
The current
backupdisk config usesPRIVATE_FILESYSTEM_DISKas its driver value:'backup' => [
'driver' => env('PRIVATE_FILESYSTEM_DISK', 'local'),
'root' => storage_path('app'),
],
This can lead to "s3_private" as the driver which is not valid (should be "s3")