-
-
Notifications
You must be signed in to change notification settings - Fork 2k
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
增加编辑保护, 如果设置,需要密码才能编辑 #7526
The head ref may contain hidden characters: "dev-\u7F16\u8F91\u4FDD\u62A4"
增加编辑保护, 如果设置,需要密码才能编辑 #7526
Conversation
使用伺服测试,存在一个逻辑问题:开启编辑保护后,app端切换只读/编辑模式,伺服端只读/编辑图标不会切换,但实际模式已经发生变化。 |
从功能上setReadonly2算是setReadonly的包装,这个命名太模糊了,建议改成setReadonlyCheck |
app和网页的只读/编辑模式是独立的, app设置密码后,模式不变, 但是网页打开思源后会自动进入只读模式,需要密码才能编辑 ; ps: 网页端也可以设置密码, 然后app打开后自动进入只读 |
实测并没有独立,任意端切换模式都会立即影响另一端 |
@Soltus |
彼此影响的后果是冲突,如果需要控制伺服端也不应当通过app端切换编辑模式这一方式。如果无法解决这一冲突,你应该使用 if browser 限定在浏览器中使用锁定,app端只配置密码而不锁定 |
docker+NAS时, 管理者也是通过网页访问思源的吧, 这个编辑保护也可以从网页端来设置, 从而影响其他网页 |
网页端模式切换应当仅作用于当前会话。
|
从使用场景上来看,只是改动文档编辑器的编辑模式无法解决问题的,因为这里只控制编辑器,不控制其他地方,浏览者依然可以创建、删除文档或者进行其他一些写入数据的操作。 要正确实现该场景目前有两个方案:
方案 1 目前已经能够工作,方案 2 需要较大工作量的开发。 感谢楼主花时间精力贡献,但是我们不能合并这个 PR,见谅。 |
测试了一下,确实,编辑器只读模式只能限制编辑区,对于删除文档这种破坏操作也是限制不了的……那这个功能虽然还有价值,但意义不大了。 |
@88250 在前端对一些会修改数据的操作进行屏蔽不就可以只读了吗? 为啥还需要单独开一个内核? 方案1需要联网,这个缺点有点大.. |
@88250 另外方案1能实现 管理者编辑文档的同时, 浏览者能实时看到修改的内容吗? |
后续可能在 局域网分享文档 #261 实现 |
@iamqiz 那就是方案二了,需要单独管理这些权限;无法实时同步。 |
@88250 我感觉把一些常用的能修改数据的操作加上权限,只读功能基本就能发布给用户使用了;对于剩下那些不常用的,后面慢慢再加上 |
能否增加一个只读端口呢? |
支持设置编辑保护密码,
如果设置了密码,
密码保存在本地, 工作空间目录\data\storage\local.json 文件里,名为editProtectPassword
使用场景
使用方法:
