Skip to content

[stable-4.0] fix(sharing): parse received timestamp from expiration date#9888

Merged
nilsding merged 1 commit intostable-4.0from
backport/9882/stable-4.0
Apr 22, 2026
Merged

[stable-4.0] fix(sharing): parse received timestamp from expiration date#9888
nilsding merged 1 commit intostable-4.0from
backport/9882/stable-4.0

Conversation

@backportbot
Copy link
Copy Markdown

@backportbot backportbot Bot commented Apr 22, 2026

Backport of PR #9882

Server version >= 33.0.1 changed the behaviour of the expiration date:
instead of expiring shares at the beginning of the day, they now expire
at the end of the day.

The client previously relied on the expiry time being defined as
`00:00:00`, this leads to the date unable to be parsed with the expiry
time set to `23:59:59`.
Solution: just™ parse the entire time returned from the server as well!
The client sets the expiration date only as `YYYY-MM-DD` anyway, so it
does not need to care about the time suggested by the server at all ...

Resolves #9859

Signed-off-by: Jyrki Gadinger <nilsding@nilsding.org>
@backportbot backportbot Bot requested a review from Aiiaiiio as a code owner April 22, 2026 08:19
@backportbot backportbot Bot requested a review from camilasan April 22, 2026 08:19
@backportbot backportbot Bot added bug feature: 👥 sharing Sharing dialog and functionalities. 3. to review labels Apr 22, 2026
@backportbot backportbot Bot added this to the 4.0.9 milestone Apr 22, 2026
@nilsding nilsding enabled auto-merge April 22, 2026 08:24
@github-actions
Copy link
Copy Markdown

Artifact containing the AppImage: nextcloud-appimage-pr-9888.zip

Digest: sha256:e8220b7ca185fd1d22c4d9f4e1dcf7cf498fff946415d99ffea5d4ba4a6e1cd9

To test this change/fix you can download the above artifact file, unzip it, and run it.

Please make sure to quit your existing Nextcloud app and backup your data.

@nilsding nilsding merged commit 03629a0 into stable-4.0 Apr 22, 2026
16 of 17 checks passed
@nilsding nilsding deleted the backport/9882/stable-4.0 branch April 22, 2026 08:48
@sonarqubecloud
Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review bug feature: 👥 sharing Sharing dialog and functionalities.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant