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

壊れたconfig.jsonが保存されることがある #1636

Closed
Hiroshiba opened this issue Nov 3, 2023 · 0 comments · Fixed by #1629
Closed

壊れたconfig.jsonが保存されることがある #1636

Hiroshiba opened this issue Nov 3, 2023 · 0 comments · Fixed by #1629
Labels
バグ 優先度：中 初心者歓迎タスク 初心者にも優しい簡単めなタスク

Comments

@Hiroshiba
Copy link
Member

Hiroshiba commented Nov 3, 2023 •
edited
Loading

不具合の内容

最新のmainブランチでの報告です。
初回起動時、利用規約に同意するかどうかを問われてる画面で、何もせずにウィンドウを閉じたら壊れた設定ファイルが作成されました。

多分2つ同時に保存が走っているんだと思っています。

その他

今回のはレアケースですが、意図せずにこういうコードが書かれている場合もありそうなので、ConfigManager._saveあたりはMutex（async-lock）になっていた方が安全かも？
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
バグ 優先度：中 初心者歓迎タスク 初心者にも優しい簡単めなタスク
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Change: ブラウザ版のConfigをBaseConfigManagerベースに sevenc-nanashi/voicevox
1 participant
@Hiroshiba