linse/frappy
0
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

log.exception instead of log.error in callbacks

Browse Source 
+ inhibit timestamps in future

Change-Id: Idef463558bf24e891573179474ffbebd4e81468f
This commit is contained in:
zolliker 2023-05-25 16:16:30 +02:00
parent 1de5455ef6
commit 0cbaffd4b6
1 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
frappy/client/__init__.py
View File

@ -245,8 +245,10 @@ class ProxyClient:
except UnregisterCallback: except UnregisterCallback:
cblist.remove(cbfunc) cblist.remove(cbfunc)
except Exception as e: except Exception as e:
# the programmer should catch all errors in callbacks
# if not, the log will be flooded with errors
if self.log: if self.log:
self.log.error('error %r calling %s%r', e, cbfunc.__name__, args) self.log.exception('error %r calling %s%r', e, cbfunc.__name__, args)
return bool(cblist) return bool(cblist)
def updateValue(self, module, param, value, timestamp, readerror): def updateValue(self, module, param, value, timestamp, readerror):
@ -398,6 +400,7 @@ class SecopClient(ProxyClient):
value = data[0] value = data[0]
readerror = None readerror = None
module, param = module_param module, param = module_param
timestamp = min(time.time(), timestamp) # no timestamps in the future!
try: try:
self.updateValue(module, param, value, timestamp, readerror) self.updateValue(module, param, value, timestamp, readerror)
except KeyError: except KeyError: