🐛 邮箱校验问题导致用户添加失败 | [Bug]

[zhangzhen-0104]

版本：0.5.18
场景：同步钉钉人员将和组织架构
操作：同步钉钉人员出错
问题：同步钉钉人员时，有的员工姓名带（）导致邮箱添加失败
期望结果：判断邮箱是否有特殊符号，如有特殊符号则填入默认邮箱比如：123456@dingtailk.com。

[eryajf]

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

Hello @zhangzhen-0104. 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]

同步钉钉人员时，有的员工姓名带（）导致邮箱添加失败

这个我晚点看看修复一下

[zhangzhen-0104]

谢谢，我已经改好了但是不会打包🤦‍♂️
在dingtalk_logic.go文件
141行
// 检查邮箱格式是否有效
func isValidEmail(email string) bool {
// 正则表达式验证邮箱格式
re := regexp.MustCompile(^[a-zA-Z0-9._%+-]+@[a-zA-Z0-9.-]+\.[a-zA-Z]{2,}$)
return re.MatchString(email)
}

150行
// 默认邮箱
defaultEmail := "default@example.com"

// 检查邮箱是否有效，如果无效就使用默认邮箱
if user.Mail == "" || !isValidEmail(user.Mail) {
	user.Mail = defaultEmail
}

[eryajf]

可以提交pr，action 会自动打包。
或者你参考Dockerfile，手动打包也可以。

[eryajf]

姓名带()和邮箱本身应该没有关系吧，还是说他的邮箱是空的？可否给个例子。

[zhangzhen-0104]

邮箱是空的，默认是名字拼音+config.yml里的邮箱后缀，如 haosheng(test)@dingtalk.com；就导致ldap邮箱验证过不去

[eryajf]

get。等我随后验证下这个场景。