Skip to content

[stable33] feat: perform share mount validation on share instead of on mount - #57727

Closed
backportbot[bot] wants to merge 5 commits into
stable33from
backport/57295/stable33
Closed

[stable33] feat: perform share mount validation on share instead of on mount#57727
backportbot[bot] wants to merge 5 commits into
stable33from
backport/57295/stable33

Conversation

@backportbot

@backportbot backportbot Bot commented Jan 22, 2026

Copy link
Copy Markdown

Backport of PR #57295

@backportbot
backportbot Bot requested a review from a team as a code owner January 22, 2026 12:58
@backportbot
backportbot Bot requested review from ArtificialOwl, CarlSchwan, artonge, icewind1991 and salmart-dev and removed request for a team January 22, 2026 12:58
@backportbot backportbot Bot added the 3. to review Waiting for reviews label Jan 22, 2026
@backportbot
backportbot Bot requested a review from leftybournes January 22, 2026 12:59
@backportbot backportbot Bot added this to the Nextcloud 33 milestone Jan 22, 2026
Signed-off-by: Robin Appelman <robin@icewind.nl>
…e positives

Signed-off-by: Robin Appelman <robin@icewind.nl>
Signed-off-by: Robin Appelman <robin@icewind.nl>
Signed-off-by: Robin Appelman <robin@icewind.nl>
Signed-off-by: Robin Appelman <robin@icewind.nl>

@nickvergessen nickvergessen 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.

Currently breaking Talk/ folder replacements unexpectingly.
So blocking until this is clarified:
nextcloud/spreed#16717

@susnux

susnux commented Feb 3, 2026

Copy link
Copy Markdown
Contributor

This seems to also break tests locally (the specific test case does not matter I see this everywhere):

  1. Test\AppFramework\Http\FileDisplayResponseTest::testHeader
    TypeError: OC\Files\Config\UserMountCache::__construct(): Argument # 1 ($connection) must be of type OCP\IDBConnection, null given, called in /server/lib/private/AppFramework/Utility/SimpleContainer.php on line 71

/server/lib/private/Files/Config/UserMountCache.php:51
/server/lib/private/AppFramework/Utility/SimpleContainer.php:71
/server/lib/private/Files/Config/UserMountCache.php:567
/server/tests/lib/TestCase.php:184

This was referenced Feb 11, 2026
@blizzz blizzz modified the milestones: Nextcloud 33, Nextcloud 33.0.1 Feb 16, 2026
@salmart-dev

Copy link
Copy Markdown
Contributor

Given that this got merged #57822, this PR can be closed, right @icewind1991 ?

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

Labels

5 participants