eryajf / go-ldap-admin

🌉 基于Go+Vue实现的openLDAP后台管理项目
http://ldapdoc.eryajf.net
Apache License 2.0
1.53k stars 259 forks source link

🐛 [Bug] 钉钉同步 员工部门信息在LDAP中是一个字符串 应该是一个列表 #267

Closed SoWhatI closed 10 months ago

SoWhatI commented 10 months ago

您使用的版本? | Your usage version?

openldap:1.4.0 phpldapadmin:0.9.0 go-ldap-admin-server:latest go-ldap-admin-ui:latest

您使用的场景? | Your usage scenarios?

钉钉同步

您做了什么操作? | What did you do?

钉钉同步

您遇到了什么问题? | What are your problems?

员工部门信息在LDAP中是一个字符串 image

您期望的结果是怎样的? | What is your expected outcome?

员工部门信息在LDAP中是一个列表 image

eryajf commented 10 months ago

您好 @SoWhatI👋,我已收到您的反馈,我将安排时间考虑您提交的信息并进行回复。-- 这条信息是由自动回复的机器人发出的。

Hello @SoWhatI. I have received your feedback, and I will arrange time to consider the information you submitted and reply. -- This message is sent by an automatic reply robot.

eryajf commented 10 months ago

不是很明白这个issue想要表达的意思,员工部门信息希望是一个列表,有没有具体的示例呢

SoWhatI commented 10 months ago

如下图,第一个是目前的,期望的是第二个 image

eryajf commented 10 months ago

或者你可以说下你的应用场景大概是什么

另外请浏览该项目的用户以及分组的设计思路:http://ldapdoc.eryajf.net/pages/5683c6/

其实已经写死了两种情况,并且应该是满足当下需求的。

SoWhatI commented 10 months ago

em,我阅读了下代码,大约是这里的问题 https://github.com/eryajf/go-ldap-admin/blob/30c9a71d438acf0e178728561c03458848676b9e/service/ildap/user_ildap.go#L22C1-L22C1add.Attribute("businessCategory", []string{user.Departments})修改为add.Attribute("businessCategory", strings.Split(user.Departments, ","))

SoWhatI commented 10 months ago

em,我阅读了下代码,大约是这里的问题 https://github.com/eryajf/go-ldap-admin/blob/30c9a71d438acf0e178728561c03458848676b9e/service/ildap/user_ildap.go#L22C1-L22C1user.Departments = strings.TrimRight(deptTmp, ",")修改为user.Departments = strings.Split(user.Departments, ",")

场景是钉钉同步后,一个开源系统在接入LDAP,在有多个部门的员工出问题了

eryajf commented 10 months ago

是这个开源的系统会基于部门进行规则过滤吗,还是什么

之前我验证过一个人归属多个部门,那就是归属多个呗,会有什么问题吗

SoWhatI commented 10 months ago

比如张三归属部门A和B,按照现有的存储,这个开源系统读取的是A,B,它就认为是A,B是一个部门