Skip to content

[stable31] fix(share): Set expiration time to end of day (23:59:59) - #58711

Merged
blizzz merged 2 commits into
stable31from
backport/58128/stable31
Aug 13, 2026
Merged

[stable31] fix(share): Set expiration time to end of day (23:59:59)#58711
blizzz merged 2 commits into
stable31from
backport/58128/stable31

Conversation

@backportbot

@backportbot backportbot Bot commented Mar 4, 2026

Copy link
Copy Markdown

Backport of #58128

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 the 3. to review Waiting for reviews label Mar 4, 2026

@jancborchardt jancborchardt left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving functionality design-wise, but not tested

Shares now expire at the end of the selected day instead of the
beginning, allowing access throughout the entire expiration day.

Also return actual stored time in API response instead of hardcoded
00:00:00 to prevent timezone-related display issues in the UI.

Signed-off-by: nfebe <fenn25.fn@gmail.com>
@susnux
susnux force-pushed the backport/58128/stable31 branch from e041c98 to d972d64 Compare August 13, 2026 13:47
@susnux susnux assigned susnux and unassigned nfebe Aug 13, 2026
@susnux
susnux removed the request for review from nfebe August 13, 2026 13:47
@susnux
susnux marked this pull request as ready for review August 13, 2026 13:48
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@susnux
susnux force-pushed the backport/58128/stable31 branch from d972d64 to d312540 Compare August 13, 2026 15:09
@blizzz
blizzz merged commit cd40539 into stable31 Aug 13, 2026
205 of 217 checks passed
@blizzz
blizzz deleted the backport/58128/stable31 branch August 13, 2026 16:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants