Skip to content

[stable27] fix(Token): add FILESYSTEM scope with SCOPE_SKIP_PASSWORD_VALIDATION - #46087

Merged
blizzz merged 1 commit into
stable27from
backport/46075/stable27
Jun 25, 2024
Merged

[stable27] fix(Token): add FILESYSTEM scope with SCOPE_SKIP_PASSWORD_VALIDATION#46087
blizzz merged 1 commit into
stable27from
backport/46075/stable27

Conversation

@backportbot

Copy link
Copy Markdown

Backport of PR #46075

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>
@backportbotbackportbotBot added bug 3. to review Waiting for reviews labels Jun 25, 2024
@backportbotbackportbotBot added this to the Nextcloud 27.1.11 milestone Jun 25, 2024
@blizzzblizzz changed the title [stable27] [stable29] fix(Token): add FILESYSTEM scope with SCOPE_SKIP_PASSWORD_VALIDATION[stable27] fix(Token): add FILESYSTEM scope with SCOPE_SKIP_PASSWORD_VALIDATIONJun 25, 2024
@ChristophWurstChristophWurst added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Jun 25, 2024
@blizzz
blizzz merged commit 5be77d3 into stable27Jun 25, 2024
@blizzz
blizzz deleted the backport/46075/stable27 branch June 25, 2024 10:21
@blizzzblizzz mentioned this pull request Jun 24, 2024
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to releaseReady to be released and/or waiting for tests to finishbug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@nickvergessen@blizzz@Altahrim@ChristophWurst