-
-
Notifications
You must be signed in to change notification settings - Fork 124
Open
Labels
enhancementEnhancement the exist featureEnhancement the exist feature
Description
- Multiple times I was stumbled why some policies wouldn't work - until I realized that the expressions had some invalid syntax yet the TryExpressionBuilder was hiding the exception:
| catch (Exception) |
IMO we should log the error using the standard logger... I am also not sure why there is no try&catch around CompileExpression() - as there could be errors in there as well.
- FYI - the actual Interpretor (DynamicExpresso) has known issues with single quotes - might be useful to change those before the actual evaluation to double quotes, e.g.
expressionString = expressionString.Replace('\'', '"');Thanks
Reactions are currently unavailable
Metadata
Metadata
Labels
enhancementEnhancement the exist featureEnhancement the exist feature