Actions
Bug #17351
closed
when we are not qutoing the default value in node property expansion, the error is not helpful
Pull Request:
Severity:
UX impact:
User visibility:
Effort required:
Priority:
0
Name check:
To do
Fix check:
To do
Regression:
Description
if i want to expand a value from node property and have a fallback default like in the follogin
${node.properties[AWS-AWS-Dynamic][instance-identity][document][accountid]|default="'value'"}
, if I don't quote the value, I get non helpful error message
⇨ Accumulated: Unexpected: Error with parameters expansion for node 'xxxxn.com' (root); cause was: Accumulated: When processing directive 'test variale parameter'; cause was: Accumulated: On variable 'GENERIC_FILE_CONTENT_PAYLOAD':; cause was: Accumulated: Unexpected: Error when parsing value "${node.properties[AWS-AWS-Dynamic][instance-identity][document][accountid]|default='value'}", error message is: end of input expected ; Unexpected: Can not configure 'root' children node because 'root' is a policy server whose configuration is in error
it should clearly state that the default value should be quoted
Updated by François ARMAND over 4 years ago
- Target version changed from 6.0.6 to 6.1.0~beta3
It's different in 6.1 (but perhaps not much better). We will change that in 6.1 because it can't be made much better with the other parser.
Updated by François ARMAND over 4 years ago
Done along with #17350.
The message is a bit better, even if it's still a parsing error:
Error when parsing value (without ''): 'some text and ${node.properties[datacenter]|default= "missing end quote} and some more text'. Error message is: Expected (string | emptyString | variable):1:54, found "\"missing e
I don't think we can do much better.
Updated by François ARMAND over 4 years ago
- Status changed from New to Pending release
Updated by François ARMAND over 4 years ago
- Status changed from Pending release to Discussion
Updated by François ARMAND over 4 years ago
- Status changed from Discussion to Resolved
Actions