Skip to content

[stable29] fix(Token): add FILESYSTEM scope with SCOPE_SKIP_PASSWORD_VALIDATION - #46075

Merged
blizzz merged 1 commit into
stable29from
backport/46071/stable29
Jun 25, 2024
Merged

[stable29] fix(Token): add FILESYSTEM scope with SCOPE_SKIP_PASSWORD_VALIDATION#46075
blizzz merged 1 commit into
stable29from
backport/46071/stable29

Conversation

@blizzz

Copy link
Copy Markdown
Member

backport of #46071

The scope design requires scopes to be either not specified, or
specified explicitely. Therefore, when setting the
skip-password-validation scope for user authentication from mechanisms
like SAML, we also have to set the filesystem scope, otherwise they will
lack access to the filesystem.
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
@blizzzblizzz added bug 3. to review Waiting for reviews labels Jun 24, 2024
@blizzzblizzz added this to the Nextcloud 29.0.3 milestone Jun 24, 2024
@blizzz
blizzz requested review from a team, ChristophWurst, artonge, icewind1991, julien-nc, juliusknorr, nickvergessen and yemkareems and removed request for a teamJune 24, 2024 14:10
@blizzz

Copy link
Copy Markdown
MemberAuthor

/backport to stable28

@blizzz

Copy link
Copy Markdown
MemberAuthor

/backport to stable27

@blizzz

Copy link
Copy Markdown
MemberAuthor

/backport to stable26

@solracsf

Copy link
Copy Markdown
Member

Will this land on very-next minor versions (actually RCs)?

@blizzz

Copy link
Copy Markdown
MemberAuthor

Will this land on very-next minor versions (actually RCs)?

Will go right into todays finals 🤞 🍀

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

Labels

3. to reviewWaiting for reviewsbug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@blizzz@solracsf@juliusknorr@artonge