Skip to content

Commit a61a1aa

Browse files
authored
26262 - change logging approach (#3279)
* change logging approach to use logger.info or logger.error instead of print()
1 parent 9cf6551 commit a61a1aa

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

legal-api/src/legal_api/models/db.py

+3-3
Original file line numberDiff line numberDiff line change
@@ -64,10 +64,10 @@ def print_versioning_info():
6464
db_versioning = flag_service.value('db-versioning')
6565
use_new_versioning = (bool(db_versioning) and bool(db_versioning.get(current_service)))
6666
current_versioning = 'new' if use_new_versioning else 'old'
67-
print(f'\033[31mService: {current_service}, db versioning={current_versioning}\033[0m')
67+
current_app.logger.info(f'\033[31mService: {current_service}, db versioning={current_versioning}\033[0m')
6868
except Exception as err:
6969
# Don't crash if something goes wrong
70-
print(f'\033[31mUnable to determine versioning type: {err}\033[0m')
70+
current_app.logger.error('Unable to read flags: %s' % repr(err), exc_info=True)
7171

7272

7373
def init_db(app):
@@ -182,7 +182,7 @@ def _switch_versioning(cls, previous, current):
182182
cls._versioning_control[previous]['disable']()
183183
cls._versioning_control[current]['enable']()
184184
# Print when versioning changes
185-
print(f'\033[31mVersioning changed: {previous} -> {current}\033[0m')
185+
current_app.logger.info(f'\033[31mVersioning changed: {previous} -> {current}\033[0m')
186186

187187
@classmethod
188188
def lock_versioning(cls, session, transaction):

0 commit comments

Comments
 (0)