Skip to content

fix(files_versions): Do not assume source exist when migrating versions to new storage - #56117

Merged
artonge merged 1 commit into
masterfrom
artonge/fix/files_versions/do_not_assume_when_migrating
Nov 6, 2025
Merged

fix(files_versions): Do not assume source exist when migrating versions to new storage#56117
artonge merged 1 commit into
masterfrom
artonge/fix/files_versions/do_not_assume_when_migrating

Conversation

@artonge

Copy link
Copy Markdown
Collaborator

No description provided.

@artonge
artonge requested a review from a team as a code ownerOctober 31, 2025 14:48
@artonge
artonge requested review from Altahrim, ArtificialOwl, icewind1991 and yemkareems and removed request for a teamOctober 31, 2025 14:48
@artongeartonge self-assigned this Oct 31, 2025
@artongeartonge added 3. to review Waiting for reviews bug feature: versions php Pull requests that update Php code labels Oct 31, 2025
@artongeartonge added this to the Nextcloud 33 milestone Oct 31, 2025
@artonge

Copy link
Copy Markdown
CollaboratorAuthor

/backport to stable32

@artonge

Copy link
Copy Markdown
CollaboratorAuthor

/backport to stable31

@artonge

Copy link
Copy Markdown
CollaboratorAuthor

/backport to stable30

@artonge

Copy link
Copy Markdown
CollaboratorAuthor

/backport to stable29

@susnuxsusnux left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

might be info level though

@artonge

Copy link
Copy Markdown
CollaboratorAuthor

might be info level though

Given that this is unexpected and could highlight a data lost or an inconsistency between FS and DB, I think a warning is appropriate.

…ns to new storage
Signed-off-by: Louis Chmn <louis@chmn.me>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsbugfeature: versionsphpPull requests that update Php code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@artonge@susnux@Altahrim@provokateurin