Psion-fanatic/platypush
1
0
Fork 0
forked from platypush/platypush
Code Pull requests Activity

Don't fail silently if there were errors in parsing ${} variables or getting the context

Browse source
This commit is contained in:
Fabio Manganiello 2021-02-26 22:43:54 +01:00
parent b4258faa29
commit 9e00428568
2 changed files with 13 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
platypush/message/request/__init__.py
View file

@ -130,8 +130,10 @@ class Request(Message):
if isinstance(v, str):
try:
exec('{}="{}"'.format(k, re.sub(r'(^|[^\\])"', '\1\\"', v)))
except:
pass
except Exception as e:
logger.warning('Could not set context variable {}={}'.format(k, v))
logger.warning('Context: {}'.format(json.dumps(context)))
logger.exception(e)
parsed_value = ''
if not isinstance(_value, str):

13
platypush/procedure/__init__.py
View file

@ -1,4 +1,5 @@
import enum
import json
import logging
import re
from functools import wraps
@ -340,8 +341,10 @@ class WhileProcedure(LoopProcedure):
# noinspection PyBroadException
try:
context[k] = eval('"{}"'.format(re.sub(r'(^|[^\\])"', '\1\\"', v)))
except:
pass
except Exception as e:
logger.warning('Could not parse value for context variable {}={}'.format(k, v))
logger.warning('Context: {}'.format(json.dumps(context)))
logger.exception(e)
return context
@ -453,8 +456,10 @@ class IfProcedure(Procedure):
if isinstance(v, str):
try:
exec('{}="{}"'.format(k, re.sub(r'(^|[^\\])"', '\1\\"', v)))
except:
pass
except Exception as e:
logger.warning('Could not set context variable {}={}'.format(k, v))
logger.warning('Context: {}'.format(json.dumps(context)))
logger.exception(e)
condition_true = eval(self.condition)
response = Response()