Bug #10304
closed
Broken variable expression when using node properties in the technique editor
Added by Avit Sidis about 7 years ago.
Updated about 6 years ago.
Category:
Web - Technique editor
Severity:
Major - prevents use of part of Rudder | no simple workaround
User visibility:
Operational - other Techniques | Technique editor | Rudder settings
Description
Whenever we use a node property in the technique editor inside a method like command, result classes contain a broken variable expression. Once we execute generated directive we encounter this error message:
error: Broken variable expression, index brackets do not balance, in 'node.properties[ENV_'
- Category set to Web - Technique editor
- Severity set to Major - prevents use of part of Rudder | no simple workaround
- User visibility set to Operational - other Techniques | Technique editor | Rudder settings
- Effort required set to Small
- Priority changed from 36 to 44
effort seems quite small, it's only the regex
regex = re.compile("[^\$\{\}a-zA-Z0-9_](?![^{}]+})|\$(?!{)")
in tools/ncf_rudder.py to change
someone competend in regex could fix it pretty easily
- Related to Bug #8586: Wrong definition of class condition if we use a variable with [ ] in it added
- Priority changed from 44 to 53
- Related to Bug #11600: Result class containing {} are not properly canonized added
- Priority changed from 53 to 60
- Target version set to 4.1.10
- Priority changed from 60 to 57
- Status changed from New to In progress
- Assignee set to Benoît PECCATTE
- Status changed from In progress to Pending technical review
- Assignee changed from Benoît PECCATTE to Vincent MEMBRÉ
- Pull Request set to https://github.com/Normation/ncf/pull/696
- Assignee changed from Vincent MEMBRÉ to Benoît PECCATTE
- Status changed from Pending technical review to Pending release
Applied in changeset ncf:commit:4ec79bb68c0b9e58be83a1a35d2d26a8d69d910a.
- Status changed from Pending release to Released
- Priority changed from 57 to 56
This bug has been fixed in Rudder 4.1.10 and 4.2.4 which were released today.
Also available in: Atom
PDF
Updated by 
Updated by 
Updated by 
Updated by 
Updated by