Skip to content

Conversation

@juliusknorr
Copy link
Member

This is a workaround until #5221 gets implemented, so that at least we stick to configured LDAP settings if the email is not part of the login filter

@juliusknorr juliusknorr requested review from blizzz and come-nc October 25, 2022 13:39
@juliusknorr juliusknorr added bug 3. to review Waiting for reviews labels Oct 25, 2022
@juliusknorr juliusknorr force-pushed the bugfix/noid/ldap-login-filter-no-mail branch from ef0da98 to 298d2b9 Compare October 26, 2022 10:30
@juliusknorr juliusknorr requested review from a team, ArtificialOwl and PVince81 and removed request for a team October 26, 2022 11:18
Copy link
Member

@PVince81 PVince81 left a comment

Choose a reason for hiding this comment

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

👍

@PVince81 PVince81 merged commit 5198fa4 into master Oct 26, 2022
@PVince81 PVince81 deleted the bugfix/noid/ldap-login-filter-no-mail branch October 26, 2022 12:12
@juliusknorr
Copy link
Member Author

/backport to stable25

@juliusknorr
Copy link
Member Author

/backport to stable24

@juliusknorr
Copy link
Member Author

/backport to stable23

@backportbot-nextcloud
Copy link

The backport to stable25 failed. Please do this backport manually.

@backportbot-nextcloud
Copy link

The backport to stable24 failed. Please do this backport manually.

@backportbot-nextcloud
Copy link

The backport to stable23 failed. Please do this backport manually.

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

Labels

3. to review Waiting for reviews bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants