Project

General

Profile

Bug #11326

chars are not escaped correctly in dsc techniques

Added by Félix DALLIDET 3 months ago. Updated 3 months ago.

Status:
Released
Priority:
N/A
Category:
Web - Config management
Target version:
Target version (plugin):
Severity:
Critical - prevents main use of Rudder | no workaround | data loss | security
User visibility:
Operational - other Techniques | Technique editor | Rudder settings
Effort required:
Priority:
76

Description

As in https://www.rudder-project.org/redmine/issues/11268 but for dsc techniques.
Escaping is done with "\" but should be done with "`"


Subtasks

Bug #11384: Broken policy generation with internal errorReleasedNicolas CHARLES

Bug #11386: Invalid patternRejectedVincent MEMBRÉ

Associated revisions

Revision 891338ec
Added by Vincent MEMBRÉ 3 months ago

Fixes #11326: chars are not escaped correctly in dsc techniques

History

#1 Updated by François ARMAND 3 months ago

  • Severity set to Critical - prevents main use of Rudder | no workaround | data loss | security
  • User visibility set to Operational - other Techniques | Technique editor | Rudder settings
  • Priority changed from 0 to 76

The escaping must not be done in JS but at (CFEngine, DSC) generation time, in ncf. We need to have the real, raw input data to be able to correctly escape for each targetted agent language.

#2 Updated by François ARMAND 3 months ago

  • Project changed from Rudder to ncf
  • Category set to Builder - API
  • Target version set to master

#3 Updated by Vincent MEMBRÉ 3 months ago

  • Status changed from New to In progress
  • Assignee set to Vincent MEMBRÉ

#4 Updated by Vincent MEMBRÉ 3 months ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from Vincent MEMBRÉ to François ARMAND
  • Pull Request set to https://github.com/Normation/rudder/pull/1736

#5 Updated by Vincent MEMBRÉ 3 months ago

  • Project changed from ncf to Rudder
  • Category changed from Builder - API to Web - Config management
  • Target version changed from master to 4.2.0~rc2

#6 Updated by Vincent MEMBRÉ 3 months ago

  • Pull Request deleted (https://github.com/Normation/rudder/pull/1736)

#7 Updated by Vincent MEMBRÉ 3 months ago

  • Pull Request set to https://github.com/Normation/rudder/pull/1737

#8 Updated by Vincent MEMBRÉ 3 months ago

  • Target version changed from 4.2.0~rc2 to 4.2.0

#9 Updated by Vincent MEMBRÉ 3 months ago

  • Status changed from Pending technical review to Pending release

#10 Updated by Vincent MEMBRÉ 3 months ago

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 4.2.0 which was released today.

Also available in: Atom PDF