[Support] Logger更新到V3后的问题。

[minefujiko9293]

Describe the problem（描述问题）

问题1：【Bug】
V3的新的配置结构，error类的Transport，配置文件名貌似不生效，感觉有bug，像我命名为 "WW_ES-error.log" ，但生成的文件还是旧的名字 "common-error.log" 。不管是在default还是clients中都不生效。file类的Transport好像是正常的。同时日志的文档，虽然说了v3的版本，好像跟 https://github.com/midwayjs/logger/blob/main/BREAKING-3.md 里面的说明也不太同步。

同时，logger配置的代码类型提示好像也不太全，即使按照文档说的，在 src/interface.ts 中加入日志库的引用。
// src/interface.ts
import type {} from '@midwayjs/logger';
也不是很全

---

---

问题2：【优化】
Windows下，启动时控制台报错误。
[FileStreamRotator] Could not create symlink file: E:\Workspace\saclglxt\midway-saclglxt-ww\tmps\logs/common-error.log -> common-error.log.2023-12-06

虽然v3的文档中已经说明了【windows 下不会生成软链】，但这个错误太刺眼啦。

看看有没有能隐藏这个错误，毕竟每次保存代码都要刺眼下。

Midway Versions（Midway 版本）

Start to check your midway component version...

✓ @midwayjs/faas-typings(not installed)
✓ @midwayjs/fc-starter(not installed)
✓ @midwayjs/serverless-http-parser(not installed)
✓ @midwayjs/async-hooks-context-manager(3.13.5)
✓ @midwayjs/axios(not installed)
✓ @midwayjs/bootstrap(3.13.5)
✓ @midwayjs/bull(3.13.5)
✓ @midwayjs/bull-board(3.13.5)
✓ @midwayjs/cache(not installed)
✓ @midwayjs/captcha(not installed)
✓ @midwayjs/casbin(not installed)
✓ @midwayjs/casbin-redis-adapter(not installed)
✓ @midwayjs/casbin-typeorm-adapter(not installed)
✓ @midwayjs/code-dye(not installed)
✓ @midwayjs/consul(not installed)
✓ @midwayjs/core(3.13.5)
✓ @midwayjs/cos(not installed)
✓ @midwayjs/cron(not installed)
✓ @midwayjs/cross-domain(3.13.5)
✓ @midwayjs/decorator(3.13.5)
✓ @midwayjs/etcd(not installed)
✓ @midwayjs/express-session(not installed)
✓ @midwayjs/faas(not installed)
✓ @midwayjs/grpc(not installed)
✓ @midwayjs/http-proxy(not installed)
✓ @midwayjs/i18n(3.13.5)
✓ @midwayjs/info(3.13.5)
✓ @midwayjs/jwt(3.13.5)
✓ @midwayjs/kafka(not installed)
✓ @midwayjs/leoric(not installed)
✓ @midwayjs/mikro(not installed)
✓ @midwayjs/mock(3.13.5)
✓ @midwayjs/mongoose(not installed)
✓ @midwayjs/oss(not installed)
✓ @midwayjs/otel(not installed)
✓ @midwayjs/passport(not installed)
✓ @midwayjs/process-agent(not installed)
✓ @midwayjs/prometheus(not installed)
✓ @midwayjs/prometheus-socket-io(not installed)
✓ @midwayjs/rabbitmq(not installed)
✓ @midwayjs/redis(3.13.5)
✓ @midwayjs/security(not installed)
✓ @midwayjs/sequelize(not installed)
✓ @midwayjs/session(3.13.5)
✓ @midwayjs/socketio(not installed)
✓ @midwayjs/static-file(not installed)
✓ @midwayjs/swagger(not installed)
✓ @midwayjs/tablestore(not installed)
✓ @midwayjs/tags(not installed)
✓ @midwayjs/typegoose(not installed)
✓ @midwayjs/typeorm(3.13.5)
✓ @midwayjs/upload(not installed)
✓ @midwayjs/validate(3.13.5)
✓ @midwayjs/version(3.13.5)
✓ @midwayjs/view(not installed)
✓ @midwayjs/view-ejs(not installed)
✓ @midwayjs/view-nunjucks(not installed)
✓ @midwayjs/web(not installed)
✓ @midwayjs/express(not installed)
✓ @midwayjs/koa(3.13.5)
✓ @midwayjs/ws(not installed)

---

Check complete, all versions are healthy.

---

[minefujiko9293]

@czy88840616

[czy88840616]

windows 的处理的确是漏了

[minefujiko9293]

哥，问题1才是重点哈。error的日志名我改不了。怎么改都是‘common-error.log’。 新旧配置的写法写了也是不行

[minefujiko9293]

@czy88840616 刚刚再仔细测了下，
情况1：
写法：整个midwayLogger配置对象旧写法
结果：【正确】自定义的WW_ES_Logger的error的文件名为 WW_ES-error.log
用例：

midwayLogger: {
      default: {
        dir: path.join(appInfo.appDir, 'tmps', 'logs'),
        maxFiles: '1d',
      },
      clients: {
        bullLogger: {
          consoleLevel: 'warn',
        },
        WW_ES_Logger: {
          dir: path.join(appInfo.appDir, 'tmps', 'logs', 'WW_ES'),
          fileLogName: 'WW_ES.log',
          errorLogName: 'WW_ES-error.log',
          maxFiles: '90d',
        },
      },
    },

情况2：
写法：遵循v3新写法
结果：【不符合预期】自定义的WW_ES_Logger的error的文件名不为WW_ES-error.log
用例：

 midwayLogger: {
      default: {
        transports: {
          console: {},
          file: {
            dir: path.join(appInfo.appDir, 'tmps', 'logs'),
            maxFiles: '1d',
          },
          error: {
            dir: path.join(appInfo.appDir, 'tmps', 'logs'),
            maxFiles: '7d',
          },
        },
      },
      clients: {
        bullLogger: {
          level: 'warn',
        },
        WW_ES_Logger: {
          transports: {
            file: {
              dir: path.join(appInfo.appDir, 'tmps', 'logs', 'WW_ES'),
              maxFiles: '90d',
              fileLogName: 'WW_ES.log',
            },
            error: {
              dir: path.join(appInfo.appDir, 'tmps', 'logs', 'WW_ES'),
              maxFiles: '90d',
              fileLogName: 'WW_ES-error.log',
            },
          },
        },
      },
    },

[czy88840616]

我测试下。

[czy88840616]

确认了，的确是bug。

[czy88840616]

midwayjs/logger#109

[czy88840616]

已修复，试试 @midwayjs/[email protected]

[minefujiko9293]

目前更新后简单测试了下暂时没有问题，后续有空时候我再测试下复制的情况。
剩下的问题就是Logger_V3版在config中的代码提示了，即使引入了

// src/interface.ts
import type {} from '@midwayjs/logger';

写具体的transports时，代码提示就只剩如下属性提示

好像缺这缺那，得看文档写配置了。
对刚入手v3的小伙伴可能懵了。

[czy88840616]

@midwayjs/[email protected]，已恢复。

[minefujiko9293]

试了ok，但是前提是得执行

// src/interface.ts
import type {} from '@midwayjs/logger';

当然这个说明只在 https://github.com/midwayjs/logger/blob/main/BREAKING-3.md 中提及。
建议高亮加入midway主文档中。或者什么时候稳定到不需要引入该代码。

刚刚看了原来v3的logger刚发布不久，幸苦了哥！
我也是为了升级midway核心版本，把我logger也升级才发现这么些问题的哈。

[czy88840616]

对的，这个是必须的，因为 ts 的 typings merge 必须先让编辑器 import 一次。