andygrunwald / andygrunwald/things-with-buzzers-websocket

Replace stdlib "log" with structured logging "logrus"

オープン
#2 コメント 0 件 リアクション 0 件 担当者 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 を短くまとめたダイジェスト。