[james58899]

Because toIso8601String will lose the time zone information, if not converted to UTC the time will be incorrect.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment

Name	Status	Preview	Comments	Updated (UTC)
immich	⬜️ Ignored (Inspect)			Aug 1, 2023 6:06am

[alextran1502]

@james58899 Hello, can you please put an example of the current behavior vs. the changes that you made to the description of this PR?

[james58899]

I had a problem like #1320, the date of the image was changed to a future time after upload, and the time difference was exactly the timezone (Exactly 8 hours later, my timezone is UTC+8).

[AngelaDMerkel]

I have exactly this issue, and commented in #2953 with metadata.

[james58899]

I have exactly this issue, and commented in #2953 with metadata.

This should be a different issue, this PR only fixes the incorrect file time when uploading on the mobile app.

Because the screenshot does not have exif data, the file time will be used.
However, the APP does not convert to UTC and flutter does not support the format including the time zone, so the server treats the local time as UTC and causes an error.

[jrasm91]

I paused my metadata extraction job and then took an image and backed it up. It does have the wrong fileCreatedAt timestamp. We do indeed want to send ISO dates with the trailing Z, which this library doesn't do unless it is first converted to utc.