Skip to content

Conversation

@backportbot
Copy link

@backportbot backportbot bot commented Aug 18, 2025

Backport of #54402

Warning, This backport's changes differ from the original and might be incomplete ⚠️

Todo

  • Review and resolve any conflicts
  • Amend HEAD commit to remove the line stating to skip CI

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

@backportbot backportbot bot added bug 3. to review Waiting for reviews hotspot: file time handling ctime, mtime, etc. handling during various operations labels Aug 18, 2025
@backportbot backportbot bot added this to the Nextcloud 31.0.9 milestone Aug 18, 2025
ZIP does not use a proper timestamp but uses something called "DOS time".
This is a weird old format with some limitations like accuracy of only
2 seconds, but also no timezone information.
Also unline UNIX time it is not relative to some specific point in time
with timezone information, but is always considered to be the local
time. Meaning we need to convert it first to the users local time.

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@susnux susnux force-pushed the backport/54402/stable31 branch from 94d6e45 to 3e61569 Compare October 14, 2025 13:27
@susnux susnux marked this pull request as ready for review October 14, 2025 13:27
@susnux susnux added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Oct 14, 2025
@susnux susnux merged commit 0634b62 into stable31 Oct 14, 2025
208 of 219 checks passed
@susnux susnux deleted the backport/54402/stable31 branch October 14, 2025 23:08
@blizzz blizzz mentioned this pull request Oct 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to release Ready to be released and/or waiting for tests to finish bug hotspot: file time handling ctime, mtime, etc. handling during various operations

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants