Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add FTP as a backup target and schedule nightly backup job #2806

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

Fluxxo
Copy link

@Fluxxo Fluxxo commented Dec 1, 2023

I made some changes to the way backups work in openwb1.9:

  1. The "Backup erstellen" menu is gone, instead leading to a configuration page
  2. Users can now decide where they want the backup to be stored: (a) Locally under /var/www/html/openWB/backup or remotely on an FTP server. The page performs a check whether the FTP credentials are correct and the remote folder is accessible.
  3. Added fields to configure FTP connection
  4. Added flag whether a nightly backup should be performed
  5. Added a flag whether to apply filename backup.tar.gz or openWB_backup_$jahr-$monat-$tag-$hourofday-$minute.tar.gz.
  6. Added a button to manually perform a backup into the desired destination.
  7. Adjusted the nightly cronjob to perform the backup if configured.
  8. Backup can still be triggered manually via HTTP, but the URL changed from backup.php to savebackup.php
  9. savebackup.php will spit out curl error codes in case it cannot store the backup.

@benderl
Copy link
Collaborator

benderl commented Dec 11, 2023

This will make existing external solutions for scheduled backups useless. These scripts rely on the behaviour of the current backup.php.

@Fluxxo
Copy link
Author

Fluxxo commented Dec 11, 2023

Would it change it I renamed the savebackup.php back to backup.php and ported the backup.php feature into a renamed settings page?
URL calls would remain the same and the default behaviour as well, while still extending current functionality.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants