slack-go / slack

Slack API in Go, originally by @nlopes; Maintainers needed, contact @parsley42
https://pkg.go.dev/github.com/slack-go/slack
BSD 2-Clause "Simplified" License
4.63k stars 1.12k forks source link

socketmode remove duplicate debug logging #1165

Closed kittydoor closed 1 year ago

kittydoor commented 1 year ago

smc.runRequestHandler and smc.runMessageReceiver are both run as goroutines.

Respectively, when debug is enabled, they log the event:

One of these should be removed. This PR removes the latter as it re-marshalls the json, and pretty prints (and was the offender that made me investigate this in the first place).

I'm open to other ideas too, as long as it means the end result is no pretty-printed log line, and no duplicate logging of the same data in full.