andygrunwald / andygrunwald/things-with-buzzers-websocket

Replace stdlib "log" with structured logging "logrus"

未关闭
#2 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看
enhancement
主要语言
Go
星标
10
派生
2
PR 合并指标
30 天内没有已合并 PR

描述

Right now we use the stdlib "log" package for logging:

```
➜ things-with-buzzers-websocket git:(master) ./twb-websocket
2019/09/29 13:15:38 ******************************************
2019/09/29 13:15:38 things with buzzers: websocket
2019/09/29 13:15:38 ******************************************
2019/09/29 13:15:38 Software buzzer requested
2019/09/29 13:15:38 Starting socket broadcast
2019/09/29 13:15:38 Webserver starting on :8080
2019/09/29 13:15:38 Buzzer emulator: tcp socket starting on :8181
2019/09/29 13:16:27 New client registered: [::1]:8080
2019/09/29 13:16:45 Buzzer emulator: client connected: [::1]:53104
2019/09/29 13:16:49 Buzzer emulator: message received from client [::1]:53104: /yellow
2019/09/29 13:16:49 Buzzer pressed: yellow
2019/09/29 13:16:49 Buzzer emulator: message sent to client [::1]:53104: OK
2019/09/29 13:16:49 Broadcasting message: {"Color":"yellow"}
2019/09/29 13:16:54 Buzzer emulator: message received from client [::1]:53104: /yellow
[...]
```

It would be cool if we could switch to [logrus](https://github.com/sirupsen/logrus).
This enables a colorful output and structured logging based on fields.

For the software buzzer emulator this would work fine because there, we have the logging 100% under our control.
When we enable the hardware buzzers, [gobot](https://github.com/hybridgroup/gobot) also starts to log with the stdlib log package.

Mixing both would end up in an ugly and unreadable logging state. There are smaller ambitions to switch to logrus inside gobot. See https://github.com/hybridgroup/gobot/issues/508

贡献指南

这个仓库没有索引到贡献指南

评估

这个 Issue 还没有评估数据。

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。