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

[3.5] server/auth: fix auth panic bug when user changes password #15512

Conversation

engow
Copy link

@engow engow commented Mar 20, 2023

Automated cherry pick of #15432

@ahrtr ahrtr changed the title server/auth: fix auth panic bug when user changes password [3.5] server/auth: fix auth panic bug when user changes password Mar 20, 2023
@engow engow force-pushed the automated-cherry-pick-of-#15432-origin-release-3.5 branch from 0961f10 to 02f937e Compare March 20, 2023 03:37
@ahrtr
Copy link
Member

ahrtr commented Mar 23, 2023

Please signoff the commits, please read https://github.com/etcd-io/etcd/pull/15512/checks?check_run_id=12118295478

tangcong and others added 2 commits March 30, 2023 19:13
Signed-off-by: tangcong <tangcong506@foxmail.com>
Signed-off-by: engow <engow@hotmail.com>
Signed-off-by: engow <engow@hotmail.com>
@engow engow force-pushed the automated-cherry-pick-of-#15432-origin-release-3.5 branch from 02f937e to f7ac9df Compare March 30, 2023 11:13
@serathius
Copy link
Member

I think we can ignore Release tests failure, as it's not related to PR.
ping @ahrtr for review.

Copy link
Member

@ahrtr ahrtr left a comment

Choose a reason for hiding this comment

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

LGTM

Thanks @engow

@ahrtr ahrtr merged commit 5872b80 into etcd-io:release-3.5 Apr 7, 2023
11 of 12 checks passed
tjungblu pushed a commit to tjungblu/etcd that referenced this pull request Jul 26, 2023
…#15432-origin-release-3.5

[3.5] server/auth: fix auth panic bug when user changes password
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants