Add checks & realpath() usage

- New util class "FileSystem"
- Add check in admin summary too
This commit is contained in:
Philipp Holzer 2019-10-22 22:47:37 +02:00
parent 0e84a843a4
commit 6b2c28e2d7
No known key found for this signature in database
GPG key ID: D8365C3D36B77D90
5 changed files with 129 additions and 60 deletions

View file

@ -6,6 +6,7 @@ use Friendica\Core\Config\Configuration;
use Friendica\Core\Logger; use Friendica\Core\Logger;
use Friendica\Database\Database; use Friendica\Database\Database;
use Friendica\Network\HTTPException\InternalServerErrorException; use Friendica\Network\HTTPException\InternalServerErrorException;
use Friendica\Util\FileSystem;
use Friendica\Util\Introspection; use Friendica\Util\Introspection;
use Friendica\Util\Logger\Monolog\DevelopHandler; use Friendica\Util\Logger\Monolog\DevelopHandler;
use Friendica\Util\Logger\Monolog\IntrospectionProcessor; use Friendica\Util\Logger\Monolog\IntrospectionProcessor;
@ -51,10 +52,11 @@ class LoggerFactory
* @param Database $database The Friendica Database instance * @param Database $database The Friendica Database instance
* @param Configuration $config The config * @param Configuration $config The config
* @param Profiler $profiler The profiler of the app * @param Profiler $profiler The profiler of the app
* @param FileSystem $fileSystem FileSystem utils
* *
* @return LoggerInterface The PSR-3 compliant logger instance * @return LoggerInterface The PSR-3 compliant logger instance
*/ */
public function create( Database $database, Configuration $config, Profiler $profiler) public function create(Database $database, Configuration $config, Profiler $profiler, FileSystem $fileSystem)
{ {
if (empty($config->get('system', 'debugging', false))) { if (empty($config->get('system', 'debugging', false))) {
$logger = new VoidLogger(); $logger = new VoidLogger();
@ -105,7 +107,7 @@ class LoggerFactory
// just add a stream in case it's either writable or not file // just add a stream in case it's either writable or not file
if (!is_file($stream) || is_writable($stream)) { if (!is_file($stream) || is_writable($stream)) {
try { try {
$logger = new StreamLogger($this->channel, $stream, $introspection, $loglevel); $logger = new StreamLogger($this->channel, $stream, $introspection, $fileSystem, $loglevel);
} catch (\Throwable $t) { } catch (\Throwable $t) {
// No logger ... // No logger ...
$logger = new VoidLogger(); $logger = new VoidLogger();
@ -137,13 +139,14 @@ class LoggerFactory
* *
* @param Configuration $config The config * @param Configuration $config The config
* @param Profiler $profiler The profiler of the app * @param Profiler $profiler The profiler of the app
* @param FileSystem $fileSystem FileSystem utils
* *
* @return LoggerInterface The PSR-3 compliant logger instance * @return LoggerInterface The PSR-3 compliant logger instance
* *
* @throws InternalServerErrorException * @throws InternalServerErrorException
* @throws \Exception * @throws \Exception
*/ */
public static function createDev(Configuration $config, Profiler $profiler) public static function createDev(Configuration $config, Profiler $profiler, FileSystem $fileSystem)
{ {
$debugging = $config->get('system', 'debugging'); $debugging = $config->get('system', 'debugging');
$stream = $config->get('system', 'dlogfile'); $stream = $config->get('system', 'dlogfile');
@ -183,7 +186,7 @@ class LoggerFactory
case 'stream': case 'stream':
default: default:
$logger = new StreamLogger(self::DEV_CHANNEL, $stream, $introspection, LogLevel::DEBUG); $logger = new StreamLogger(self::DEV_CHANNEL, $stream, $introspection, $fileSystem, LogLevel::DEBUG);
break; break;
} }
@ -225,7 +228,7 @@ class LoggerFactory
case "5": case "5":
// legacy ALL // legacy ALL
case "4": case "4":
return LogLevel::DEBUG; return LogLevel::DEBUG;
// default if nothing set // default if nothing set
default: default:
return $level; return $level;

View file

@ -14,6 +14,7 @@ use Friendica\Model\Register;
use Friendica\Module\BaseAdminModule; use Friendica\Module\BaseAdminModule;
use Friendica\Util\ConfigFileLoader; use Friendica\Util\ConfigFileLoader;
use Friendica\Util\DateTimeFormat; use Friendica\Util\DateTimeFormat;
use Friendica\Util\FileSystem;
use Friendica\Util\Network; use Friendica\Util\Network;
class Summary extends BaseAdminModule class Summary extends BaseAdminModule
@ -76,11 +77,21 @@ class Summary extends BaseAdminModule
// Check logfile permission // Check logfile permission
if (Config::get('system', 'debugging')) { if (Config::get('system', 'debugging')) {
$stream = Config::get('system', 'logfile'); $file = Config::get('system', 'logfile');
if (is_file($stream) && /** @var FileSystem $fileSystem */
!is_writeable($stream)) { $fileSystem = self::getClass(FileSystem::class);
$warningtext[] = L10n::t('The logfile \'%s\' is not writable. No logging possible', $stream);
try {
$stream = $fileSystem->createStream($file);
if (is_file($stream) &&
!is_writeable($stream)) {
$warningtext[] = L10n::t('The logfile \'%s\' is not writable. No logging possible', $stream);
}
} catch (\Throwable $exception) {
$warningtext[] = L10n::t('The logfile \'%s\' is not usable. No logging possible (error: \'%s\')', $file, $exception->getMessage());
} }
$stream = Config::get('system', 'dlogfile'); $stream = Config::get('system', 'dlogfile');

64
src/Util/FileSystem.php Normal file
View file

@ -0,0 +1,64 @@
<?php
namespace Friendica\Util;
final class FileSystem
{
/**
* @var string a error message
*/
private $errorMessage;
public function createDir(string $file)
{
$dirname = null;
$pos = strpos($file, '://');
if (!$pos) {
$dirname = realpath(dirname($file));
}
if (substr($file, 0, 7) === 'file://') {
$dirname = realpath(dirname(substr($file, 7)));
}
if (isset($dirname) && !is_dir($dirname)) {
set_error_handler([$this, 'customErrorHandler']);
$status = mkdir($dirname, 0777, true);
restore_error_handler();
if (!$status && !is_dir($dirname)) {
throw new \UnexpectedValueException(sprintf('Directory "%s" cannot get created: ' . $this->errorMessage, $dirname));
}
return $dirname;
} elseif (isset($dirname) && is_dir($dirname)) {
return $dirname;
} else {
return '';
}
}
public function createStream(string $url)
{
$directory = $this->createDir($url);
set_error_handler([$this, 'customErrorHandler']);
if (!empty($directory)) {
$url = $directory . DIRECTORY_SEPARATOR . pathinfo($url, PATHINFO_BASENAME);
}
$stream = fopen($url, 'ab');
restore_error_handler();
if (!is_resource($stream)) {
throw new \UnexpectedValueException(sprintf('The stream or file "%s" could not be opened: ' . $this->errorMessage, $url));
}
return $stream;
}
private function customErrorHandler($code, $msg)
{
$this->errorMessage = preg_replace('{^(fopen|mkdir)\(.*?\): }', '', $msg);
}
}

View file

@ -3,6 +3,7 @@
namespace Friendica\Util\Logger; namespace Friendica\Util\Logger;
use Friendica\Util\DateTimeFormat; use Friendica\Util\DateTimeFormat;
use Friendica\Util\FileSystem;
use Friendica\Util\Introspection; use Friendica\Util\Introspection;
use Psr\Log\LogLevel; use Psr\Log\LogLevel;
@ -35,11 +36,11 @@ class StreamLogger extends AbstractLogger
*/ */
private $pid; private $pid;
/** /**
* An error message * @var FileSystem
* @var string
*/ */
private $errorMessage; private $fileSystem;
/** /**
* Translates LogLevel log levels to integer values * Translates LogLevel log levels to integer values
@ -63,8 +64,10 @@ class StreamLogger extends AbstractLogger
* *
* @throws \Exception * @throws \Exception
*/ */
public function __construct($channel, $stream, Introspection $introspection, $level = LogLevel::DEBUG) public function __construct($channel, $stream, Introspection $introspection, FileSystem $fileSystem, $level = LogLevel::DEBUG)
{ {
$this->fileSystem = $fileSystem;
parent::__construct($channel, $introspection); parent::__construct($channel, $introspection);
if (is_resource($stream)) { if (is_resource($stream)) {
@ -157,43 +160,6 @@ class StreamLogger extends AbstractLogger
throw new \LogicException('Missing stream URL.'); throw new \LogicException('Missing stream URL.');
} }
$this->createDir(); $this->stream = $this->fileSystem->createStream($this->url);
set_error_handler([$this, 'customErrorHandler']);
$this->stream = fopen($this->url, 'ab');
restore_error_handler();
if (!is_resource($this->stream)) {
$this->stream = null;
throw new \UnexpectedValueException(sprintf('The stream or file "%s" could not be opened: ' . $this->errorMessage, $this->url));
}
}
private function createDir()
{
$dirname = null;
$pos = strpos($this->url, '://');
if (!$pos) {
$dirname = dirname($this->url);
}
if (substr($this->url, 0, 7) === 'file://') {
$dirname = dirname(substr($this->url, 7));
}
if (isset($dirname) && !is_dir($dirname)) {
set_error_handler([$this, 'customErrorHandler']);
$status = mkdir($dirname, 0777, true);
restore_error_handler();
if (!$status && !is_dir($dirname)) {
throw new \UnexpectedValueException(sprintf('Directory "%s" cannot get created: ' . $this->errorMessage, $dirname));
}
}
}
private function customErrorHandler($code, $msg)
{
$this->errorMessage = preg_replace('{^(fopen|mkdir)\(.*?\): }', '', $msg);
} }
} }

View file

@ -2,6 +2,7 @@
namespace Friendica\Test\src\Util\Logger; namespace Friendica\Test\src\Util\Logger;
use Friendica\Util\FileSystem;
use Friendica\Test\Util\VFSTrait; use Friendica\Test\Util\VFSTrait;
use Friendica\Util\Logger\StreamLogger; use Friendica\Util\Logger\StreamLogger;
use org\bovigo\vfs\vfsStream; use org\bovigo\vfs\vfsStream;
@ -22,11 +23,18 @@ class StreamLoggerTest extends AbstractLoggerTest
*/ */
private $logfile; private $logfile;
/**
* @var Filesystem
*/
private $fileSystem;
protected function setUp() protected function setUp()
{ {
parent::setUp(); parent::setUp();
$this->setUpVfsDir(); $this->setUpVfsDir();
$this->fileSystem = new Filesystem();
} }
/** /**
@ -37,7 +45,7 @@ class StreamLoggerTest extends AbstractLoggerTest
$this->logfile = vfsStream::newFile('friendica.log') $this->logfile = vfsStream::newFile('friendica.log')
->at($this->root); ->at($this->root);
$this->logger = new StreamLogger('test', $this->logfile->url(), $this->introspection, $level); $this->logger = new StreamLogger('test', $this->logfile->url(), $this->introspection, $this->fileSystem, $level);
return $this->logger; return $this->logger;
} }
@ -60,7 +68,7 @@ class StreamLoggerTest extends AbstractLoggerTest
$filehandler = fopen($logfile->url(), 'ab'); $filehandler = fopen($logfile->url(), 'ab');
$logger = new StreamLogger('test', $filehandler, $this->introspection); $logger = new StreamLogger('test', $filehandler, $this->introspection, $this->fileSystem);
$logger->emergency('working'); $logger->emergency('working');
$text = $logfile->getContent(); $text = $logfile->getContent();
@ -76,7 +84,7 @@ class StreamLoggerTest extends AbstractLoggerTest
$logfile = vfsStream::newFile('friendica.log') $logfile = vfsStream::newFile('friendica.log')
->at($this->root); ->at($this->root);
$logger = new StreamLogger('test', $logfile->url(), $this->introspection); $logger = new StreamLogger('test', $logfile->url(), $this->introspection, $this->fileSystem);
$logger->emergency('working'); $logger->emergency('working');
$logger->close(); $logger->close();
// close doesn't affect // close doesn't affect
@ -94,7 +102,7 @@ class StreamLoggerTest extends AbstractLoggerTest
*/ */
public function testNoUrl() public function testNoUrl()
{ {
$logger = new StreamLogger('test', '', $this->introspection); $logger = new StreamLogger('test', '', $this->introspection, $this->fileSystem);
$logger->emergency('not working'); $logger->emergency('not working');
} }
@ -109,7 +117,7 @@ class StreamLoggerTest extends AbstractLoggerTest
$logfile = vfsStream::newFile('friendica.log') $logfile = vfsStream::newFile('friendica.log')
->at($this->root)->chmod(0); ->at($this->root)->chmod(0);
$logger = new StreamLogger('test', $logfile->url(), $this->introspection); $logger = new StreamLogger('test', $logfile->url(), $this->introspection, $this->fileSystem);
$logger->emergency('not working'); $logger->emergency('not working');
} }
@ -123,7 +131,7 @@ class StreamLoggerTest extends AbstractLoggerTest
{ {
$this->markTestIncomplete('We need a platform independent way to set directory to readonly'); $this->markTestIncomplete('We need a platform independent way to set directory to readonly');
$logger = new StreamLogger('test', '/$%/wrong/directory/file.txt', $this->introspection); $logger = new StreamLogger('test', '/$%/wrong/directory/file.txt', $this->introspection, $this->fileSystem);
$logger->emergency('not working'); $logger->emergency('not working');
} }
@ -135,7 +143,7 @@ class StreamLoggerTest extends AbstractLoggerTest
*/ */
public function testWrongMinimumLevel() public function testWrongMinimumLevel()
{ {
$logger = new StreamLogger('test', 'file.text', $this->introspection, 'NOPE'); $logger = new StreamLogger('test', 'file.text', $this->introspection, $this->fileSystem, 'NOPE');
} }
/** /**
@ -148,7 +156,7 @@ class StreamLoggerTest extends AbstractLoggerTest
$logfile = vfsStream::newFile('friendica.log') $logfile = vfsStream::newFile('friendica.log')
->at($this->root); ->at($this->root);
$logger = new StreamLogger('test', $logfile->url(), $this->introspection); $logger = new StreamLogger('test', $logfile->url(), $this->introspection, $this->fileSystem);
$logger->log('NOPE', 'a test'); $logger->log('NOPE', 'a test');
} }
@ -160,6 +168,23 @@ class StreamLoggerTest extends AbstractLoggerTest
*/ */
public function testWrongFile() public function testWrongFile()
{ {
$logger = new StreamLogger('test', null, $this->introspection); $logger = new StreamLogger('test', null, $this->introspection, $this->fileSystem);
}
/**
* Test a relative path
*/
public function testRealPath()
{
$this->markTestSkipped('vfsStream isn\'t compatible with chdir, so not testable.');
$logfile = vfsStream::newFile('friendica.log')
->at($this->root);
chdir($this->root->getChild('logs')->url());
$logger = new StreamLogger('test', '../friendica.log' , $this->introspection, $this->fileSystem);
$logger->info('Test');
} }
} }