-
Notifications
You must be signed in to change notification settings - Fork 16
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
Unify the setting place of path where event log is saved. #126
Comments
It is difficult to fix where you are reading logback.xml.
The related sources are as follows.
|
The logback.xml setting value variable has been modified to refer to personium-unit-config.properties. |
How about making https://github.com/personium/personium-core/blob/v1.8.1/src/main/resources/logback.xml#L62 before <file>/personium_nfs/personium-core/eventlog/${eventlog_path}/current/default.log</file> after <file>${eventlog_path}/current/default.log</file> |
Current specifications.
eventログの書き込みと読み込みでそれぞれ異なる設定箇所のPathを参照している。
今はデフォルトで同一の設定になっているので問題ないが、ユーザがどちらかを修正すると正常に動作しなくなる。
修正:https://github.com/KourokiTakuto/personium-core/commit/1b2aa0735b05da3fbc7243f00f1d860afe1821c7 でcoreとしての修正は完了しているが、logback.xmlの修正によりansibleにも影響が出る。同時にリリースが必要。
The text was updated successfully, but these errors were encountered: