Project

General

Profile

Actions

User story #3281

closed

User story #2947: A special textarea-type field should exist for the Techniques that automatically escapes some characters

Change Rudder variable to allow for automatic escaping or raw value

Added by Nicolas CHARLES about 11 years ago. Updated about 9 years ago.

Status:
Released
Priority:
1
Assignee:
-
Category:
Web - Config management
Target version:
UX impact:
Suggestion strength:
User visibility:
Effort required:
Name check:
Fix check:
Regression:

Description

CFEngine accepts only certins type of string ( quote must be escaped, must not finish by a \ otherwise it escape the final quote ), so we need to escape the text. So far it is done manually, which is far from perfect.
We need to do it automatically, and create a specific constraint for raw values


Related issues 1 (1 open0 closed)

Related to Rudder - Architecture #5981: \ (backslashes) are replaced a bit to eagerly by \\ in the generated promisesNewActions
Actions #1

Updated by Nicolas CHARLES about 11 years ago

The PR is here
https://github.com/Normation/cf-clerk/pull/7

In a nutshell:

  • systemVariable will behave the same (non escaped automatically)
  • all variables by default are escaped
  • a contraint "raw" is created, allowing to define variable that we don't want to be escaped automatically
Actions #2

Updated by Nicolas CHARLES about 11 years ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from Nicolas CHARLES to François ARMAND
Actions #3

Updated by Nicolas CHARLES about 11 years ago

The pull request for reporting is there
https://github.com/Normation/rudder/pull/123

Actions #4

Updated by François ARMAND about 11 years ago

The two pull request seems OK, but to follow the convention, I'm going to move the second one into its own ticket (reminder: one ticket by pull-request).

Actions #5

Updated by François ARMAND about 11 years ago

  • Status changed from Pending technical review to Discussion
Actions #6

Updated by François ARMAND about 11 years ago

  • Status changed from Discussion to 10
  • Assignee changed from François ARMAND to Jonathan CLARKE
Actions #7

Updated by François ARMAND about 11 years ago

The second pull request (123) is now linked to ticket #3317

Actions #8

Updated by François ARMAND about 11 years ago

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

Updated by François ARMAND about 11 years ago

  • Status changed from Pending technical review to 12
  • Assignee deleted (Jonathan CLARKE)
Actions #10

Updated by Nicolas CHARLES about 11 years ago

  • Status changed from 12 to Pending release
  • % Done changed from 0 to 100

Applied in changeset clerk:commit:8df226faf7717e7c336b02139c85f77e6ed81256.

Actions #11

Updated by Matthieu CERDA about 11 years ago

Applied in changeset clerk:commit:3fbb8c4cf9964c415b1fd38146beed0f56ded726.

Actions #12

Updated by Jonathan CLARKE about 11 years ago

  • Status changed from Pending release to Released

This ticket has been addressed in version 2.6.0~beta1 of Rudder, which has just been released. Please see the changelog here: https://www.rudder-project.org/foswiki/System/Documentation:ChangeLog26.

Actions #13

Updated by Benoît PECCATTE about 9 years ago

  • Category changed from 14 to Web - Config management
Actions #14

Updated by Benoît PECCATTE about 9 years ago

  • Tracker changed from Enhancement to User story
Actions

Also available in: Atom PDF