Skip to content
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.

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

[v9.5.x] LDAP: FIX Enable users on successfull login #75192

Merged
merged 1 commit into from Sep 28, 2023

Conversation

gamab
Copy link
Contributor

@gamab gamab commented Sep 20, 2023

Backport c8149d5 from #75073


What is this feature?

In previous versions, before the new auth broker service was introduced, we used to enable users upon successful ldap login.
We had a regression when switching to the auth broker service.
This PR intends to fix this issue.

Why do we need this feature?

[Add a description of the problem the feature is trying to solve.]

Who is this feature for?

[Add information on what kind of user the feature is for.]

Which issue(s) does this PR fix?:

Fixes #

Special notes for your reviewer:

Please check that:

  • It works as expected from a user's perspective.
  • If this is a pre-GA feature, it is behind a feature toggle.
  • The docs are updated, and if this is a notable improvement, it's added to our What's New doc.

@gamab gamab added this to the 9.5.x milestone Sep 20, 2023
@gamab gamab requested a review from a team as a code owner September 20, 2023 19:17
@gamab gamab requested review from mgyongyosi, danielkenlee, linoman and Jguer and removed request for a team September 20, 2023 19:17
@gamab gamab reopened this Sep 28, 2023
@gamab gamab merged commit 960f9c1 into v9.5.x Sep 28, 2023
12 checks passed
@gamab gamab deleted the backport-75073-to-v9.5.x branch September 28, 2023 07:43
@aangelisc aangelisc modified the milestones: 9.5.x, 9.5.14 Nov 13, 2023
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.

None yet

3 participants