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

[console] Update console logging system #2341

Merged
merged 8 commits into from
Dec 17, 2022
Merged

[console] Update console logging system #2341

merged 8 commits into from
Dec 17, 2022

Conversation

Karlatemp
Copy link
Member

主修改

  • 优化 MiraiLogger (in console) 性能
  • SLF4J 支持

配置文件修改

修改均为默认配置, 先前已经生成的配置不会进行修改

Logger.yml

 # 默认日志输出等级
 # 可选值: ALL, VERBOSE, DEBUG, INFO, WARNING, ERROR, NONE
 defaultPriority: INFO
 # 特定日志记录器输出等级
 loggers: 
   example.logger: NONE
   console.debug: NONE
   Bot: ALL
+  org.eclipse.aether.internal: INFO
+  org.apache.http.wire: INFO

+# 是否启动外部日志框架桥接
+binding: 
+  slf4j: true

ABI Changes

注: 实际上 console 并没有直接的 abi changes

依赖更新: org.slf4j:slf4j-api:1.7.32 -> 2.0.3

此依赖的更新只会影响 slf4j-api 的对接, 并不会影响 slf4j-api 的单纯使用

即不会对插件有任何影响, 只会对部分对 console 进行高度自定义的会有少许影响

@Karlatemp Karlatemp added t:enhancement 类型: 现有功能上的优化 t:feature 类型: 新特性 s:console 子系统: mirai-console x:abi-change 属性: ABI 变更 labels Nov 10, 2022
@Karlatemp Karlatemp added this to the 2.14.0-RC milestone Nov 10, 2022
@Karlatemp Karlatemp force-pushed the console/logging branch 2 times, most recently from 745e9ae to 2db597e Compare November 11, 2022 05:37
mirai-core-all/build.gradle.kts Outdated Show resolved Hide resolved
@Him188 Him188 added the z:ready-to-merge 状态: PR 已经可以合并, 正在等待一些安排 label Dec 3, 2022
@Him188 Him188 merged commit 108ef31 into dev Dec 17, 2022
@Him188 Him188 deleted the console/logging branch December 17, 2022 20:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
s:console 子系统: mirai-console t:enhancement 类型: 现有功能上的优化 t:feature 类型: 新特性 x:abi-change 属性: ABI 变更 z:ready-to-merge 状态: PR 已经可以合并, 正在等待一些安排
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants