Skip to content

Commit

Permalink
enable microseconds in log timestamps
Browse files Browse the repository at this point in the history
  • Loading branch information
schnidrig authored and DeepDiver1975 committed Nov 12, 2015
1 parent af05be1 commit d0464bf
Show file tree
Hide file tree
Showing 2 changed files with 68 additions and 1 deletion.
2 changes: 1 addition & 1 deletion lib/private/log/owncloud.php
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,7 @@ public static function write($app, $message, $level) {
} catch (Exception $e) {
$timezone = new DateTimeZone('UTC');
}
$time = new DateTime(null, $timezone);
$time = DateTime::createFromFormat("U.u", microtime(true), $timezone);
$request = \OC::$server->getRequest();
$reqId = $request->getId();
$remoteAddr = $request->getRemoteAddress();
Expand Down
67 changes: 67 additions & 0 deletions tests/lib/log/owncloud.php
Original file line number Diff line number Diff line change
@@ -0,0 +1,67 @@
<?php
/**
*
* This library is free software; you can redistribute it and/or
* modify it under the terms of the GNU AFFERO GENERAL PUBLIC LICENSE
* License as published by the Free Software Foundation; either
* version 3 of the License, or any later version.
*
* This library is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
* GNU AFFERO GENERAL PUBLIC LICENSE for more details.
*
* You should have received a copy of the GNU Affero General Public
* License along with this library. If not, see <http://www.gnu.org/licenses/>.
*/

class Test_Log_Owncloud extends Test\TestCase
{
private $restore_logfile;
private $restore_logdateformat;

protected function setUp() {
parent::setUp();
$restore_logfile = OC_Config::getValue("logfile");
$restore_logdateformat = OC_Config::getValue('logdateformat');

OC_Config::setValue("logfile", OC_Config::getValue('datadirectory') . "/logtest");
OC_Log_Owncloud::init();
}
protected function tearDown() {
if (isset($this->restore_logfile)) {
OC_Config::setValue("logfile", $this->restore_logfile);
} else {
OC_Config::deleteKey("logfile");
}
if (isset($this->restore_logdateformat)) {
OC_Config::setValue("logdateformat", $this->restore_logdateformat);
} else {
OC_Config::deleteKey("restore_logdateformat");
}
OC_Log_Owncloud::init();
parent::tearDown();
}

public function testMicrosecondsLogTimestamp() {
# delete old logfile
unlink(OC_Config::getValue('logfile'));

# set format & write log line
OC_Config::setValue('logdateformat', 'u');
OC_Log_Owncloud::write('test', 'message', \OCP\Util::ERROR);

# read log line
$handle = @fopen(OC_Config::getValue('logfile'), 'r');
$line = fread($handle, 1000);
fclose($handle);

# check timestamp has microseconds part
$values = (array) json_decode($line);
$microseconds = $values['time'];
$this->assertNotEquals(0, $microseconds);

}


}

0 comments on commit d0464bf

Please sign in to comment.