Fix time zone handling by phiresky · Pull Request #3496 · LemmyNet/lemmy
github.com
The whole code base uses naive date time and TIMESTAMP WITHOUT TIME ZONE. Now there's conversion issues appearing in federation where timezones are not handled correctly. Using NaiveDateTime is ......

The whole code base uses naive date time and TIMESTAMP WITHOUT TIME ZONE.

Now there’s conversion issues appearing in federation where timezones are not handled correctly.

Using NaiveDateTime is … naive? So this PR removes all instances of naive time and adds defined timezones to everything (usually Utc).

Now, no conversions are needed anymore.

The database migration is extensive but it is free (no rewrite needed) since PG 12.

Originally posted by phiresky in #3496

  • issue_tracking_bot@lemm.eeOPB
    1·
    1 year ago

    After this, they will. We don’t manipulate the columns coming back from postgres, so UTC timestamp without timezone doesn’t include timestamp info for the 8601 format.

    Originally posted by dessalines in #3496