Skip to content

fix: check for mountpoint conflict before rename in api#4271

Merged
provokateurin merged 1 commit intomasterfrom
setmountpoint-api-check
Jan 23, 2026
Merged

fix: check for mountpoint conflict before rename in api#4271
provokateurin merged 1 commit intomasterfrom
setmountpoint-api-check

Conversation

@icewind1991
Copy link
Copy Markdown
Member

Else it doesn't actually block a conflicting rename, and will always give an error.

Note that the webui uses a different endpoint, so the issue only happens for manual api usage.

@icewind1991 icewind1991 added this to the Nextcloud 33 milestone Jan 20, 2026
@icewind1991 icewind1991 added the 3. to review Items that need to be reviewed label Jan 20, 2026
@provokateurin provokateurin force-pushed the setmountpoint-api-check branch from 5e11ac2 to e49b64c Compare January 23, 2026 08:31
@provokateurin
Copy link
Copy Markdown
Member

/backport to stable33

@provokateurin
Copy link
Copy Markdown
Member

/backport to stable32

@provokateurin
Copy link
Copy Markdown
Member

/backport to stable31

Signed-off-by: Robin Appelman <robin@icewind.nl>
@provokateurin provokateurin force-pushed the setmountpoint-api-check branch from e49b64c to c7de16e Compare January 23, 2026 08:33
@provokateurin provokateurin merged commit 0944e28 into master Jan 23, 2026
53 of 54 checks passed
@provokateurin provokateurin deleted the setmountpoint-api-check branch January 23, 2026 08:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Items that need to be reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants