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.

Sign up for GitHub

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

Jump to bottom

NEO_1.4.2存在无意中覆盖用户配置文件到默认配置的情况 #75

Closed
RichardGuyNotFavMHY opened this issue Aug 10, 2023 · 4 comments · Fixed by #77
Closed

NEO_1.4.2存在无意中覆盖用户配置文件到默认配置的情况 #75

RichardGuyNotFavMHY opened this issue Aug 10, 2023 · 4 comments · Fixed by #77
Labels
fix Something isn't working

Comments

@RichardGuyNotFavMHY
Copy link

现象：NEO_1.4.2存在无意中覆盖用户配置文件到默认配置的情况。
推断：在config.ini被外部编辑、占用或者ngapost2md.exe存在多个实例同时调用config.ini时，NEO_1.4.2有可能会直接将用户的有效配置文件覆写为空白的配置文件。
复现方法：模拟config.ini暂时不可用的场景(例如外部编辑或同时调用)，随后使用一个脚本来观察文件是否有更改。
本地复现结果：
图片
使用的脚本：
filetest.zip
@ludoux ludoux added the fix Something isn't working label Aug 10, 2023
@ludoux
Copy link
Owner

ludoux commented Aug 10, 2023

这个计划下一个版本优化一下配置文件读取，#72 ，在版本号相异的情况下才更新保存配置文件，希望可以解决此文件

@ludoux
Copy link
Owner

ludoux commented Aug 10, 2023 •
edited
Loading

app-amd64.zip
你好，方便的时候可以用这个测试一下么？由 9b90155 编译而来，不知道能不能解决此问题。谢谢

@RichardGuyNotFavMHY
Copy link
Author

RichardGuyNotFavMHY commented Aug 10, 2023 •
edited
Loading

可行，但是需要等待一会。服务器down了联系不上机房。预计10个小时后有答复。
测试已经开始。

@RichardGuyNotFavMHY
Copy link
Author

目前不会替换文件了，导出默认配置文件的功能也正常。

@ludoux ludoux mentioned this issue Aug 11, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
fix Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

仅在配置文件版本号相异时覆盖本地配置文件 ludoux/ngapost2md
2 participants
@ludoux @RichardGuyNotFavMHY