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

    rather than a UTC one they have to hack on a Z string to

    Ah, so that’s what I’m missing.

    That said, why not just send all dates in ISO 8601 format directly?

    Originally posted by NatoBoram in #3496