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

[前端] ldap地址格式校验? #436

Closed
wklken opened this issue May 13, 2022 · 1 comment
Closed

[前端] ldap地址格式校验? #436

wklken opened this issue May 13, 2022 · 1 comment
Assignees
Labels
Layer: frontend Front pages related Module: LDAP/MAD ldap/mad related Type: enhancement Enhancement for existing feature
Milestone

Comments

@wklken
Copy link
Collaborator

wklken commented May 13, 2022

image

需要表单做一个正则校验? 非法不允许提交表单, 并提示错误

连接地址

支持:

  1. 域名: www.aaa.com
  2. 域名+端口号 www.aaa.com:389
  3. IP+端口号 127.0.0.1:389
  4. 不使用ssl, ldap://x.x.x.x:389 (x.x.x.x为合法域名或 IP, 端口需要根据自己环境服务提供端口修改)
  5. 使用ssl, ldaps://x.x.x.x:636 (x.x.x.x为合法域名或 IP,端口需要根据自己环境服务提供端口修改)

SSL加密方式

  • 如果连接地址填写的是ldaps开头的, 这里必须选择ssl
  • 如果连接地址填写的是其他情况, 这里必须选择

其他

这里推荐使用 ldap://ldaps://的方式来填写连接地址

默认框里面的placeholder使用示例 ldap://127.0.0.1:389 ldap://127.0.0.1:636 ?

@wklken wklken added Type: enhancement Enhancement for existing feature Layer: frontend Front pages related Module: LDAP/MAD ldap/mad related Priority: High labels May 13, 2022
@wklken wklken self-assigned this May 13, 2022
@wklken wklken added this to the Y2022M20-21 milestone May 16, 2022
yuri0528 pushed a commit to yuri0528/bk-user that referenced this issue May 20, 2022
EmilyMei pushed a commit that referenced this issue May 20, 2022
@wklken
Copy link
Collaborator Author

wklken commented May 23, 2022

提示优化:

  • 请选择正确的加密方式, ldap://必须选择, ldaps://必须选择SSL`
  • 请输入合法的ldap地址, 例如ldap://127.0.0.1:389ldaps://127.0.0.1:636`

问题:

  • 只输入域名www.aaa.com 或 只输入127.0.0.1/127.0.0.1:389目前表单都会提示: 请输入合法的ldap地址

yuri0528 pushed a commit to yuri0528/bk-user that referenced this issue May 23, 2022
EmilyMei pushed a commit that referenced this issue May 26, 2022
@wklken wklken closed this as completed May 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Layer: frontend Front pages related Module: LDAP/MAD ldap/mad related Type: enhancement Enhancement for existing feature
Projects
None yet
Development

No branches or pull requests

2 participants