Josh Bleecher Snyder 788d0d0bad wgengine/wglog: improve wireguard-go logging rate limiting ... Prior to wireguard-go using printf-style logging, all wireguard-go logging occurred using format string "%s". We fixed that but continued to use %s when we rewrote peer identifiers into Tailscale style. This commit removes that %sl, which makes rate limiting work correctly. As a happy side-benefit, it should generate less garbage. Instead of replacing all wireguard-go peer identifiers that might occur anywhere in a fully formatted log string, assume that they only come from args. Check all args for things that look like *device.Peers and replace them with appropriately reformatted strings. There is a variety of ways that this could go wrong (unusual format verbs or modifiers, peer identifiers occurring as part of a larger printed object, future API changes), but none of them occur now, are likely to be added, or would be hard to work around if they did. Signed-off-by: Josh Bleecher Snyder <josharian@gmail.com>		2021-05-03 12:24:02 -07:00
..
wglog_test.go	wgengine/wglog: improve wireguard-go logging rate limiting	2021-05-03 12:24:02 -07:00
wglog.go	wgengine/wglog: improve wireguard-go logging rate limiting	2021-05-03 12:24:02 -07:00