diff options
| author | Alejandro Sirgo Rica <asirgo@soleta.eu> | 2024-03-18 09:36:55 +0100 |
|---|---|---|
| committer | lupoDharkael <izhe@hotmail.es> | 2024-03-21 10:29:57 +0100 |
| commit | 0cbf16461e05bc5f15d3436763667b1d34869826 (patch) | |
| tree | 117371efe019e9f3d88b45917253fc04fec989ff /src | |
| parent | 80125623027283c9a9d94a98bea442b1f0c11206 (diff) | |
ogclient: consolidate logging in client core
The main function must be able to handle the login of critical
error in the main ogClient class instance. Add a try except block
to the ogCLient run logic and move the relevant error logs into
the except block.
Delegate the error messages to the exception message. This is the
first step towards error message deduplication.
Diffstat (limited to 'src')
| -rw-r--r-- | src/ogClient.py | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/src/ogClient.py b/src/ogClient.py index 31ee5d3..abacc12 100644 --- a/src/ogClient.py +++ b/src/ogClient.py @@ -31,8 +31,7 @@ class ogClient: self.mode = self.CONFIG['opengnsys']['mode'] if self.mode not in {'virtual', 'live', 'linux', 'windows'}: - logging.critical('Invalid ogClient mode') - raise ValueError('Mode not supported.') + raise ValueError(f'Invalid ogClient mode: {self.mode}.') if self.mode in {'linux', 'windows'}: self.event_sock = socket.socket(socket.AF_INET, socket.SOCK_DGRAM) self.event_sock.setblocking(0) @@ -191,4 +190,4 @@ class ogClient: message = event_sock.recv(4096).decode('utf-8').rstrip() self.send_event_hint(message) else: - logging.critical('Invalid state: %s', str(state)) + raise ValueError(f'Invalid ogClient run state: {str(state)}.') |