Project

General

Profile

Actions

Bug #9901

closed

Error when using quotes on class parameters

Added by Alexis Mousset over 5 years ago. Updated 4 months ago.

Status:
Released
Priority:
N/A
Category:
Web - Technique editor
Target version:
Severity:
UX impact:
User visibility:
Effort required:
Priority:
0
Regression:

Description

The class changes when reloading the page when there is a backquote.

For example using the

test $(rpm -qa|sed -ne \'s/package-//p\') == "0.1-1" 

command execution result.


Related issues 2 (0 open2 closed)

Related to Rudder - Bug #10723: Escaping backslash+double quote in NCF method file_ensure_lines_presentResolvedActions
Related to Rudder - Bug #10809: Invalid handling of escaped quotes/double quotesResolvedActions
Actions #1

Updated by Alexis Mousset over 5 years ago

  • Subject changed from Error when using backquotes on class parameters to Error when using quotes on class parameters

When using a quote, it is replaced by "\'"

Actions #2

Updated by Vincent MEMBRÉ over 5 years ago

  • Status changed from New to In progress
Actions #3

Updated by Vincent MEMBRÉ over 5 years ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from Vincent MEMBRÉ to Benoît PECCATTE
  • Pull Request set to https://github.com/Normation/ncf/pull/508
Actions #4

Updated by Vincent MEMBRÉ over 5 years ago

  • Status changed from Pending technical review to Pending release
  • % Done changed from 0 to 100

Applied in changeset commit:62655c7f838393902b0804c23bb5993b02117670.

Actions #5

Updated by Nicolas CHARLES over 5 years ago

this change seems to prevent backquoting quotes :/

Actions #6

Updated by Nicolas CHARLES over 5 years ago

ha, it was also impossible before, so it simply inverts the impossible behaviour

Actions #7

Updated by Nicolas CHARLES over 5 years ago

  • Related to Bug #10723: Escaping backslash+double quote in NCF method file_ensure_lines_present added
Actions #8

Updated by Nicolas CHARLES over 5 years ago

  • Related to Bug #10809: Invalid handling of escaped quotes/double quotes added
Actions #9

Updated by Vincent MEMBRÉ almost 5 years ago

  • Status changed from Pending release to Released
  • Priority set to 0
Actions #10

Updated by Alexis Mousset 4 months ago

  • Target version changed from 0.x to ncf-0.x
Actions #11

Updated by Alexis Mousset 4 months ago

  • Project changed from 41 to Rudder
  • Category changed from Technique editor - API to Web - Technique editor
Actions

Also available in: Atom PDF