Issues unzipping demo-content when wordpress root is on NFS about unyson-backups-extension HOT 6 CLOSED

Hey @cu12,

Have you tried to swap out the implementation for get_tmp_dir()? Did it helped you?

from unyson-backups-extension.

hey @andreiglingeanu

Indeed, dumbly modifying the function like this, makes this working

        public function get_tmp_dir() {
                return get_temp_dir() . '/tmp';
        }

it's a good idea to use the canonical function anyway.

from unyson-backups-extension.

That's true, but I'm not entirely sure why that method was implemented that way in the first place. I'll have to look it up before swaping out the implementation.

cc @ViorelEremia

from unyson-backups-extension.

This function return path to folder fw-backup/tmp dir created in folder uploads this is not the same as default get_temp_dir() directly in tmp this is added and for another reasons it can be rewritten from config.php param dirs.destination, some plugins are badly programmed by deleting all of that folder it results that it will delete and your backup( plugins what works with downloaded files cache etc but not only plugins touch this folder).
@cu12 can you check this function what path returns wp_upload_dir I think here is the problem.

from unyson-backups-extension.

@ViorelEremia thanks for working on this, the function returns something like the following:

Array
(
    [path] => /var/www/wp-content/uploads/2017/11
    [url] => redacted
    [subdir] => /2017/11
    [basedir] => /var/www/wp-content/uploads
    [baseurl] => redacted
    [error] =>
)

Now the problem is not the path, but the filesystem behind it which is Gluster used via NFS. After the download finishes, it's not guaranteed that it's already been fsync'd to the filesystem, therefore an immediate read could end up in a "bad" zip file.

Making the download defaulting in /tmp would almost all the time guarantee that extraction would succeed right after the download as in 99% of the cases it's not a network filesystem.

from unyson-backups-extension.

I do not think I have to focus on this issue, and I do not know how to do this so as not to affect the others so if you find a solution create a pull request.

from unyson-backups-extension.