Project

General

Profile

Bug #6395

Technique userManagement does not change shell

Added by François ARMAND almost 4 years ago. Updated 5 months ago.

Status:
Released
Priority:
3
Category:
Techniques
Target version:
Severity:
Major - prevents use of part of Rudder | no simple workaround
User visibility:
Getting started - demo | first install | level 1 Techniques
Effort required:
Small
Priority:
74

Description

In technique userManagement v6, if we have an user added with (for example) /bin/bash in the field " Shell for this account: ", and that we latter change the value to "/bin/zsh" (because we find back our sanity), the shell for the poor user is not updated and he is forced to stay with a bad one.

Perhaps the shell is only set at user creation, but if so, it is not very clear, and certainly not one wants (certainly not the poor user forced to stay with bash).


Subtasks

Bug #12892: Same as parent ticket for branch 4.3ReleasedAlexis MOUSSET

Associated revisions

Revision c137d086 (diff)
Added by Nicolas CHARLES 5 months ago

Fixes #6395: Technique userManagement does not change shell

History

#1 Updated by François ARMAND almost 4 years ago

  • Reproduced changed from No to Yes
  • Found in version(s) old 3.0.2 added

#2 Updated by Matthieu CERDA almost 4 years ago

  • Subject changed from Techniqye userManagement does not change shell to Technique userManagement does not change shell

#3 Updated by Vincent MEMBRÉ over 3 years ago

  • Target version changed from 2.10.12 to 2.10.13

#4 Updated by Vincent MEMBRÉ over 3 years ago

  • Target version changed from 2.10.13 to 2.10.14

#5 Updated by Vincent MEMBRÉ over 3 years ago

  • Target version changed from 2.10.14 to 2.10.15

#6 Updated by Vincent MEMBRÉ over 3 years ago

  • Target version changed from 2.10.15 to 2.10.16

#7 Updated by Florian Dhomps over 3 years ago

Hi

On Debian 7&8 Rudder 3.0.7.

" Shell for this account: " work at creation only.

NCF Technique "Command execution" with "chsh -s /bin/sh user" return repaired everytime :-s. (work on Debian 8 not on 7)

You have a better solution?
Thx

#8 Updated by Vincent MEMBRÉ over 3 years ago

  • Target version changed from 2.10.16 to 2.10.17

#9 Updated by Vincent MEMBRÉ about 3 years ago

  • Target version changed from 2.10.17 to 2.10.18

#10 Updated by Vincent MEMBRÉ about 3 years ago

  • Target version changed from 2.10.18 to 2.10.19

#11 Updated by Vincent MEMBRÉ about 3 years ago

  • Target version changed from 2.10.19 to 2.10.20

#12 Updated by Vincent MEMBRÉ about 3 years ago

  • Target version changed from 2.10.20 to 277

#13 Updated by Vincent MEMBRÉ about 3 years ago

  • Target version changed from 277 to 2.11.18

#14 Updated by Vincent MEMBRÉ almost 3 years ago

  • Target version changed from 2.11.18 to 2.11.19

#15 Updated by Vincent MEMBRÉ almost 3 years ago

  • Target version changed from 2.11.19 to 2.11.20

#16 Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 2.11.20 to 2.11.21

#17 Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 2.11.21 to 2.11.22

#18 Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 2.11.22 to 2.11.23

#19 Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 2.11.23 to 2.11.24

#20 Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 2.11.24 to 308

#21 Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 308 to 3.1.14

#22 Updated by Vincent MEMBRÉ about 2 years ago

  • Target version changed from 3.1.14 to 3.1.15

#23 Updated by Vincent MEMBRÉ about 2 years ago

  • Target version changed from 3.1.15 to 3.1.16

#24 Updated by Vincent MEMBRÉ about 2 years ago

  • Target version changed from 3.1.16 to 3.1.17

#25 Updated by Vincent MEMBRÉ about 2 years ago

  • Target version changed from 3.1.17 to 3.1.18

#26 Updated by Vincent MEMBRÉ almost 2 years ago

  • Target version changed from 3.1.18 to 3.1.19

#27 Updated by Jonathan CLARKE over 1 year ago

  • Severity set to Major - prevents use of part of Rudder | no simple workaround
  • User visibility set to Getting started - demo | first install | level 1 Techniques

#28 Updated by Benoît PECCATTE over 1 year ago

  • Priority set to 45

#29 Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 3.1.19 to 3.1.20

#30 Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 3.1.20 to 3.1.21

#31 Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 3.1.21 to 3.1.22

#32 Updated by Benoît PECCATTE over 1 year ago

  • Priority changed from 45 to 58

#33 Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 3.1.22 to 3.1.23

#34 Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 3.1.23 to 3.1.24

#35 Updated by Vincent MEMBRÉ about 1 year ago

  • Target version changed from 3.1.24 to 3.1.25

#36 Updated by Vincent MEMBRÉ about 1 year ago

  • Target version changed from 3.1.25 to 387

#37 Updated by Nicolas CHARLES 12 months ago

it is still the case in version 8 of the technique

#38 Updated by Vincent MEMBRÉ 12 months ago

  • Target version changed from 387 to 4.1.10

#39 Updated by Vincent MEMBRÉ 10 months ago

  • Target version changed from 4.1.10 to 4.1.11

#40 Updated by Vincent MEMBRÉ 8 months ago

  • Target version changed from 4.1.11 to 4.1.12
  • Priority changed from 58 to 59

#41 Updated by Vincent MEMBRÉ 7 months ago

  • Target version changed from 4.1.12 to 4.1.13

#42 Updated by Vincent MEMBRÉ 6 months ago

  • Assignee set to Nicolas CHARLES
  • Effort required set to Small
  • Priority changed from 59 to 74

#43 Updated by Nicolas CHARLES 6 months ago

Code doesn't do anything about shell if user already exists - we could do the same as for fullname

#44 Updated by Nicolas CHARLES 6 months ago

  • Status changed from New to In progress

#45 Updated by Nicolas CHARLES 5 months ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from Nicolas CHARLES to Alexis MOUSSET
  • Pull Request set to https://github.com/Normation/rudder-techniques/pull/1326

#46 Updated by Nicolas CHARLES 5 months ago

  • Status changed from Pending technical review to Pending release

#47 Updated by Vincent MEMBRÉ 5 months ago

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 4.1.13, 4.2.7 and 4.3.3 which were released today.

Also available in: Atom PDF