Skip to content

fix(files_external): Allow editing auth mechanism on saved mounts#61312

Merged
susnux merged 2 commits into
masterfrom
fix/files-external-edit-auth-mechanism
Jun 22, 2026
Merged

fix(files_external): Allow editing auth mechanism on saved mounts#61312
susnux merged 2 commits into
masterfrom
fix/files-external-edit-auth-mechanism

Conversation

@nfebe

@nfebe nfebe commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

Restores the pre-33 behaviour where admins can change the authentication mechanism of an existing external storage mount through the settings UI. After the External Storage settings rewrite the dropdown was locked read-only for any mount with an id, leaving the value configured at creation time as the only one ever shown. Backend type stays locked because changing it would invalidate the mount; authentication mechanism is independent and was always meant to remain editable.

@nfebe nfebe requested review from a team and skjnldsv as code owners June 15, 2026 23:01
@nfebe nfebe requested review from sorbaugh and susnux and removed request for a team June 15, 2026 23:01
@nfebe nfebe force-pushed the fix/files-external-edit-auth-mechanism branch 2 times, most recently from 260dce5 to 2e66d5b Compare June 19, 2026 12:34
@susnux susnux added this to the Nextcloud 35 milestone Jun 22, 2026
Restores the pre-33 behaviour where admins can change the authentication
mechanism of an existing external storage mount through the settings UI.
After the External Storage settings rewrite the dropdown was locked
read-only for any mount with an id, leaving the value configured at
creation time as the only one ever shown. Backend type stays locked
because changing it would invalidate the mount; authentication
mechanism is independent and was always meant to remain editable.

Switching the mechanism now also clears the previously stored options
that the newly selected mechanism does not use, so stale credentials are
no longer kept in the saved configuration.

Signed-off-by: nfebe <fenn25.fn@gmail.com>
@susnux

susnux commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

/compile

@susnux susnux force-pushed the fix/files-external-edit-auth-mechanism branch from 2e66d5b to d5905ac Compare June 22, 2026 18:36
Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@susnux

susnux commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

/backport to stable34

@susnux

susnux commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

/backport to stable33

@susnux susnux enabled auto-merge June 22, 2026 20:47
@susnux susnux merged commit f8ad326 into master Jun 22, 2026
198 of 207 checks passed
@susnux susnux deleted the fix/files-external-edit-auth-mechanism branch June 22, 2026 20:47
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.

4 participants