Project

General

Profile

Actions
Copy link

User story #5942

open

Let user define the full gecos field in user management technique

Added by Alexander Brunhirl over 9 years ago. Updated almost 2 years ago.

Status:
New
Priority:
N/A
Assignee:
-
Category:
Techniques
Target version:
Ideas (not version specific)
Pull Request:
https://github.com/Normation/rudder-t...
UX impact:
Suggestion strength:
User visibility:
Effort required:
Name check:
Fix check:
Regression:

Description

Hi,

if there's a comma in the GECOS field, the rule is not be actually applied.

Example Output:
Status Success "The user testuser1 ( test user 1 111 49089-XYZ 00_1 ) is already present on the system"
Status Repaired "The user testuser2 ( test user 2, 111, 49089-XYZ, 00_2 ) had a wrong fullname"

Directive: User management
Directive version: 5.0
Rudder version: Rudder v.2.11.5~rc1~git201411220241

Related issues 1 (0 open — 1 closed)

Related to Rudder - Bug #5327: On 2.11 rudder-agent, "Repair" status is always defined, even if nothing is modifiedReleasedJonathan CLARKE2014-07-30Actions
Actions
Copy link
 #1

Updated by Nicolas CHARLES over 9 years ago

Hi alex,

Thank you for your bug report, I'll investigate the issue

Actions
Copy link
 #3

Updated by Nicolas CHARLES over 9 years ago

  • Status changed from New to Pending technical review
  • Assignee set to Jonathan CLARKE
  • Pull Request set to https://github.com/Normation/rudder-techniques/pull/602/files
Actions
Copy link
 #4

Updated by Jonathan CLARKE over 9 years ago

  • Status changed from Pending technical review to Discussion
  • Assignee changed from Jonathan CLARKE to Alexander Brunhirl

Hi Alex!

Actually, I think that what you're trying to do here is unsupported. The GECOS field in /etc/passwd should contain the following fields: Full name,Room number,Work phone,Home Phone,Other. The comma is the separator here.

By using commas in the "Full name" field in Rudder's Technique, you're effectively trying to use the "Full name" field as "GECOS" field. This is not supported, the point of this field is to fill in the "Full name".

However, there is indeed a bug here: we should filter out commas to avoid "breaking" the GECOS field, and make sure the Technique only changes the first field.

We could, of course, add a new Technique version to also manage the others fields from GECOS. Would this be useful to you?

Actions
Copy link
 #5

Updated by Alexander Brunhirl over 9 years ago

Hi Jonathan,

thanks for the explaining :-)
I thought with the "Full name" field you meant the entire GECOS field, so I tried this.

It isn't necessary for me, but maybe it's quite useful for others to add this feature.

Actions
Copy link
 #6

Updated by Vincent MEMBRÉ over 9 years ago

  • Target version changed from 2.11.5 to 2.11.6
Actions
Copy link
 #7

Updated by Vincent MEMBRÉ about 9 years ago

  • Target version changed from 2.11.6 to 2.11.7
Actions
Copy link
 #8

Updated by Vincent MEMBRÉ about 9 years ago

  • Target version changed from 2.11.7 to 2.11.8
Actions
Copy link
 #9

Updated by Vincent MEMBRÉ about 9 years ago

  • Target version changed from 2.11.8 to 2.11.9
Actions
Copy link
 #10

Updated by Vincent MEMBRÉ about 9 years ago

  • Target version changed from 2.11.9 to 2.11.10
Actions
Copy link
 #11

Updated by Vincent MEMBRÉ about 9 years ago

  • Target version changed from 2.11.10 to 2.11.11
Actions
Copy link
 #12

Updated by Benoît PECCATTE almost 9 years ago

  • Assignee deleted (Alexander Brunhirl)
Actions
Copy link
 #13

Updated by Vincent MEMBRÉ almost 9 years ago

  • Target version changed from 2.11.11 to 2.11.12
Actions
Copy link
 #14

Updated by Vincent MEMBRÉ almost 9 years ago

  • Target version changed from 2.11.12 to 2.11.13
Actions
Copy link
 #15

Updated by Vincent MEMBRÉ over 8 years ago

  • Target version changed from 2.11.13 to 2.11.14
Actions
Copy link
 #16

Updated by Vincent MEMBRÉ over 8 years ago

  • Target version changed from 2.11.14 to 2.11.15
Actions
Copy link
 #17

Updated by Vincent MEMBRÉ over 8 years ago

  • Target version changed from 2.11.15 to 2.11.16
Actions
Copy link
 #18

Updated by Vincent MEMBRÉ over 8 years ago

  • Target version changed from 2.11.16 to 2.11.17
Actions
Copy link
 #19

Updated by Vincent MEMBRÉ over 8 years ago

  • Target version changed from 2.11.17 to 2.11.18
Actions
Copy link
 #20

Updated by Vincent MEMBRÉ about 8 years ago

  • Target version changed from 2.11.18 to 2.11.19
Actions
Copy link
 #21

Updated by Vincent MEMBRÉ about 8 years ago

  • Target version changed from 2.11.19 to 2.11.20
Actions
Copy link
 #22

Updated by Vincent MEMBRÉ about 8 years ago

  • Target version changed from 2.11.20 to 2.11.21
Actions
Copy link
 #23

Updated by Vincent MEMBRÉ almost 8 years ago

  • Target version changed from 2.11.21 to 2.11.22
Actions
Copy link
 #24

Updated by Vincent MEMBRÉ almost 8 years ago

  • Target version changed from 2.11.22 to 2.11.23
Actions
Copy link
 #25

Updated by Vincent MEMBRÉ over 7 years ago

  • Target version changed from 2.11.23 to 2.11.24
Actions
Copy link
 #26

Updated by Vincent MEMBRÉ over 7 years ago

  • Target version changed from 2.11.24 to 308
Actions
Copy link
 #27

Updated by Vincent MEMBRÉ over 7 years ago

  • Target version changed from 308 to 3.1.14
Actions
Copy link
 #28

Updated by Vincent MEMBRÉ over 7 years ago

  • Target version changed from 3.1.14 to 3.1.15
Actions
Copy link
 #29

Updated by Vincent MEMBRÉ over 7 years ago

  • Target version changed from 3.1.15 to 3.1.16
Actions
Copy link
 #30

Updated by Vincent MEMBRÉ over 7 years ago

  • Target version changed from 3.1.16 to 3.1.17
Actions
Copy link
 #31

Updated by Vincent MEMBRÉ over 7 years ago

  • Target version changed from 3.1.17 to 3.1.18
Actions
Copy link
 #32

Updated by Vincent MEMBRÉ about 7 years ago

  • Target version changed from 3.1.18 to 3.1.19
Actions
Copy link
 #33

Updated by Jonathan CLARKE about 7 years ago

  • Status changed from Discussion to New
  • Severity set to Minor - inconvenience | misleading | easy workaround
  • User visibility set to Getting started - demo | first install | level 1 Techniques
Actions
Copy link
 #34

Updated by Vincent MEMBRÉ about 7 years ago

  • Target version changed from 3.1.19 to 3.1.20
  • Priority set to 0
Actions
Copy link
 #35

Updated by Vincent MEMBRÉ almost 7 years ago

  • Target version changed from 3.1.20 to 3.1.21
Actions
Copy link
 #36

Updated by Vincent MEMBRÉ almost 7 years ago

  • Target version changed from 3.1.21 to 3.1.22
Actions
Copy link
 #37

Updated by Benoît PECCATTE almost 7 years ago

  • Subject changed from GECOS field to Comas in the full name fiemld make the users technique fail
  • Priority changed from 0 to 29

We should not accept comas here.

Actions
Copy link
 #38

Updated by Benoît PECCATTE almost 7 years ago

  • Priority changed from 29 to 42
Actions
Copy link
 #39

Updated by Benoît PECCATTE almost 7 years ago

  • Effort required set to Small
  • Priority changed from 42 to 45
Actions
Copy link
 #40

Updated by Vincent MEMBRÉ over 6 years ago

  • Target version changed from 3.1.22 to 3.1.23
Actions
Copy link
 #41

Updated by Vincent MEMBRÉ over 6 years ago

  • Target version changed from 3.1.23 to 3.1.24
Actions
Copy link
 #42

Updated by Vincent MEMBRÉ over 6 years ago

  • Target version changed from 3.1.24 to 3.1.25
Actions
Copy link
 #43

Updated by Benoît PECCATTE over 6 years ago

  • Priority changed from 45 to 55
Actions
Copy link
 #44

Updated by Vincent MEMBRÉ over 6 years ago

  • Target version changed from 3.1.25 to 387
Actions
Copy link
 #45

Updated by Alexis Mousset over 6 years ago

  • Subject changed from Comas in the full name fiemld make the users technique fail to Comas in the full name field make the users technique fail
Actions
Copy link
 #46

Updated by Vincent MEMBRÉ over 6 years ago

  • Target version changed from 387 to 4.1.10
Actions
Copy link
 #47

Updated by Vincent MEMBRÉ about 6 years ago

  • Target version changed from 4.1.10 to 4.1.11
  • Priority changed from 55 to 56
Actions
Copy link
 #48

Updated by Vincent MEMBRÉ about 6 years ago

  • Target version changed from 4.1.11 to 4.1.12
  • Priority changed from 56 to 57
Actions
Copy link
 #49

Updated by Vincent MEMBRÉ almost 6 years ago

  • Target version changed from 4.1.12 to 4.1.13
  • Priority changed from 57 to 58
Actions
Copy link
 #50

Updated by Vincent MEMBRÉ almost 6 years ago

  • Target version changed from 4.1.13 to 4.1.14
  • Priority changed from 58 to 59
Actions
Copy link
 #51

Updated by Nicolas CHARLES almost 6 years ago

  • Tracker changed from Bug to User story
  • Subject changed from Comas in the full name field make the users technique fail to Let user define the full gecos field in user management technique
  • Target version changed from 4.1.14 to Ideas (not version specific)
  • Severity deleted (Minor - inconvenience | misleading | easy workaround)
  • User visibility deleted (Getting started - demo | first install | level 1 Techniques)
  • Effort required deleted (Small)
  • Priority deleted (59)

After rereading the whole thread, the idea of this ticket is to set the GECOS field, not simply the full user name
So making it an user story

Actions
Copy link
 #52

Updated by Alexander Brunhirl over 5 years ago

Nicolas CHARLES wrote:

After rereading the whole thread, the idea of this ticket is to set the GECOS field, not simply the full user name
So making it an user story

Thank you

Actions
Copy link
 #53

Updated by Nicolas CHARLES about 5 years ago

  • Translation missing: en.field_tag_list set to Sponsored
Actions
Copy link

Also available in: Atom PDF