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

Fix: buggy config #158

Merged
merged 2 commits into from
Sep 10, 2020
Merged

Fix: buggy config #158

merged 2 commits into from
Sep 10, 2020

Conversation

HoshinoTented
Copy link
Collaborator

在不存在 authKey 字段的时候,mah 将会用随机的 authKey 代替,并且写入配置文件,下次启动时将会沿用上次随机的 authKey,这可能有点问题。

@HoshinoTented HoshinoTented marked this pull request as draft September 10, 2020 05:10
@HoshinoTented HoshinoTented marked this pull request as ready for review September 10, 2020 06:40
@ryoii
Copy link
Collaborator

ryoii commented Sep 10, 2020

AuthKey不应该回写,就让它保留只读配置就好了

@HoshinoTented
Copy link
Collaborator Author

但是就我目前所了解,by value(default) 是会把 default 写入数据结构的

@ryoii
Copy link
Collaborator

ryoii commented Sep 10, 2020

那你改好了吗

@HoshinoTented
Copy link
Collaborator Author

改什么

@HoshinoTented HoshinoTented changed the title Fix: mah will initialize authKey at random Fix: buggy config Sep 10, 2020
@ryoii ryoii merged commit 74ce843 into project-mirai:master Sep 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants