do not write htaccess file if disk space is too low

[szaimen]

I copied the logic from

server/lib/private/Config.php

Lines 288 to 295 in 1b18b97

	// Never write file back if disk space should be too low
	if (function_exists('disk_free_space')) {
	$df = disk_free_space($this->configDir);
	$size = strlen($content) + 10240;
	if ($df !== false && $df < (float)$size) {
	throw new \Exception($this->configDir . " does not have enough space for writing the config file! Not writing it back!");
	}
	}

[szaimen]

/backport to stable27

[kesselb]

Why?

[szaimen]

Why?

Because the file got truncated on @tobiasKaminsky's instance due to low disk space...

[Altahrim]

You can also check if all bytes have been written. Should be something like this:

Suggested change

	return (bool)@file_put_contents($setupHelper->pathToHtaccess(), $htaccessContent . $content . "\n");
	$fullContent = $htaccessContent . $content . "\n";
	return @file_put_contents($setupHelper->pathToHtaccess(), $$fullContent) === strlen($fullContent);

[szaimen]

This is not going to help here. The prpblem is that file_pu_content truncates the file if there is not enough space which should never happen as the file is very important to work...

[szaimen]

We had the same prpblem with config.php before and fixed in in the same way.

[Altahrim]

It's another security in case something happen during the copy.
The file will already be erased, but at least you'll have the info

[szaimen]

Ah I see.