User story #5942
openLet user define the full gecos field in user management technique
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
Updated by Nicolas CHARLES almost 10 years ago
Hi alex,
Thank you for your bug report, I'll investigate the issue
Updated by Nicolas CHARLES almost 10 years ago
Ok, this looks a lot like http://www.rudder-project.org/redmine/issues/5327, which is related to https://dev.cfengine.com/issues/6467
Updated by Nicolas CHARLES almost 10 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
Updated by Jonathan CLARKE almost 10 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?
Updated by Alexander Brunhirl almost 10 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.
Updated by Vincent MEMBRÉ almost 10 years ago
- Target version changed from 2.11.5 to 2.11.6
Updated by Vincent MEMBRÉ almost 10 years ago
- Target version changed from 2.11.6 to 2.11.7
Updated by Vincent MEMBRÉ almost 10 years ago
- Target version changed from 2.11.7 to 2.11.8
Updated by Vincent MEMBRÉ almost 10 years ago
- Target version changed from 2.11.8 to 2.11.9
Updated by Vincent MEMBRÉ over 9 years ago
- Target version changed from 2.11.9 to 2.11.10
Updated by Vincent MEMBRÉ over 9 years ago
- Target version changed from 2.11.10 to 2.11.11
Updated by Benoît PECCATTE over 9 years ago
- Assignee deleted (
Alexander Brunhirl)
Updated by Vincent MEMBRÉ over 9 years ago
- Target version changed from 2.11.11 to 2.11.12
Updated by Vincent MEMBRÉ over 9 years ago
- Target version changed from 2.11.12 to 2.11.13
Updated by Vincent MEMBRÉ over 9 years ago
- Target version changed from 2.11.13 to 2.11.14
Updated by Vincent MEMBRÉ about 9 years ago
- Target version changed from 2.11.14 to 2.11.15
Updated by Vincent MEMBRÉ about 9 years ago
- Target version changed from 2.11.15 to 2.11.16
Updated by Vincent MEMBRÉ about 9 years ago
- Target version changed from 2.11.16 to 2.11.17
Updated by Vincent MEMBRÉ almost 9 years ago
- Target version changed from 2.11.17 to 2.11.18
Updated by Vincent MEMBRÉ almost 9 years ago
- Target version changed from 2.11.18 to 2.11.19
Updated by Vincent MEMBRÉ almost 9 years ago
- Target version changed from 2.11.19 to 2.11.20
Updated by Vincent MEMBRÉ over 8 years ago
- Target version changed from 2.11.20 to 2.11.21
Updated by Vincent MEMBRÉ over 8 years ago
- Target version changed from 2.11.21 to 2.11.22
Updated by Vincent MEMBRÉ over 8 years ago
- Target version changed from 2.11.22 to 2.11.23
Updated by Vincent MEMBRÉ over 8 years ago
- Target version changed from 2.11.23 to 2.11.24
Updated by Vincent MEMBRÉ over 8 years ago
- Target version changed from 2.11.24 to 308
Updated by Vincent MEMBRÉ about 8 years ago
- Target version changed from 308 to 3.1.14
Updated by Vincent MEMBRÉ about 8 years ago
- Target version changed from 3.1.14 to 3.1.15
Updated by Vincent MEMBRÉ about 8 years ago
- Target version changed from 3.1.15 to 3.1.16
Updated by Vincent MEMBRÉ about 8 years ago
- Target version changed from 3.1.16 to 3.1.17
Updated by Vincent MEMBRÉ about 8 years ago
- Target version changed from 3.1.17 to 3.1.18
Updated by Vincent MEMBRÉ almost 8 years ago
- Target version changed from 3.1.18 to 3.1.19
Updated by Jonathan CLARKE over 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
Updated by Vincent MEMBRÉ over 7 years ago
- Target version changed from 3.1.19 to 3.1.20
- Priority set to 0
Updated by Vincent MEMBRÉ over 7 years ago
- Target version changed from 3.1.20 to 3.1.21
Updated by Vincent MEMBRÉ over 7 years ago
- Target version changed from 3.1.21 to 3.1.22
Updated by Benoît PECCATTE over 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.
Updated by Benoît PECCATTE over 7 years ago
- Effort required set to Small
- Priority changed from 42 to 45
Updated by Vincent MEMBRÉ over 7 years ago
- Target version changed from 3.1.22 to 3.1.23
Updated by Vincent MEMBRÉ over 7 years ago
- Target version changed from 3.1.23 to 3.1.24
Updated by Vincent MEMBRÉ about 7 years ago
- Target version changed from 3.1.24 to 3.1.25
Updated by Vincent MEMBRÉ almost 7 years ago
- Target version changed from 3.1.25 to 387
Updated by Alexis Mousset almost 7 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
Updated by Vincent MEMBRÉ almost 7 years ago
- Target version changed from 387 to 4.1.10
Updated by Vincent MEMBRÉ almost 7 years ago
- Target version changed from 4.1.10 to 4.1.11
- Priority changed from 55 to 56
Updated by Vincent MEMBRÉ over 6 years ago
- Target version changed from 4.1.11 to 4.1.12
- Priority changed from 56 to 57
Updated by Vincent MEMBRÉ over 6 years ago
- Target version changed from 4.1.12 to 4.1.13
- Priority changed from 57 to 58
Updated by Vincent MEMBRÉ over 6 years ago
- Target version changed from 4.1.13 to 4.1.14
- Priority changed from 58 to 59
Updated by Nicolas CHARLES over 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
Updated by Alexander Brunhirl about 6 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
Updated by Nicolas CHARLES almost 6 years ago
- Translation missing: en.field_tag_list set to Sponsored