Skip to content

Use cached user backend info for password login - #27876

Merged
skjnldsv merged 1 commit into
masterfrom
bugfix/noid/use-cached-user-backend-info-for-password-login
Oct 22, 2021
Merged

Use cached user backend info for password login#27876
skjnldsv merged 1 commit into
masterfrom
bugfix/noid/use-cached-user-backend-info-for-password-login

Conversation

@nickvergessen

Copy link
Copy Markdown
Member

Save 617 queries from our famous query log

@nickvergessen

Copy link
Copy Markdown
MemberAuthor

Ping, please review

@blizzz

Copy link
Copy Markdown
Member

CI is not so happy

@nickvergessen
nickvergessenforce-pushed the bugfix/noid/use-cached-user-backend-info-for-password-login branch from 77cacbf to fa96fb7CompareJuly 16, 2021 11:02
@nickvergessen

Copy link
Copy Markdown
MemberAuthor

Fixed

Comment threadlib/private/User/Manager.php Outdated
@szaimenszaimen added 2. developing Work in progress and removed 3. to review Waiting for reviews labels Aug 31, 2021
Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
nickvergessenforce-pushed the bugfix/noid/use-cached-user-backend-info-for-password-login branch from fa96fb7 to 1b8ebf2CompareSeptember 14, 2021 06:40
@nickvergessennickvergessen added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Sep 14, 2021
@nickvergessen

Copy link
Copy Markdown
MemberAuthor

Second review please?
@icewind1991@CarlSchwan@skjnldsv

@skjnldsvskjnldsv mentioned this pull request Oct 13, 2021
@skjnldsv
skjnldsv merged commit aef40c7 into masterOct 22, 2021
@skjnldsv
skjnldsv deleted the bugfix/noid/use-cached-user-backend-info-for-password-login branch October 22, 2021 09:37
@skjnldsvskjnldsv added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Oct 22, 2021
@skjnldsvskjnldsv mentioned this pull request Oct 25, 2021
25 tasks
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 finishbugfeature: users and groups

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@nickvergessen@blizzz@icewind1991@skjnldsv@szaimen