Skip to content

Fix/more missing userid cases#58144

Open
salmart-dev wants to merge 2 commits intomasterfrom
fix/more-missing-userid-cases
Open

Fix/more missing userid cases#58144
salmart-dev wants to merge 2 commits intomasterfrom
fix/more-missing-userid-cases

Conversation

@salmart-dev
Copy link
Contributor

@salmart-dev salmart-dev commented Feb 6, 2026

Summary

Sets the user id header in DAV responses in beforeMethod instead of afterMethod, this ensures that if some plugins stop the event propagation, the user id header will still be set.

Adds the same header in the redirect response when calling GET /index.php which redirects to the default app.

Checklist

Signed-off-by: Salvatore Martire <4652631+salmart-dev@users.noreply.github.com>
Signed-off-by: Salvatore Martire <4652631+salmart-dev@users.noreply.github.com>
@salmart-dev salmart-dev added this to the Nextcloud 34 milestone Feb 6, 2026
@salmart-dev salmart-dev self-assigned this Feb 6, 2026
@salmart-dev salmart-dev requested a review from a team as a code owner February 6, 2026 19:28
@salmart-dev salmart-dev added bug 3. to review Waiting for reviews labels Feb 6, 2026
@salmart-dev salmart-dev requested review from Altahrim, ArtificialOwl, come-nc and icewind1991 and removed request for a team February 6, 2026 19:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant