Project

General

Profile

Actions
Copy link

Bug #14372

closed

Ncf techniques used in windows can not start with a numerical char

Added by Félix DALLIDET almost 6 years ago. Updated 7 months ago.

Status:
Released
Priority:
N/A
Assignee:
Vincent MEMBRÉ
Category:
Web - Technique editor
Target version:
7.3.15
Pull Request:
Severity:
Major - prevents use of part of Rudder | no simple workaround
UX impact:
User visibility:
Getting started - demo | first install | Technique editor and level 1 Techniques
Effort required:
Priority:
0
Name check:
Fix check:
Regression:
No

Description

Creating a ncf tehcnique starting with a numerical char like "0" will break the DSC agent, at least if the name contains a "-" in the name.
ex: "00-Kb-deployment"

Powershell will not understand and try to interpret the "00" part as a cmdlet and "-Kb" as a given cmdlet option.

Related issues 1 (0 open — 1 closed)

Related to Rudder - Bug #14371: At least on windows, technique editor parameters can not handle "_" charReleasedFélix DALLIDETActions
Actions
Copy link
 #1

Updated by François ARMAND over 5 years ago

  • Target version changed from 5.0.7 to 5.0.9
Actions
Copy link
 #2

Updated by François ARMAND over 5 years ago

  • Severity set to Major - prevents use of part of Rudder | no simple workaround
  • User visibility set to Getting started - demo | first install | Technique editor and level 1 Techniques
  • Priority changed from 0 to 102

I set it to major as it can be extremelly hard for an user to understand what is happening given logs, and we advice to use that nomenclature in staring guide etc.

Actions
Copy link
 #4

Updated by François ARMAND over 5 years ago

  • Assignee set to Vincent MEMBRÉ

That needs to be escaped on file system.

Actions
Copy link
 #5

Updated by Vincent MEMBRÉ over 5 years ago

  • Target version changed from 5.0.9 to 5.0.10
  • Priority changed from 102 to 101
Actions
Copy link
 #6

Updated by Vincent MEMBRÉ over 5 years ago

  • Target version changed from 5.0.10 to 5.0.11
  • Priority changed from 101 to 99
Actions
Copy link
 #7

Updated by Vincent MEMBRÉ over 5 years ago

  • Target version changed from 5.0.11 to 5.0.12
Actions
Copy link
 #8

Updated by Vincent MEMBRÉ over 5 years ago

  • Target version changed from 5.0.12 to 5.0.13
  • Priority changed from 99 to 97
Actions
Copy link
 #9

Updated by Vincent MEMBRÉ about 5 years ago

  • Target version changed from 5.0.13 to 5.0.14
  • Priority changed from 97 to 94
Actions
Copy link
 #10

Updated by François ARMAND about 5 years ago

  • Priority changed from 94 to 93

This is an hard limit of powershell, and besides doing some encoding on our side, we can't do
We will try to make some encoding for it => prefix generated method name ONLY ONE DSC AGENT with one character (like "r" or whatever).

Actions
Copy link
 #11

Updated by Vincent MEMBRÉ about 5 years ago

  • Target version changed from 5.0.14 to 5.0.15
  • Priority changed from 93 to 92
Actions
Copy link
 #12

Updated by Vincent MEMBRÉ almost 5 years ago

  • Target version changed from 5.0.15 to 5.0.16
  • Priority changed from 92 to 89
Actions
Copy link
 #13

Updated by Alexis Mousset almost 5 years ago

  • Target version changed from 5.0.16 to 5.0.17
  • Priority changed from 89 to 85
Actions
Copy link
 #14

Updated by Vincent MEMBRÉ over 4 years ago

  • Target version changed from 5.0.17 to 5.0.18
Actions
Copy link
 #15

Updated by Nicolas CHARLES over 4 years ago

  • Related to Bug #14371: At least on windows, technique editor parameters can not handle "_" char added
Actions
Copy link
 #16

Updated by François ARMAND over 4 years ago

  • Target version changed from 5.0.18 to 6.0.8
Actions
Copy link
 #17

Updated by Vincent MEMBRÉ over 4 years ago

  • Target version changed from 6.0.8 to 6.0.9
Actions
Copy link
 #18

Updated by Vincent MEMBRÉ about 4 years ago

  • Target version changed from 6.0.9 to 6.0.10
Actions
Copy link
 #19

Updated by Vincent MEMBRÉ about 4 years ago

  • Target version changed from 6.0.10 to 798
Actions
Copy link
 #20

Updated by Benoît PECCATTE over 3 years ago

  • Target version changed from 798 to 6.1.14
Actions
Copy link
 #21

Updated by Vincent MEMBRÉ over 3 years ago

  • Target version changed from 6.1.14 to 6.1.15
Actions
Copy link
 #22

Updated by Vincent MEMBRÉ over 3 years ago

  • Target version changed from 6.1.15 to 6.1.16
Actions
Copy link
 #23

Updated by Vincent MEMBRÉ about 3 years ago

  • Target version changed from 6.1.16 to 6.1.17
Actions
Copy link
 #24

Updated by Vincent MEMBRÉ about 3 years ago

  • Target version changed from 6.1.17 to 6.1.18
Actions
Copy link
 #25

Updated by Vincent MEMBRÉ almost 3 years ago

  • Target version changed from 6.1.18 to 6.1.19
Actions
Copy link
 #26

Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 6.1.19 to 6.1.20
  • Priority changed from 85 to 86
Actions
Copy link
 #27

Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 6.1.20 to 6.1.21
  • Priority changed from 86 to 88
Actions
Copy link
 #29

Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 6.1.21 to old 6.1 issues to relocate
  • Priority changed from 88 to 60
Actions
Copy link
 #30

Updated by Alexis Mousset about 1 year ago

  • Target version changed from old 6.1 issues to relocate to 7.3.10
  • Priority changed from 60 to 0
Actions
Copy link
 #31

Updated by Vincent MEMBRÉ 12 months ago

  • Target version changed from 7.3.10 to 7.3.11
Actions
Copy link
 #32

Updated by Vincent MEMBRÉ 10 months ago

  • Target version changed from 7.3.11 to 7.3.12
Actions
Copy link
 #33

Updated by Vincent MEMBRÉ 9 months ago

  • Target version changed from 7.3.12 to 7.3.13
Actions
Copy link
 #34

Updated by Vincent MEMBRÉ 9 months ago

  • Target version changed from 7.3.13 to 7.3.14
Actions
Copy link
 #35

Updated by Vincent MEMBRÉ 7 months ago

  • Target version changed from 7.3.14 to 7.3.15
Actions
Copy link
 #36

Updated by Alexis Mousset 7 months ago

  • Status changed from New to Released
  • Regression set to No

We fixed it by adding a prefix for windows techniques

Actions
Copy link

Also available in: Atom PDF