Skip to content

Conversation

@icewind1991
Copy link
Member

Fixes a performance regression from #50711 (review)

The dav propfind will have already made sure we have access to the node.

The check is left in place for other callers

Signed-off-by: Robin Appelman <robin@icewind.nl>
@icewind1991 icewind1991 added the 3. to review Waiting for reviews label Apr 25, 2025
@icewind1991 icewind1991 added this to the Nextcloud 32 milestone Apr 25, 2025
@icewind1991 icewind1991 requested review from artonge and come-nc April 25, 2025 18:36
@icewind1991 icewind1991 requested a review from a team as a code owner April 25, 2025 18:36
@icewind1991 icewind1991 requested review from Altahrim and removed request for a team April 25, 2025 18:36
@AndyScherzinger
Copy link
Member

/backport to stable31

@AndyScherzinger
Copy link
Member

/backport to stable30

@AndyScherzinger AndyScherzinger merged commit 0d19a13 into master Apr 28, 2025
197 of 199 checks passed
@AndyScherzinger AndyScherzinger deleted the reminder-dont-validiate-node-dav branch April 28, 2025 12:51
@backportbot
Copy link

backportbot bot commented Apr 28, 2025

The backport to stable30 failed. Please do this backport manually.

# Switch to the target branch and update it
git checkout stable30
git pull origin stable30

# Create the new backport branch
git checkout -b backport/52443/stable30

# Cherry pick the change from the commit sha1 of the change against the default branch
# This might cause conflicts, resolve them
git cherry-pick 3d36625a

# Push the cherry pick commit to the remote repository and open a pull request
git push origin backport/52443/stable30

Error: Failed to check for changes with origin/stable30: No changes found in backport branch


Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants