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.

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

[bug] ldap测试使用的数据是已保存的数据而不是表单中的数据 #447

Closed
wklken opened this issue May 17, 2022 · 0 comments
Closed

[bug] ldap测试使用的数据是已保存的数据而不是表单中的数据 #447

wklken opened this issue May 17, 2022 · 0 comments
Assignees
@wklken
Labels
Layer: api Api module related Module: LDAP/MAD ldap/mad related Priority: High Type: bug Something isn't working
Milestone
Y2022M20-21

Comments

@wklken
Copy link
Collaborator

wklken commented May 17, 2022

https://github.com/wklken/bk-user/blob/2107771e845f0bfe38828b0ab4cc7ec822e823f5/src/api/bkuser_core/categories/plugins/ldap/client.py#L32-L43

原因: __post_init__优先于initialize执行
@wklken wklken self-assigned this May 17, 2022
@wklken wklken added Type: bug Something isn't working Layer: api Api module related Module: LDAP/MAD ldap/mad related Priority: High labels May 17, 2022
@wklken wklken added this to the Y2022M20-21 milestone May 17, 2022
wklken added a commit to wklken/bk-user that referenced this issue May 17, 2022
@wklken
fix(api/ldap): test_connection use data in db, should use the data fr…
23f20e6 
…om form

close TencentBlueKing#447
@wklken wklken mentioned this issue May 17, 2022
Merged
@wklken wklken closed this as completed May 17, 2022
wklken added a commit that referenced this issue May 19, 2022
@wklken
fix(api/ldap): test_connection use data in db, should use the data fr…
a95035f 
…om form (#448)

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

@wklken wklken

Labels
Layer: api Api module related Module: LDAP/MAD ldap/mad related Priority: High Type: bug Something isn't working
Projects
None yet
Milestone
Y2022M20-21
Development

No branches or pull requests
1 participant
@wklken