[PATCH] time.cgi: add current date-time to this WebGUI page

[PATCH] time.cgi: add current date-time to this WebGUI page

[Jon Murphy]

[-- Attachment #1: Type: text/plain, Size: 1678 bytes --]

- added words and date-time format to english (en.pl)
- other languages are needed
- seconds included since time is accurate to < .1s
https://git.ipfire.org/?p=ipfire-2.x.git;a=commit;h=2234e8aacac2e0d0b06dac4513585c15c2b3b440

Code-by: Leo-Andres Hofmann <hofmann(a)leo-andres.de>
Signed-off-by: Jon Murphy <jon.murphy(a)ipfire.org>
---
 html/cgi-bin/time.cgi  | 4 ++++
 langs/en/cgi-bin/en.pl | 2 ++
 2 files changed, 6 insertions(+)

diff --git a/html/cgi-bin/time.cgi b/html/cgi-bin/time.cgi
index 57a02a4b6..04c1e771f 100644
--- a/html/cgi-bin/time.cgi
+++ b/html/cgi-bin/time.cgi
@@ -287,6 +287,10 @@ print <<END
 </table>
 END
 ;
+
+my $now = strftime($Lang::tr{'timeformat'}, localtime);
+print "<hr>$Lang::tr{'system time'}: $now";
+
 &Header::closebox();
 &Header::openbox('100%',1,$Lang::tr{'ntp sync'});
 print <<END
diff --git a/langs/en/cgi-bin/en.pl b/langs/en/cgi-bin/en.pl
index 667fb5d7e..2e036adb3 100644
--- a/langs/en/cgi-bin/en.pl
+++ b/langs/en/cgi-bin/en.pl
@@ -2462,6 +2462,7 @@
 'system log viewer' => 'System Log Viewer',
 'system logs' => 'System Logs',
 'system status information' => 'System Status Information',
+'system time' => 'System time (as of last page load)',
 'ta key' => 'TLS-Authentification-Key',
 'taa zombieload2' => 'TSX Async Abort/ZombieLoad v2',
 'tcp more reliable' => 'TCP (more reliable)',
@@ -2490,6 +2491,7 @@
 'time' => 'Time',
 'time date manually reset' => 'Time/Date manually reset.',
 'time server' => 'Time Server',
+'timeformat' => '%Y-%m-%d at %H:%M:%S %Z',
 'timeout must be a number' => 'Timeout must be a number.',
 'title' => 'Title',
 'to' => 'To',
-- 
2.30.2

Re: [PATCH] time.cgi: add current date-time to this WebGUI page

[Michael Tremer]

[-- Attachment #1: Type: text/plain, Size: 1904 bytes --]

Hello Jon,

What is the motivation for this patch?

-Michael

> On 11 Mar 2024, at 23:45, Jon Murphy <jon.murphy(a)ipfire.org> wrote:
> 
> - added words and date-time format to english (en.pl)
> - other languages are needed
> - seconds included since time is accurate to < .1s
> https://git.ipfire.org/?p=ipfire-2.x.git;a=commit;h=2234e8aacac2e0d0b06dac4513585c15c2b3b440
> 
> Code-by: Leo-Andres Hofmann <hofmann(a)leo-andres.de>
> Signed-off-by: Jon Murphy <jon.murphy(a)ipfire.org>
> ---
> html/cgi-bin/time.cgi  | 4 ++++
> langs/en/cgi-bin/en.pl | 2 ++
> 2 files changed, 6 insertions(+)
> 
> diff --git a/html/cgi-bin/time.cgi b/html/cgi-bin/time.cgi
> index 57a02a4b6..04c1e771f 100644
> --- a/html/cgi-bin/time.cgi
> +++ b/html/cgi-bin/time.cgi
> @@ -287,6 +287,10 @@ print <<END
> </table>
> END
> ;
> +
> +my $now = strftime($Lang::tr{'timeformat'}, localtime);
> +print "<hr>$Lang::tr{'system time'}: $now";
> +
> &Header::closebox();
> &Header::openbox('100%',1,$Lang::tr{'ntp sync'});
> print <<END
> diff --git a/langs/en/cgi-bin/en.pl b/langs/en/cgi-bin/en.pl
> index 667fb5d7e..2e036adb3 100644
> --- a/langs/en/cgi-bin/en.pl
> +++ b/langs/en/cgi-bin/en.pl
> @@ -2462,6 +2462,7 @@
> 'system log viewer' => 'System Log Viewer',
> 'system logs' => 'System Logs',
> 'system status information' => 'System Status Information',
> +'system time' => 'System time (as of last page load)',
> 'ta key' => 'TLS-Authentification-Key',
> 'taa zombieload2' => 'TSX Async Abort/ZombieLoad v2',
> 'tcp more reliable' => 'TCP (more reliable)',
> @@ -2490,6 +2491,7 @@
> 'time' => 'Time',
> 'time date manually reset' => 'Time/Date manually reset.',
> 'time server' => 'Time Server',
> +'timeformat' => '%Y-%m-%d at %H:%M:%S %Z',
> 'timeout must be a number' => 'Timeout must be a number.',
> 'title' => 'Title',
> 'to' => 'To',
> -- 
> 2.30.2
> 

Re: [PATCH] time.cgi: add current date-time to this WebGUI page

[Michael Tremer]

[-- Attachment #1: Type: text/plain, Size: 5072 bytes --]

Hello,

> On 14 Mar 2024, at 17:19, jon <jon.murphy(a)ipfire.org> wrote:
> 
> Comments below.
> 
> Jon
> 
> 
>> On Mar 14, 2024, at 11:35 AM, Adolf Belka <adolf.belka(a)ipfire.org> wrote:
>> 
>> Hi Jon,
>> 
>> On 14/03/2024 16:48, jon wrote:
>>> Michael - is this good to go??
>>> 
>>> Jon
>>> 
>>> 
>>>> On Mar 12, 2024, at 11:18 AM, jon <jon.murphy(a)ipfire.org> wrote:
>>>> 
>>>> Back when I was playing with NTP and tried to make it better in 2022 I thought it would have been helpful but it wasn’t my focus and I did not have the skills to add it.  Recently there were mentions about adding date/time in the community.  So it seemed like a good time to add (no pun intended).
>>>> 
>>>> Jon Murphy
>>>> jon.murphy(a)ipfire.org
>>>> 
>>>> 
>>>> 
>>>>> On Mar 12, 2024, at 5:10 AM, Michael Tremer <michael.tremer(a)ipfire.org> wrote:
>>>>> 
>>>>> Hello Jon,
>>>>> 
>>>>> What is the motivation for this patch?
>>>>> 
>>>>> -Michael
>>>>> 
>>>>>> On 11 Mar 2024, at 23:45, Jon Murphy <jon.murphy(a)ipfire.org> wrote:
>>>>>> 
>>>>>> - added words and date-time format to english (en.pl)
>>>>>> - other languages are needed
>>>>>> - seconds included since time is accurate to < .1s
>> 
>> I think this could be an issue. The time might be accurate to 0.1 secs but it will only be updated when the browser page is refreshed. So I can imagine people saying that the clock is not working because the seconds haven't changed.
>> 
>> I have had that sort of problem in the past with the Connections Status page where people complained that the Connection expiry time was not updating unless they refreshed the browser page and they were sure it used to update every second in the past.
>> 
>> Certainly don't want to be ending up refreshing the browser page every second or less to show a change in the seconds value.
>> 
>> People will likely have a clock on the status panel of their OS desktop anyway so if the time matches on the hours and the minutes it will likely match the seconds as well.

I agree that showing time statically isn’t a good way, but I don’t know what we can improve really. If we use the browser’s time we are not showing the system’s time and that would be useless.

I believe for debugging this is good enough. It should just be there to figure out if your system is badly out of time.

> I understand the concern and that was the reason for the "(as of last page load)" was added the the system time.  The output looks like:
> 
> "System time (as of last page load): 2024-03-14 at 11:40:22 CDT"
> 
> And can easily be changed with the language files (I only did the `en.pl` file)

I added a German translation:

  https://git.ipfire.org/?p=ipfire-2.x.git;a=commitdiff;h=16492046bc7f50e2c63e2908a7ace13d1548e764

> And since many other WebGUI pages do not auto-update, I think this will be OK. 
> 
> PS -  If it becomes an issue, we could always add an "Update" button (like the log pages) to the right of the time.

I think a page refresh will do.

I we wanted to do it properly live, we would need to send requests to the web service and ask for the time repeatedly.

-Michael

> 
> 
>> Regards,
>> 
>> Adolf.
>> 
>>>>>> https://git.ipfire.org/?p=ipfire-2.x.git;a=commit;h=2234e8aacac2e0d0b06dac4513585c15c2b3b440
>>>>>> 
>>>>>> Code-by: Leo-Andres Hofmann <hofmann(a)leo-andres.de>
>>>>>> Signed-off-by: Jon Murphy <jon.murphy(a)ipfire.org>
>>>>>> ---
>>>>>> html/cgi-bin/time.cgi  | 4 ++++
>>>>>> langs/en/cgi-bin/en.pl | 2 ++
>>>>>> 2 files changed, 6 insertions(+)
>>>>>> 
>>>>>> diff --git a/html/cgi-bin/time.cgi b/html/cgi-bin/time.cgi
>>>>>> index 57a02a4b6..04c1e771f 100644
>>>>>> --- a/html/cgi-bin/time.cgi
>>>>>> +++ b/html/cgi-bin/time.cgi
>>>>>> @@ -287,6 +287,10 @@ print <<END
>>>>>> </table>
>>>>>> END
>>>>>> ;
>>>>>> +
>>>>>> +my $now = strftime($Lang::tr{'timeformat'}, localtime);
>>>>>> +print "<hr>$Lang::tr{'system time'}: $now";
>>>>>> +
>>>>>> &Header::closebox();
>>>>>> &Header::openbox('100%',1,$Lang::tr{'ntp sync'});
>>>>>> print <<END
>>>>>> diff --git a/langs/en/cgi-bin/en.pl b/langs/en/cgi-bin/en.pl
>>>>>> index 667fb5d7e..2e036adb3 100644
>>>>>> --- a/langs/en/cgi-bin/en.pl
>>>>>> +++ b/langs/en/cgi-bin/en.pl
>>>>>> @@ -2462,6 +2462,7 @@
>>>>>> 'system log viewer' => 'System Log Viewer',
>>>>>> 'system logs' => 'System Logs',
>>>>>> 'system status information' => 'System Status Information',
>>>>>> +'system time' => 'System time (as of last page load)',
>>>>>> 'ta key' => 'TLS-Authentification-Key',
>>>>>> 'taa zombieload2' => 'TSX Async Abort/ZombieLoad v2',
>>>>>> 'tcp more reliable' => 'TCP (more reliable)',
>>>>>> @@ -2490,6 +2491,7 @@
>>>>>> 'time' => 'Time',
>>>>>> 'time date manually reset' => 'Time/Date manually reset.',
>>>>>> 'time server' => 'Time Server',
>>>>>> +'timeformat' => '%Y-%m-%d at %H:%M:%S %Z',
>>>>>> 'timeout must be a number' => 'Timeout must be a number.',
>>>>>> 'title' => 'Title',
>>>>>> 'to' => 'To',
>>>>>> -- 
>>>>>> 2.30.2

Re: [PATCH] time.cgi: add current date-time to this WebGUI page

[Michael Tremer]

[-- Attachment #1: Type: text/plain, Size: 2887 bytes --]

Hello Jon,

Yes, this is good to go. I just didn’t have time to test the patch, yet.

I merged it yesterday.

-Michael

> On 14 Mar 2024, at 15:48, jon <jon.murphy(a)ipfire.org> wrote:
> 
> Michael - is this good to go??
> 
> Jon
> 
> 
>> On Mar 12, 2024, at 11:18 AM, jon <jon.murphy(a)ipfire.org> wrote:
>> 
>> Back when I was playing with NTP and tried to make it better in 2022 I thought it would have been helpful but it wasn’t my focus and I did not have the skills to add it.  Recently there were mentions about adding date/time in the community.  So it seemed like a good time to add (no pun intended).
>> 
>> Jon Murphy
>> jon.murphy(a)ipfire.org
>> 
>> 
>> 
>>> On Mar 12, 2024, at 5:10 AM, Michael Tremer <michael.tremer(a)ipfire.org> wrote:
>>> 
>>> Hello Jon,
>>> 
>>> What is the motivation for this patch?
>>> 
>>> -Michael
>>> 
>>>> On 11 Mar 2024, at 23:45, Jon Murphy <jon.murphy(a)ipfire.org> wrote:
>>>> 
>>>> - added words and date-time format to english (en.pl)
>>>> - other languages are needed
>>>> - seconds included since time is accurate to < .1s
>>>> https://git.ipfire.org/?p=ipfire-2.x.git;a=commit;h=2234e8aacac2e0d0b06dac4513585c15c2b3b440
>>>> 
>>>> Code-by: Leo-Andres Hofmann <hofmann(a)leo-andres.de>
>>>> Signed-off-by: Jon Murphy <jon.murphy(a)ipfire.org>
>>>> ---
>>>> html/cgi-bin/time.cgi  | 4 ++++
>>>> langs/en/cgi-bin/en.pl | 2 ++
>>>> 2 files changed, 6 insertions(+)
>>>> 
>>>> diff --git a/html/cgi-bin/time.cgi b/html/cgi-bin/time.cgi
>>>> index 57a02a4b6..04c1e771f 100644
>>>> --- a/html/cgi-bin/time.cgi
>>>> +++ b/html/cgi-bin/time.cgi
>>>> @@ -287,6 +287,10 @@ print <<END
>>>> </table>
>>>> END
>>>> ;
>>>> +
>>>> +my $now = strftime($Lang::tr{'timeformat'}, localtime);
>>>> +print "<hr>$Lang::tr{'system time'}: $now";
>>>> +
>>>> &Header::closebox();
>>>> &Header::openbox('100%',1,$Lang::tr{'ntp sync'});
>>>> print <<END
>>>> diff --git a/langs/en/cgi-bin/en.pl b/langs/en/cgi-bin/en.pl
>>>> index 667fb5d7e..2e036adb3 100644
>>>> --- a/langs/en/cgi-bin/en.pl
>>>> +++ b/langs/en/cgi-bin/en.pl
>>>> @@ -2462,6 +2462,7 @@
>>>> 'system log viewer' => 'System Log Viewer',
>>>> 'system logs' => 'System Logs',
>>>> 'system status information' => 'System Status Information',
>>>> +'system time' => 'System time (as of last page load)',
>>>> 'ta key' => 'TLS-Authentification-Key',
>>>> 'taa zombieload2' => 'TSX Async Abort/ZombieLoad v2',
>>>> 'tcp more reliable' => 'TCP (more reliable)',
>>>> @@ -2490,6 +2491,7 @@
>>>> 'time' => 'Time',
>>>> 'time date manually reset' => 'Time/Date manually reset.',
>>>> 'time server' => 'Time Server',
>>>> +'timeformat' => '%Y-%m-%d at %H:%M:%S %Z',
>>>> 'timeout must be a number' => 'Timeout must be a number.',
>>>> 'title' => 'Title',
>>>> 'to' => 'To',
>>>> -- 
>>>> 2.30.2
>>>> 
>>> 
>> 
> 

Re: [PATCH] time.cgi: add current date-time to this WebGUI page

[jon]

[-- Attachment #1: Type: text/plain, Size: 4267 bytes --]

Comments below.

Jon


> On Mar 14, 2024, at 11:35 AM, Adolf Belka <adolf.belka(a)ipfire.org> wrote:
> 
> Hi Jon,
> 
> On 14/03/2024 16:48, jon wrote:
>> Michael - is this good to go??
>> 
>> Jon
>> 
>> 
>>> On Mar 12, 2024, at 11:18 AM, jon <jon.murphy(a)ipfire.org> wrote:
>>> 
>>> Back when I was playing with NTP and tried to make it better in 2022 I thought it would have been helpful but it wasn’t my focus and I did not have the skills to add it.  Recently there were mentions about adding date/time in the community.  So it seemed like a good time to add (no pun intended).
>>> 
>>> Jon Murphy
>>> jon.murphy(a)ipfire.org
>>> 
>>> 
>>> 
>>>> On Mar 12, 2024, at 5:10 AM, Michael Tremer <michael.tremer(a)ipfire.org> wrote:
>>>> 
>>>> Hello Jon,
>>>> 
>>>> What is the motivation for this patch?
>>>> 
>>>> -Michael
>>>> 
>>>>> On 11 Mar 2024, at 23:45, Jon Murphy <jon.murphy(a)ipfire.org> wrote:
>>>>> 
>>>>> - added words and date-time format to english (en.pl)
>>>>> - other languages are needed
>>>>> - seconds included since time is accurate to < .1s
> 
> I think this could be an issue. The time might be accurate to 0.1 secs but it will only be updated when the browser page is refreshed. So I can imagine people saying that the clock is not working because the seconds haven't changed.
> 
> I have had that sort of problem in the past with the Connections Status page where people complained that the Connection expiry time was not updating unless they refreshed the browser page and they were sure it used to update every second in the past.
> 
> Certainly don't want to be ending up refreshing the browser page every second or less to show a change in the seconds value.
> 
> People will likely have a clock on the status panel of their OS desktop anyway so if the time matches on the hours and the minutes it will likely match the seconds as well.
> 

I understand the concern and that was the reason for the "(as of last page load)" was added the the system time.  The output looks like:

"System time (as of last page load): 2024-03-14 at 11:40:22 CDT"

And can easily be changed with the language files (I only did the `en.pl` file)

And since many other WebGUI pages do not auto-update, I think this will be OK. 

PS -  If it becomes an issue, we could always add an "Update" button (like the log pages) to the right of the time.


> Regards,
> 
> Adolf.
> 
>>>>> https://git.ipfire.org/?p=ipfire-2.x.git;a=commit;h=2234e8aacac2e0d0b06dac4513585c15c2b3b440
>>>>> 
>>>>> Code-by: Leo-Andres Hofmann <hofmann(a)leo-andres.de>
>>>>> Signed-off-by: Jon Murphy <jon.murphy(a)ipfire.org>
>>>>> ---
>>>>> html/cgi-bin/time.cgi  | 4 ++++
>>>>> langs/en/cgi-bin/en.pl | 2 ++
>>>>> 2 files changed, 6 insertions(+)
>>>>> 
>>>>> diff --git a/html/cgi-bin/time.cgi b/html/cgi-bin/time.cgi
>>>>> index 57a02a4b6..04c1e771f 100644
>>>>> --- a/html/cgi-bin/time.cgi
>>>>> +++ b/html/cgi-bin/time.cgi
>>>>> @@ -287,6 +287,10 @@ print <<END
>>>>> </table>
>>>>> END
>>>>> ;
>>>>> +
>>>>> +my $now = strftime($Lang::tr{'timeformat'}, localtime);
>>>>> +print "<hr>$Lang::tr{'system time'}: $now";
>>>>> +
>>>>> &Header::closebox();
>>>>> &Header::openbox('100%',1,$Lang::tr{'ntp sync'});
>>>>> print <<END
>>>>> diff --git a/langs/en/cgi-bin/en.pl b/langs/en/cgi-bin/en.pl
>>>>> index 667fb5d7e..2e036adb3 100644
>>>>> --- a/langs/en/cgi-bin/en.pl
>>>>> +++ b/langs/en/cgi-bin/en.pl
>>>>> @@ -2462,6 +2462,7 @@
>>>>> 'system log viewer' => 'System Log Viewer',
>>>>> 'system logs' => 'System Logs',
>>>>> 'system status information' => 'System Status Information',
>>>>> +'system time' => 'System time (as of last page load)',
>>>>> 'ta key' => 'TLS-Authentification-Key',
>>>>> 'taa zombieload2' => 'TSX Async Abort/ZombieLoad v2',
>>>>> 'tcp more reliable' => 'TCP (more reliable)',
>>>>> @@ -2490,6 +2491,7 @@
>>>>> 'time' => 'Time',
>>>>> 'time date manually reset' => 'Time/Date manually reset.',
>>>>> 'time server' => 'Time Server',
>>>>> +'timeformat' => '%Y-%m-%d at %H:%M:%S %Z',
>>>>> 'timeout must be a number' => 'Timeout must be a number.',
>>>>> 'title' => 'Title',
>>>>> 'to' => 'To',
>>>>> -- 
>>>>> 2.30.2
>>>>> 
>>>> 
>>> 
>> 

Re: [PATCH] time.cgi: add current date-time to this WebGUI page

[Adolf Belka]

[-- Attachment #1: Type: text/plain, Size: 3532 bytes --]

Hi Jon,

On 14/03/2024 16:48, jon wrote:
> Michael - is this good to go??
>
> Jon
>
>
>> On Mar 12, 2024, at 11:18 AM, jon <jon.murphy(a)ipfire.org> wrote:
>>
>> Back when I was playing with NTP and tried to make it better in 2022 I thought it would have been helpful but it wasn’t my focus and I did not have the skills to add it.  Recently there were mentions about adding date/time in the community.  So it seemed like a good time to add (no pun intended).
>>
>> Jon Murphy
>> jon.murphy(a)ipfire.org
>>
>>
>>
>>> On Mar 12, 2024, at 5:10 AM, Michael Tremer <michael.tremer(a)ipfire.org> wrote:
>>>
>>> Hello Jon,
>>>
>>> What is the motivation for this patch?
>>>
>>> -Michael
>>>
>>>> On 11 Mar 2024, at 23:45, Jon Murphy <jon.murphy(a)ipfire.org> wrote:
>>>>
>>>> - added words and date-time format to english (en.pl)
>>>> - other languages are needed
>>>> - seconds included since time is accurate to < .1s

I think this could be an issue. The time might be accurate to 0.1 secs but it will only be updated when the browser page is refreshed. So I can imagine people saying that the clock is not working because the seconds haven't changed.

I have had that sort of problem in the past with the Connections Status page where people complained that the Connection expiry time was not updating unless they refreshed the browser page and they were sure it used to update every second in the past.

Certainly don't want to be ending up refreshing the browser page every second or less to show a change in the seconds value.

People will likely have a clock on the status panel of their OS desktop anyway so if the time matches on the hours and the minutes it will likely match the seconds as well.

Regards,

Adolf.

>>>> https://git.ipfire.org/?p=ipfire-2.x.git;a=commit;h=2234e8aacac2e0d0b06dac4513585c15c2b3b440
>>>>
>>>> Code-by: Leo-Andres Hofmann <hofmann(a)leo-andres.de>
>>>> Signed-off-by: Jon Murphy <jon.murphy(a)ipfire.org>
>>>> ---
>>>> html/cgi-bin/time.cgi  | 4 ++++
>>>> langs/en/cgi-bin/en.pl | 2 ++
>>>> 2 files changed, 6 insertions(+)
>>>>
>>>> diff --git a/html/cgi-bin/time.cgi b/html/cgi-bin/time.cgi
>>>> index 57a02a4b6..04c1e771f 100644
>>>> --- a/html/cgi-bin/time.cgi
>>>> +++ b/html/cgi-bin/time.cgi
>>>> @@ -287,6 +287,10 @@ print <<END
>>>> </table>
>>>> END
>>>> ;
>>>> +
>>>> +my $now = strftime($Lang::tr{'timeformat'}, localtime);
>>>> +print "<hr>$Lang::tr{'system time'}: $now";
>>>> +
>>>> &Header::closebox();
>>>> &Header::openbox('100%',1,$Lang::tr{'ntp sync'});
>>>> print <<END
>>>> diff --git a/langs/en/cgi-bin/en.pl b/langs/en/cgi-bin/en.pl
>>>> index 667fb5d7e..2e036adb3 100644
>>>> --- a/langs/en/cgi-bin/en.pl
>>>> +++ b/langs/en/cgi-bin/en.pl
>>>> @@ -2462,6 +2462,7 @@
>>>> 'system log viewer' => 'System Log Viewer',
>>>> 'system logs' => 'System Logs',
>>>> 'system status information' => 'System Status Information',
>>>> +'system time' => 'System time (as of last page load)',
>>>> 'ta key' => 'TLS-Authentification-Key',
>>>> 'taa zombieload2' => 'TSX Async Abort/ZombieLoad v2',
>>>> 'tcp more reliable' => 'TCP (more reliable)',
>>>> @@ -2490,6 +2491,7 @@
>>>> 'time' => 'Time',
>>>> 'time date manually reset' => 'Time/Date manually reset.',
>>>> 'time server' => 'Time Server',
>>>> +'timeformat' => '%Y-%m-%d at %H:%M:%S %Z',
>>>> 'timeout must be a number' => 'Timeout must be a number.',
>>>> 'title' => 'Title',
>>>> 'to' => 'To',
>>>> -- 
>>>> 2.30.2
>>>>
>>>
>>
>