Skip to content

fix(dav): check the owner displayName scope before giving attribute - #52535

Merged
skjnldsv merged 2 commits into
masterfrom
fix/public-owner-scope
Apr 30, 2025
Merged

fix(dav): check the owner displayName scope before giving attribute#52535
skjnldsv merged 2 commits into
masterfrom
fix/public-owner-scope

Conversation

@skjnldsv

Copy link
Copy Markdown
Member

Caught after #45652

@skjnldsvskjnldsv added this to the Nextcloud 32 milestone Apr 29, 2025
@skjnldsvskjnldsv self-assigned this Apr 29, 2025
@skjnldsv
skjnldsv requested review from a team as code ownersApril 29, 2025 14:43
@skjnldsv
skjnldsv requested review from come-nc, nfebe, susnux, szaimen and yemkareems and removed request for a teamApril 29, 2025 14:43
@skjnldsv

skjnldsv commented Apr 29, 2025

Copy link
Copy Markdown
MemberAuthor

/backport 58aadde to stable31

@come-nccome-nc 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.

Good for me apart from typos between usermanager and accountmanager

Comment threadapps/dav/lib/Server.php Outdated
@szaimen
szaimen removed their request for review April 29, 2025 15:00
@skjnldsv
skjnldsvforce-pushed the fix/public-owner-scope branch 3 times, most recently from 03c10b6 to e80194aCompareApril 30, 2025 06:30
@skjnldsv

Copy link
Copy Markdown
MemberAuthor

Added tests

Signed-off-by: skjnldsv <skjnldsv@protonmail.com>
@skjnldsv
skjnldsvforce-pushed the fix/public-owner-scope branch from e80194a to 58aaddeCompareApril 30, 2025 06:37
@skjnldsv

Copy link
Copy Markdown
MemberAuthor

/compile

Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@skjnldsv

Copy link
Copy Markdown
MemberAuthor

/backport 58aadde to stable30

Sign up for freeto 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.

4 participants

@skjnldsv@susnux@come-nc@nextcloud-command