Project

General

Profile

Actions
Copy link

Bug #13562

closed

"File key-value present" Generic Method and "Manage keys-values in file" Technique does not support | as separator

Added by Félix DALLIDET over 5 years ago. Updated almost 2 years ago.

Status:
Released
Priority:
N/A
Assignee:
Nicolas CHARLES
Category:
Generic methods
Target version:
4.1.17
Pull Request:
https://github.com/Normation/ncf/pull...
Severity:
Minor - inconvenience | misleading | easy workaround
UX impact:
User visibility:
Operational - other Techniques | Technique editor | Rudder settings
Effort required:
Very Small
Priority:
0
Name check:
Fix check:
Regression:

Description

When using "|" as separator, it fails and results in a broken key-value file.
Rudder does not detect that the result is not as expected and returns a repaired/success report.
It should either support the "|" or not allow it.

Actions
Copy link
 #1

Updated by Nicolas CHARLES over 5 years ago

I think it is interpreted as a regex character. You should try to escape it : \|

Actions
Copy link
 #2

Updated by Alexis Mousset over 5 years ago

  • Effort required set to Very Small
Actions
Copy link
 #3

Updated by Alexis Mousset over 5 years ago

  • Severity set to Minor - inconvenience | misleading | easy workaround
  • User visibility set to Operational - other Techniques | Technique editor | Rudder settings
  • Priority changed from 0 to 61
Actions
Copy link
 #4

Updated by Vincent MEMBRÉ over 5 years ago

  • Target version changed from 4.1.15 to 4.1.16
Actions
Copy link
 #5

Updated by Vincent MEMBRÉ over 5 years ago

  • Target version changed from 4.1.16 to 4.1.17
Actions
Copy link
 #6

Updated by Benoît PECCATTE over 5 years ago

  • Status changed from New to In progress
  • Assignee set to Benoît PECCATTE
  • Priority changed from 61 to 0
Actions
Copy link
 #7

Updated by Benoît PECCATTE over 5 years ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from Benoît PECCATTE to Nicolas CHARLES
  • Pull Request set to https://github.com/Normation/ncf/pull/872
Actions
Copy link
 #8

Updated by Benoît PECCATTE over 5 years ago

  • Status changed from Pending technical review to Pending release

Applied in changeset commit:9c097186342cf8593d6c26c9c48201ad67e40aa3.

Actions
Copy link
 #9

Updated by Nicolas CHARLES over 5 years ago

  • Subject changed from file_keys_values_present does not suport some separators to Generic Method file_keys_values_present does not support some separators
Actions
Copy link
 #10

Updated by Nicolas CHARLES over 5 years ago

  • Subject changed from Generic Method file_keys_values_present does not support some separators to Generic Method file_keys_values_present and technique "Manage keys-values in file" does not support some separators
Actions
Copy link
 #11

Updated by Vincent MEMBRÉ over 5 years ago

  • Subject changed from Generic Method file_keys_values_present and technique "Manage keys-values in file" does not support some separators to "File key-value present" Generic Method and "Manage keys-values in file" Technique does not support | as separator
Actions
Copy link
 #12

Updated by Vincent MEMBRÉ over 5 years ago

  • Status changed from Pending release to Released
This bug has been fixed in Rudder 4.1.17, 4.3.7 and 5.0.3 which were released today. 
Changelog
 Changelog
 Changelog
Actions
Copy link
 #13

Updated by Alexis Mousset almost 2 years ago

  • Project changed from 41 to Rudder
  • Category changed from Generic methods - File Management to Generic methods
Actions
Copy link

Also available in: Atom PDF