reddit/baseplate.py

Ensure all integer fields in JSON logging are encoded as strings

Closed this issue · 1 comments

Issue experienced in Baseplate.go with our logging vendor: uber-go/zap#884

It looks like of the standard stuff we'll only be emitting line numbers, thread IDs, and PIDs as integers right now? https://github.com/reddit/baseplate.py/blob/develop/baseplate/server/__init__.py#L123

The only number here that seems to get large enough to matter is the thread ID.