Let's call the base class' c'tor first, eh?

[python_utils.git] / logging_utils.py

diff --git a/logging_utils.py b/logging_utils.py
index a16a31b43c6e88fd84c850cab851ce90d58049e2..8875b2fcb3d833f900c1107e7203abc27a49f54c 100644 (file)
--- a/logging_utils.py
+++ b/logging_utils.py
@@ -8,15 +8,15 @@ import datetime
 import enum
 import io
 import logging
-from logging.config import fileConfig
-from logging.handlers import RotatingFileHandler, SysLogHandler
 import os
 import random
 import sys
+from logging.config import fileConfig
+from logging.handlers import RotatingFileHandler, SysLogHandler
 from typing import Any, Callable, Dict, Iterable, List, Mapping, Optional
 
-from overrides import overrides
 import pytz
+from overrides import overrides
 
 # This module is commonly used by others in here and should avoid
 # taking any unnecessary dependencies back on them.
@@ -223,8 +223,8 @@ class SquelchRepeatedMessagesFilter(logging.Filter):
     """
 
     def __init__(self) -> None:
-        self.counters: collections.Counter = collections.Counter()
         super().__init__()
+        self.counters: collections.Counter = collections.Counter()
 
     @overrides
     def filter(self, record: logging.LogRecord) -> bool:
@@ -511,7 +511,7 @@ def initialize_logging(logger=None) -> logging.Logger:
             if config.config['logging_syslog_facility']:
                 facility_name = 'LOG_' + config.config['logging_syslog_facility']
             facility = SysLogHandler.__dict__.get(facility_name, SysLogHandler.LOG_USER)  # type: ignore
-            assert facility
+            assert facility is not None
             handler = SysLogHandler(facility=facility, address='/dev/log')
             handler.setFormatter(
                 MillisecondAwareFormatter(