Updates logger codebase-wise to match the Logger updates

Logger instances now specify the logger prefix so the logger can be properly loaded, fixing the issues with the cli file logger

pisa/api.py

@@ -1,8 +1,9 @@
 import os
 import json
+import logging
 from flask import Flask, request, abort, jsonify
 
-from pisa import HOST, PORT, logging
+from pisa import HOST, PORT, LOG_PREFIX
 from common.logger import Logger
 from pisa.inspector import Inspector
 from common.appointment import Appointment
@@ -13,7 +14,7 @@ from common.constants import HTTP_OK, HTTP_BAD_REQUEST, HTTP_SERVICE_UNAVAILABLE
 
 # ToDo: #5-add-async-to-api
 app = Flask(__name__)
-logger = Logger("API")
+logger = Logger(actor="API", log_name_prefix=LOG_PREFIX)
 
 
 class API:

pisa/block_processor.py

@@ -1,8 +1,10 @@
 from common.logger import Logger
+
+from pisa import LOG_PREFIX
 from pisa.tools import bitcoin_cli
 from pisa.utils.auth_proxy import JSONRPCException
 
-logger = Logger("BlockProcessor")
+logger = Logger(actor="BlockProcessor", log_name_prefix=LOG_PREFIX)
 
 
 class BlockProcessor:

pisa/carrier.py

@@ -1,10 +1,11 @@
+from pisa import LOG_PREFIX
 from pisa.rpc_errors import *
 from common.logger import Logger
 from pisa.tools import bitcoin_cli
 from pisa.utils.auth_proxy import JSONRPCException
 from pisa.errors import UNKNOWN_JSON_RPC_EXCEPTION, RPC_TX_REORGED_AFTER_BROADCAST
 
-logger = Logger("Carrier")
+logger = Logger(actor="Carrier", log_name_prefix=LOG_PREFIX)
 
 # FIXME: This class is not fully covered by unit tests
 

pisa/chain_monitor.py

@@ -2,11 +2,12 @@ import zmq
 import binascii
 from threading import Thread, Event, Condition
 
+from pisa import LOG_PREFIX
 from common.logger import Logger
 from pisa.conf import FEED_PROTOCOL, FEED_ADDR, FEED_PORT, POLLING_DELTA, BLOCK_WINDOW_SIZE
 from pisa.block_processor import BlockProcessor
 
-logger = Logger("ChainMonitor")
+logger = Logger(actor="ChainMonitor", log_name_prefix=LOG_PREFIX)
 
 
 class ChainMonitor:

pisa/cleaner.py

@@ -1,6 +1,8 @@
+from pisa import LOG_PREFIX
+
 from common.logger import Logger
 
-logger = Logger("Cleaner")
+logger = Logger(actor="Cleaner", log_name_prefix=LOG_PREFIX)
 
 
 class Cleaner:

pisa/db_manager.py

@@ -1,9 +1,11 @@
 import json
 import plyvel
 
+from pisa import LOG_PREFIX
+
 from common.logger import Logger
 
-logger = Logger("DBManager")
+logger = Logger(actor="DBManager", log_name_prefix=LOG_PREFIX)
 
 WATCHER_PREFIX = "w"
 WATCHER_LAST_BLOCK_KEY = "bw"

pisa/inspector.py

@@ -4,12 +4,12 @@ from binascii import unhexlify
 from common.constants import LOCATOR_LEN_HEX
 from common.cryptographer import Cryptographer
 
-from pisa import errors
+from pisa import errors, LOG_PREFIX
 from common.logger import Logger
 from common.appointment import Appointment
 from pisa.block_processor import BlockProcessor
 
-logger = Logger("Inspector")
+logger = Logger(actor="Inspector", log_name_prefix=LOG_PREFIX)
 
 # FIXME: The inspector logs the wrong messages sent form the users. A possible attack surface would be to send a really
 #        long field that, even if not accepted by PISA, would be stored in the logs. This is a possible DoS surface

pisa/responder.py

@@ -2,6 +2,7 @@ import json
 from queue import Queue
 from threading import Thread
 
+from pisa import LOG_PREFIX
 from common.logger import Logger
 from pisa.cleaner import Cleaner
 from pisa.carrier import Carrier
@@ -10,7 +11,7 @@ from pisa.block_processor import BlockProcessor
 CONFIRMATIONS_BEFORE_RETRY = 6
 MIN_CONFIRMATIONS = 6
 
-logger = Logger("Responder")
+logger = Logger(actor="Responder", log_name_prefix=LOG_PREFIX)
 
 
 class TransactionTracker:

pisa/watcher.py

@@ -8,11 +8,12 @@ from common.tools import compute_locator
 
 from common.logger import Logger
 
+from pisa import LOG_PREFIX
 from pisa.cleaner import Cleaner
 from pisa.responder import Responder
 from pisa.block_processor import BlockProcessor
 
-logger = Logger("Watcher")
+logger = Logger(actor="Watcher", log_name_prefix=LOG_PREFIX)
 
 
 class Watcher: