Skip to content

fix ldap update user #128

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Nov 29, 2022
Merged

fix ldap update user #128

merged 1 commit into from
Nov 29, 2022

Conversation

Foustdg
Copy link
Contributor

@Foustdg Foustdg commented Nov 29, 2022

修复bug 修改用户的时候会在LDAP中误把cn值存为 Nickname，正确值应为 Username

在提出此拉取请求时，我确认了以下几点（请复选框）：

  • 我已阅读并理解贡献者指南。
  • 我已检查没有与此请求重复的拉取请求。
  • 我已经考虑过，并确认这份呈件对其他人很有价值。
  • 我接受此提交可能不会被使用，并根据维护人员的意愿关闭拉取请求。

填写PR内容：

  • 修复bug：修改用户的时候会在LDAP中误把cn值存为 Nickname，正确值应为 Username

@Foustdg
Update user_ildap.go
3281375 
修复bug 修改用户的时候会在LDAP中误把cn值存为 Nickname，正确值应为 Username
@eryajf eryajf changed the title Update user_ildap.go fix ldap update user Nov 29, 2022
@eryajf eryajf merged commit 3cea73d into opsre:main Nov 29, 2022
@eryajf eryajf mentioned this pull request Nov 29, 2022
Closed
@eryajf eryajf added the fix bug修复 | bug fix label Feb 8, 2023
yunkaiOr2 pushed a commit to yunkaiOr2/go-ldap-admin that referenced this pull request Jan 12, 2025
@Foustdg
fix ldap update user (opsre#128)
4529b73
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
fix bug修复 | bug fix
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Foustdg @eryajf