wbrawner/server
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Always log cron execution

Browse source 
There was a setting to disable the last execution of cron. There is no known
problem with this write access and it was also questioned when this feature
was build in https://github.com/owncloud/core/pull/7689#issuecomment-38264707

Recently there was also a bug report about a non-visible last cron execution
(#6088) - let's better remove this.

Signed-off-by: Morris Jobke <hey@morrisjobke.de>
This commit is contained in:
Morris Jobke 2017-08-16 20:55:38 +02:00
parent f0eb882c2e
commit 1c38a83927
5 changed files with 22 additions and 40 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
config/config.sample.php
View file

@ -738,13 +738,6 @@ $CONFIG = array(
*/ */
'log_query' => false, 'log_query' => false,
/**
* Log successful cron runs.
*
* Defaults to ``true``
*/
'cron_log' => true,
/** /**
* Enables log rotation and limits the total size of logfiles. The default is 0, * Enables log rotation and limits the total size of logfiles. The default is 0,
* or no rotation. Specify a size in bytes, for example 104857600 (100 megabytes * or no rotation. Specify a size in bytes, for example 104857600 (100 megabytes

2
cron.php
View file

@ -152,9 +152,7 @@ try {
} }
// Log the successful cron execution // Log the successful cron execution
if (\OC::$server->getConfig()->getSystemValue('cron_log', true)) {
\OC::$server->getConfig()->setAppValue('core', 'lastcron', time()); \OC::$server->getConfig()->setAppValue('core', 'lastcron', time());
}
exit(); exit();
} catch (Exception $ex) { } catch (Exception $ex) {

1
lib/private/Settings/Admin/Server.php
View file

@ -130,7 +130,6 @@ class Server implements ISettings {
// Background jobs // Background jobs
'backgroundjobs_mode' => $this->config->getAppValue('core', 'backgroundjobs_mode', 'ajax'), 'backgroundjobs_mode' => $this->config->getAppValue('core', 'backgroundjobs_mode', 'ajax'),
'cron_log' => $this->config->getSystemValue('cron_log', true),
'lastcron' => $this->config->getAppValue('core', 'lastcron', false), 'lastcron' => $this->config->getAppValue('core', 'lastcron', false),
'cronErrors' => $this->config->getAppValue('core', 'cronErrors'), 'cronErrors' => $this->config->getAppValue('core', 'cronErrors'),
'cli_based_cron_possible' => function_exists('posix_getpwuid'), 'cli_based_cron_possible' => function_exists('posix_getpwuid'),

2
settings/templates/settings/admin/server.php
View file

@ -156,7 +156,6 @@
<div class="section" id="backgroundjobs"> <div class="section" id="backgroundjobs">
<h2 class="inlineblock"><?php p($l->t('Background jobs'));?></h2> <h2 class="inlineblock"><?php p($l->t('Background jobs'));?></h2>
<?php if ($_['cron_log']): ?>
<p class="cronlog inlineblock"> <p class="cronlog inlineblock">
<?php if ($_['lastcron'] !== false): <?php if ($_['lastcron'] !== false):
$relative_time = relative_modified_date($_['lastcron']); $relative_time = relative_modified_date($_['lastcron']);
@ -177,7 +176,6 @@
<?php p($l->t("Background job didn’t run yet!")); <?php p($l->t("Background job didn’t run yet!"));
endif; ?> endif; ?>
</p> </p>
<?php endif; ?>
<a target="_blank" rel="noreferrer" class="icon-info" <a target="_blank" rel="noreferrer" class="icon-info"
title="<?php p($l->t('Open documentation'));?>" title="<?php p($l->t('Open documentation'));?>"
href="<?php p(link_to_docs('admin-background-jobs')); ?>"></a> href="<?php p(link_to_docs('admin-background-jobs')); ?>"></a>

10
tests/lib/Settings/Admin/ServerTest.php
View file

@ -80,12 +80,12 @@ class ServerTest extends TestCase {
->with('core', 'backgroundjobs_mode', 'ajax') ->with('core', 'backgroundjobs_mode', 'ajax')
->willReturn('ajax'); ->willReturn('ajax');
$this->config $this->config
->expects($this->at(4)) ->expects($this->at(3))
->method('getAppValue') ->method('getAppValue')
->with('core', 'lastcron', false) ->with('core', 'lastcron', false)
->willReturn(false); ->willReturn(false);
$this->config $this->config
->expects($this->at(5)) ->expects($this->at(4))
->method('getAppValue') ->method('getAppValue')
->with('core', 'cronErrors') ->with('core', 'cronErrors')
->willReturn(''); ->willReturn('');
@ -94,11 +94,6 @@ class ServerTest extends TestCase {
->method('getSystemValue') ->method('getSystemValue')
->with('check_for_working_wellknown_setup', true) ->with('check_for_working_wellknown_setup', true)
->willReturn(true); ->willReturn(true);
$this->config
->expects($this->at(3))
->method('getSystemValue')
->with('cron_log', true)
->willReturn(true);
$this->l10n $this->l10n
->expects($this->at(0)) ->expects($this->at(0))
->method('t') ->method('t')
@ -139,7 +134,6 @@ class ServerTest extends TestCase {
// Background jobs // Background jobs
'backgroundjobs_mode' => 'ajax', 'backgroundjobs_mode' => 'ajax',
'cron_log' => true,
'lastcron' => false, 'lastcron' => false,
'cronErrors' => '', 'cronErrors' => '',
'cli_based_cron_possible' => true, 'cli_based_cron_possible' => true,