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

改进 Windows 端第三方同步盘检测 #7777

Closed
88250 opened this issue Mar 25, 2023 · 13 comments
Closed

改进 Windows 端第三方同步盘检测 #7777

88250 opened this issue Mar 25, 2023 · 13 comments
Assignees
Milestone

Comments

@88250
Copy link
Member

88250 commented Mar 25, 2023

查询 工作空间/data/ 属性,如果返回不为空的话说明该文件在同步盘路径下。

@88250 88250 added this to the 2.8.1 milestone Mar 25, 2023
@88250 88250 self-assigned this Mar 25, 2023
@Soltus
Copy link
Contributor

Soltus commented Mar 25, 2023

希望可以解决误杀问题

@88250
Copy link
Member Author

88250 commented Mar 25, 2023

希望可以解决误杀问题

那个应该已经解决了 #7744

@Soltus
Copy link
Contributor

Soltus commented Mar 25, 2023

希望可以解决误杀问题

那个应该已经解决了 #7744

没有

@88250
Copy link
Member Author

88250 commented Mar 25, 2023

请描述一下问题和重现步骤。

@Soltus
Copy link
Contributor

Soltus commented Mar 25, 2023

请描述一下问题和重现步骤。

日志记录:

filelock.go:135: write file 某个文件.tmp Access is denied.

@88250
Copy link
Member Author

88250 commented Mar 25, 2023

估计是有其他软件正在访问文件导致的,检查一下吧。

@Soltus
Copy link
Contributor

Soltus commented Mar 25, 2023

出现类似文件写入操作错误应该是无法预测和完全避免的。我觉得文件系统读写错误不应该作为内核退出的依据,希望可以通过其他方式避免文件冲突,或者用更准确和友好的判断机制告知用户。

@88250
Copy link
Member Author

88250 commented Mar 25, 2023

这个是数据安全的基础,一旦文件写入报错就退出内核,没有补救方案的。

@Soltus
Copy link
Contributor

Soltus commented Mar 25, 2023

用户数据可以理解,recent-doc.json 这种配置文件就没有必要了吧

@88250
Copy link
Member Author

88250 commented Mar 25, 2023

这个也是用户数据的,记录了最近打开的文档。

@88250
Copy link
Member Author

88250 commented Mar 25, 2023

其他配置数据同样重要,整个文件系统必须可用,否则报错以后无法排查问题,也无法恢复数据。

@Zuoqiu-Yingyi
Copy link
Contributor

Zuoqiu-Yingyi commented Mar 25, 2023

强烈建议不要使用该方案, 这个调用行为很敏感...

@88250 88250 reopened this Mar 26, 2023
@88250
Copy link
Member Author

88250 commented Mar 26, 2023

强烈建议不要使用该方案, 这个调用行为很敏感...

稍后将 PowerShell 实现更换为 https://github.com/go-ole/go-ole

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants