Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.It'd be useful to check the permissions of the backup destination(s) via hook_requirements, just to make sure everything is ok.
| Comment | File | Size | Author |
|---|---|---|---|
| #8 | backup_migrate-n2920311-8.interdiff.txt | 1.47 KB | DamienMcKenna |
| #8 | backup_migrate-n2920311-8.patch | 5.82 KB | DamienMcKenna |
| |||
Comments
Comment #2
DamienMcKennaHow does this look?
Comment #3
DamienMcKennaThis should check for the public path and manual paths too, and adds some extra inline comments.
Comment #4
gregglesComment #5
David_Rothstein CreditAttribution: David_Rothstein as a volunteer commentedI wonder if the Drupal 8 version of the module needs something like this as well.
It was pointed out to me that the README in Drupal 8 doesn't even mention the potential issue with web accessibility (whereas the Drupal 7 README does, under the headline "VERY IMPORTANT SECURITY NOTE"). I am not sure if that's an oversight, or if the module has changed in Drupal 8 such that it has other protections against this.
Comment #6
DamienMcKennaComment #7
DamienMcKennaAlso check that the individual files in the directory are not accessible.
Comment #8
DamienMcKennaRemoved the dpm() calls. Doh.
Comment #11
DamienMcKennaCommitted. We can do further work in other issues.