Opened 4 years ago

Closed 3 months ago

#719 closed Feature Wish (fixed)

replace ctime with POSIX time format

Reported by: Gert Döring Owned by:
Priority: major Milestone: release 2.5
Component: Generic / unclassified Version: OpenVPN git master branch (Community Ed)
Severity: Not set (select this one, unless your'e a OpenVPN developer) Keywords:
Cc:

Description

if we time-stamp "things" today, we use time_string() in otime.c, which uses ctime() underneath - which has this weird "Thu Nov 24 18:22:48 1986" time format - neither sortable, nor particularily nice for humans either.

People have approached me to replace this with POSIX time stamping (2016-08-11 10:45:03).

The change is trivial, but the potential users of time_string need to be looked at, if anything relies on that time format.

(Logging using --machine-readable-timestamp is not affected, neither is syslogging)

Change History (4)

comment:1 Changed 4 years ago by Gert Döring

Milestone: release 2.4

comment:2 Changed 4 years ago by Gert Döring

Milestone: release 2.4release 2.5

we decided that we're not going to make a change that fundamental "in the release end-run", so bumping to 2.5

comment:3 Changed 3 months ago by Gert Döring

comment:4 Changed 3 months ago by Gert Döring

Resolution: fixed
Status: newclosed

commit ff063b6f19e035da56fbf49c891e6376543b391d
Author: Gert Doering <gert@…>
Date: Sat Jun 20 19:23:03 2020 +0200

Change timestamps in file-based logging to ISO 8601 time format.

merged & pushed (for 2.5-only)

Note: See TracTickets for help on using tickets.