Skip to content

chore: remove unused OC_User::post_login hook - #65396

Open
susnux wants to merge 1 commit into
masterfrom
chore/remove-legacy-hook
Open

chore: remove unused OC_User::post_login hook#65396
susnux wants to merge 1 commit into
masterfrom
chore/remove-legacy-hook

Conversation

@susnux

@susnux susnux commented Oct 11, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Summary

Hooks are deprecated for > 10 versions, but also this specific hook is not listened anymore by any of our apps.
Alternatives are available for many years:

  • OCP\User\Events\UserLoggedInEvent
  • OCP\User\Events\UserLoggedInWithCookieEvent

Checklist

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

@susnux
chore: remove unused OC_User::post_login hook
5d4c67f 
Hooks are deprecated for > 10 versions, but also this specific hook
is not listened anymore by any of our apps.
Alternatives are available for many years:

- `OCP\User\Events\UserLoggedInEvent`
- `OCP\User\Events\UserLoggedInWithCookieEvent`

Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@susnux
susnux requested a review from a team as a code owner October 11, 2026 12:23
@susnux
susnux requested review from icewind1991, leftybournes, provokateurin and salmart-dev and removed request for a team October 11, 2026 12:23
@susnux susnux added 3. to review Waiting for reviews ♻️ refactor Refactor code (not a bug fix, not a feature just refactoring) labels Oct 11, 2026
@susnux susnux added this to the Nextcloud 36 milestone Oct 11, 2026
@susnux susnux added the pending documentation This pull request needs an associated documentation update label Oct 11, 2026

This branch has not been deployed

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

Reviewers

@icewind1991 icewind1991 Awaiting requested review from icewind1991 icewind1991 is a code owner automatically assigned from nextcloud/server-backend
@salmart-dev salmart-dev Awaiting requested review from salmart-dev salmart-dev is a code owner automatically assigned from nextcloud/server-backend
@leftybournes leftybournes Awaiting requested review from leftybournes leftybournes is a code owner automatically assigned from nextcloud/server-backend
@provokateurin provokateurin Awaiting requested review from provokateurin provokateurin is a code owner automatically assigned from nextcloud/server-backend

At least 2 approving reviews are required to merge this pull request.

Assignees

No one assigned

Labels

3. to review Waiting for reviews pending documentation This pull request needs an associated documentation update ♻️ refactor Refactor code (not a bug fix, not a feature just refactoring)

Projects

None yet

Milestone

Nextcloud 36

Development

Successfully merging this pull request may close these issues.

1 participant

@susnux