-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
增加系统通用字符串脱敏规则-三段式通用脱敏规则、脱敏字段支持* #2698
Conversation
1cf19f6
to
089b392
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #2698 +/- ##
==========================================
+ Coverage 77.39% 77.43% +0.04%
==========================================
Files 119 119
Lines 16330 16359 +29
==========================================
+ Hits 12639 12668 +29
Misses 3691 3691 ☔ View full report in Codecov by Sentry. |
不好意思,我没太理解这里的通用脱敏规则的含义,能否帮忙再详细说明一下使用方法? 方便的话写到代码中去一下。 |
能否将名字改为"三段式通用脱敏规则", 这样更加好理解一些, 你觉得如何? |
|
不用加01,你预留的槽位可以再加 01,02 一直到10 |
可以考虑适配下数字类型? |
懵了。“model里预留几个槽位” 这句话不明白意思了。get不到点。 如果放开正则,正则配置是在“脱敏规则配置” 菜单里面配置的。 |
数字类型,没有具体的需求哎。 可能需求:把数字 乘以一个随机数? 现在的代码把数字转为str类型,去脱敏了。 |
不好意思, 我才关注到可以使用 脱敏规则设置和 脱敏字段配置 两个配置来结合, 配置脱敏规则, 那这样的话, 你当前的这个需求, 是不是可以直接用
我建议本次 pr 先加这个功能, 然后你用 "其他" 这个规则进行关联. 如果你想扩展这个功能, 比如说可以设置无限个脱敏规则, 可以扩展 其中的 对于数字类型的字段, 我比较建议转换为文本然后再做实现. |
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
经过沟通确实了解了这个功能的实际用途, 请将规则的名字改为 "三段式通用脱敏规则", 其他 LGTM
@LeoQuote 已修复,麻烦再看一下。2024-7-31 |
rebase下到最新版,有冲突 |
已rebase。为啥我本地没有显示冲突?好奇怪。 |
--------- Co-authored-by: 王飞 <fei.wang@xgo.one>
需求背景:
增加功能:
使用建议:
在配置脱敏字段时,只需要找出系统中phone,moblie,passowd等敏感的的字段名,向data_masking_columns表插入数据,不用再关心库名和表名。
不同长度的敏感字段脱敏效果: