Skip to content

fix(encryption): Listen for user login and logout to set encryption key #51310

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
susnux merged 1 commit into from
Mar 6, 2025
Merged

fix(encryption): Listen for user login and logout to set encryption key #51310

susnux merged 1 commit into from
Mar 6, 2025

Conversation

@susnux
Copy link
Contributor

@susnux susnux commented Mar 6, 2025 •
edited
Loading

Summary

Missing in #48332 so caused a regression.
The event listener has the correct methods but I forgot to register the events 🙈

Checklist

@susnux susnux added this to the Nextcloud 32 milestone Mar 6, 2025
@susnux susnux marked this pull request as ready for review March 6, 2025 16:40
@susnux susnux requested a review from a team as a code owner March 6, 2025 16:40
@susnux susnux requested review from Altahrim, nfebe and skjnldsv and removed request for a team March 6, 2025 16:40
@susnux
Copy link
Contributor Author

susnux commented Mar 6, 2025

/backport to stable31

nfebe
nfebe approved these changes Mar 6, 2025
View reviewed changes
Copy link
Contributor

@nfebe nfebe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

artonge
artonge approved these changes Mar 6, 2025
View reviewed changes
Copy link
Contributor

@artonge artonge left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🫂

@susnux susnux merged commit 561397c into master Mar 6, 2025
194 checks passed
@susnux susnux deleted the fix/encryption-events branch March 6, 2025 17:52
@backportbot backportbot bot mentioned this pull request Mar 6, 2025
Merged
@SystemKeeper SystemKeeper mentioned this pull request Mar 24, 2025
Closed
8 tasks
@nextcloud-bot nextcloud-bot mentioned this pull request Aug 19, 2025
Merged
@skjnldsv skjnldsv modified the milestones: Nextcloud 32, Nextcloud 33 Sep 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@AndyScherzinger AndyScherzinger AndyScherzinger approved these changes

@artonge artonge artonge approved these changes

@nfebe nfebe nfebe approved these changes

@come-nc come-nc come-nc approved these changes

@Altahrim Altahrim Awaiting requested review from Altahrim Altahrim is a code owner automatically assigned from nextcloud/server-backend

@skjnldsv skjnldsv Awaiting requested review from skjnldsv skjnldsv is a code owner automatically assigned from nextcloud/server-backend

Assignees

No one assigned

Labels

3. to review Waiting for reviews bug feature: encryption (server-side) regression

Projects

None yet

Milestone

Nextcloud 32

Development

Successfully merging this pull request may close these issues.

[Bug]: Encryption app is enabled, but your keys are not initialized. Please log-out and log-in again. (upgrade 30.0.6 to 31)

7 participants

@susnux @AndyScherzinger @artonge @nfebe @come-nc @skjnldsv