Make filter_by_level processor work with std lib logs

[ZlatyChlapec]

When using filter_by_level processor with ProcessorFormatter it fails on std lib logs because of

structlog/src/structlog/stdlib.py

Line 333 in c0aeab5

if logger.isEnabledFor(_NAME_TO_LEVEL[name]):

being called on None.

[hynek]

Also pls changelog.

[hynek]

Is there a better way to achieve this? It looks expensive to call getLogger on each log entry.

[ZlatyChlapec]

I've got three ideas in my mind:

1. We could make it so that users would have to manually set logger to ProcessorFormatter and then make ProcessorFormatter to take value from there.

logger = logging.getLogger()
formatter = ProcessorFormatter(JSONRenderer())

handler = StreamHandler(sys.stdout)
handler.setFormatter(formatter)
handler.formatter.logger = logger
# Or via setter which we would create handler.formatter.set_logger(logger)

logger.addHandler(handler)

2. Make users patch logging.Logger.makeRecord so that it includes self as _std_logger.

3. Create some kind of first use only cache (probably some dict with key name of logger and value logger itself) which would look up logger only first time, store it into cache and take it from there next time.

Do you have anything better in mind?

[rasto2211]

I'd prefer 1). ProcessorFormatter could take the logger as a constructor argument.

[rasto2211]

Is anyone working on this issue?

[hynek]

I'm afraid I dropped the ball here unfortunately. You still game @ZlatyChlapec ?

[hynek]

fixed in #219, thanks anyway and sorry for my failure to respond in adequate time!