Skamasle / sk-import-cpanel-backup-to-vestacp

Import cPanel backup in vestacp
35 stars 31 forks source link

Two small updates #4

Closed AlperSakarya closed 6 years ago

AlperSakarya commented 6 years ago

Fixed two issues I saw during migration.

1 - Added absolute path "/usr/local/vesta/bin/" for all v- vesta commands, since this path does not get added to user's path during Vesta installation. Also, some sections had the absolute path and some sections did not have it, so to make it consistent and make it more error-free for boxes like mine with no path I added the full path.

2- I added folder content check for "${sk_importer_in}/ssl/". If source website did not have SSL cert this folder is empty on default and erroring out with below message. mv: cannot stat ‘/root/sk_tmp/backup-1.17.2018_19-32-39_test/ssl/*’: No such file or directory Lines 275 to 279 does check the contents first and if it's empty it skips to avoid errors.

Skamasle commented 6 years ago

Thanks AlperSakarya

And yes the path fail in some installs, mostly in ubuntu or debian, ssl restore is just a alpha I not test it well I think not works fine in most backups, with letsencrypt I think is not needed if not works fine.