Ticket #434 (closed defect: fixed)

Opened 4 years ago

Last modified 9 months ago

Translate all times to UTC

Reported by: athomas Assigned to: athomas
Priority: normal Component: XmlRpcPlugin
Severity: normal Keywords:
Cc: Trac Release: 0.10

Description

All times should be in UTC, converted to/from localtime during transport.

Attachments

xmlrpc-utc.patch (480 bytes) - added by stp on 08/30/06 20:46:10.
Expects date object to be in UTC

Change History

08/26/06 19:08:48 changed by steffenp@gmx.de

Does that mean all time stamps returned by XML-RPC currently in the server's time zone? If yes, is there a way to retrieve that time zone?

08/30/06 20:46:10 changed by stp

  • attachment xmlrpc-utc.patch added.

08/30/06 20:46:10 changed by stp

I have looked into the time conversion and I think the attached patch will at least resolve this report for the ticket API. It expects date objects to be in UTC.

All returned time stamps are integers (seconds elapsed since 1970 in UTC) so nothing needs be changed there.

08/30/06 20:46:10 changed by stp

Expects date object to be in UTC

12/24/06 14:37:51 changed by athomas

  • status changed from new to closed.
  • resolution set to fixed.

(In [1735]) Applied patches from gotoh and stp, thanks. Closes #434 and #845.

12/24/06 14:39:29 changed by athomas

  • status changed from closed to reopened.
  • resolution deleted.

Reopening because, as stp notes, this doesn't convert some timestamps in returned structures correctly.

06/12/09 22:39:13 changed by osimons

As far as my testing goes I cannot see anything wrong when using current trunk and a fresh Trac 0.11.4+. If anyone has details/example of something that does not return expected value, please add it. If not I'll re-close the ticket again.

06/13/09 00:03:16 changed by osimons

  • status changed from reopened to closed.
  • resolution set to fixed.

Found some. #3011 and #4539 at least. Preparing a patch on #3011, so following it up there.

Add/Change #434 (Translate all times to UTC)




Change Properties
Action