egg-logger 日期格式化增加时区信息 #5145

Activity

1. fengmk2 commented on Feb 14, 2023

   

   fengmk2

   on Feb 14, 2023

   Member

   More actions

   你是怎样拿到这个数据的？
2. sjfkai commented on Feb 15, 2023

   

   sjfkai

   on Feb 15, 2023

   ContributorAuthor

   More actions

   把日志配置改成JSON config.default.js:

   logger: {
       outputJSON: true,
   }

   日志输出 egg-web.json.log：

   {"level":"WARN","date":"2023-02-15 09:15:30,242","pid":88174,"hostname":"xxx","message":"[ClusterClient:Watcher] follower closed, and try to init it again"}

   看了一下 egg-logger 源码:

   meta.date = utility.logDate(',') 类似 moment().format('YYYY-MM-DD HH:mm:ss.SSS')
3. fengmk2 commented on Feb 15, 2023

   

   fengmk2

   on Feb 15, 2023

   Member

   More actions

   嗯，这里确实是一个问题，默认的是当前服务器所在时区的 local 时间。可以新增一个选型，输出 Date 标准的 JSON 格式。
4. fengmk2 commented on Feb 15, 2023

   

   fengmk2

   on Feb 15, 2023

   Member

   More actions

   @sjfkai 来尝试pr实现？
5. fengmk2

   added

   deps: egg-logger

   type: feature

   on Feb 15, 2023
6. sjfkai commented on Mar 1, 2023

   

   sjfkai

   on Mar 1, 2023

   ContributorAuthor

   More actions

   @sjfkai 来尝试pr实现？

   @fengmk2 其实跟 JSON 的关系不大，默认输出的 Data 也有相同问题。
   新的选项如何命名有什么建议吗？ dateISOFormat: true ?
   实现上就用 Date.prototype.toISOString() 替换当前的 utility.logDate ?
7. fengmk2 commented on Mar 1, 2023

   

   fengmk2

   on Mar 1, 2023

   Member

   More actions

   @sjfkai 来尝试pr实现？

   @fengmk2 其实跟 JSON 的关系不大，默认输出的 Data 也有相同问题。 新的选项如何命名有什么建议吗？ dateISOFormat: true ? 实现上就用 Date.prototype.toISOString() 替换当前的 utility.logDate ?

   可以的。
8. fengmk2

   closed this as completedin egg-logger#88on Mar 3, 2023
9. fengmk2

   added a commit that references this issue on Mar 3, 2023

   feat: add dateISOFormat option (#88)

   ...

   14919d8