Project

General

Profile

Actions

Bug #11613

closed

generic_variable_definition inserts spaces in multi lined values

Added by Janos Mattyasovszky over 6 years ago. Updated over 6 years ago.

Status:
Released
Priority:
N/A
Category:
Techniques
Target version:
Severity:
Major - prevents use of part of Rudder | no simple workaround
UX impact:
User visibility:
Operational - other Techniques | Technique editor | Rudder settings
Effort required:
Priority:
52
Name check:
Fix check:
Regression:

Description

If you have a generic_variable_definition value that consists of multiple lines, due to how the .st file is built up, it inserts spaces as of the second line, indenting the content.

    &GENERIC_VARIABLE_NAME, GENERIC_VARIABLE_CONTENT:{name, content |"&name&" string => "&content&",

https://github.com/Normation/rudder-techniques/blob/91dc9d9e48d300afd76edcf37a961fa7069a3d5a/techniques/systemSettings/misc/genericVariableDefinition/2.0/genericVariableDefinition.st#L24

Resulting generated code example:

    "var_authorized_key_final" string => "${generic_variable_definition.auth_key_1}
    ${generic_variable_definition.auth_key_2}",

That results in a content, when written to a file:

ssh-rsa AAAAB3NzaC1yc2EAAA
    ssh-rsa AAAAB3NzaC1yc2


Subtasks 1 (0 open1 closed)

Bug #11629: Same as parent ticket for technique "Variables (any)"ReleasedAlexis MoussetActions
Actions #1

Updated by Nicolas CHARLES over 6 years ago

Hi Janos
How can you multiline with this technique ?

Actions #2

Updated by Janos Mattyasovszky over 6 years ago

  • Description updated (diff)
Actions #3

Updated by Janos Mattyasovszky over 6 years ago

  • Description updated (diff)
Actions #4

Updated by Florian Heigl over 6 years ago

it is my understanding that this use was never indented.
now the question is if it would be possible to fix it to actually work, since it seems to be pretty close to working.

Actions #5

Updated by Janos Matya over 6 years ago

You just have to remove some spaces in front of the variable substitution of string template, the diff is basically about 6x [space]...

Actions #6

Updated by Nicolas CHARLES over 6 years ago

  • Assignee set to Nicolas CHARLES
  • Target version set to 3.1.24
Actions #7

Updated by Nicolas CHARLES over 6 years ago

  • Status changed from New to In progress
Actions #8

Updated by Nicolas CHARLES over 6 years ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from Nicolas CHARLES to Alexis Mousset
  • Pull Request set to https://github.com/Normation/rudder-techniques/pull/1212
Actions #9

Updated by Nicolas CHARLES over 6 years ago

  • Status changed from Pending technical review to Pending release
Actions #10

Updated by Vincent MEMBRÉ over 6 years ago

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 3.1.24, 4.1.8 and 4.2.1 which were released today.

Actions

Also available in: Atom PDF