Project

General

Profile

Actions

Bug #16296

closed

Updating informations by submitting empty fields invalidates user's loggin

Added by Elaad FURREEDAN over 4 years ago. Updated about 4 years ago.

Status:
Released
Priority:
N/A
Target version:
Severity:
UX impact:
User visibility:
Effort required:
Priority:
0
Name check:
To do
Fix check:
To do
Regression:

Description

When I try to update informations by submitting empty field(s) in input boxes, invalidate informations : It replace and update in file the empty field by the correspondig field from admin informations
Exemple :

Initial state

Username : admin
Password : passadmin
roles : administrator

Username : toto
Password : 123badpasswd
roles : inventory

-> update user "toto" : change username "toto" to "titi" and keep password empty

Final state :
Username : admin
Password : passadmin
roles : administrator

Username : titi
Password : passadmin <------- ERROR
roles : inventory

Actions #1

Updated by Elaad FURREEDAN over 4 years ago

  • Target version set to 6.0-1.4
Actions #2

Updated by Elaad FURREEDAN over 4 years ago

  • Status changed from New to In progress
  • Assignee set to Elaad FURREEDAN
Actions #3

Updated by Vincent MEMBRÉ over 4 years ago

  • Target version changed from 6.0-1.4 to 6.0-1.5
Actions #4

Updated by Elaad FURREEDAN over 4 years ago

  • Target version changed from 6.0-1.5 to 6.0-1.6
Actions #5

Updated by Elaad FURREEDAN over 4 years ago

The bug comes when we save authentification informations, in Firefox in my case. When we add an user, the two inputs fields was autofilled with Firefox's autocompletion for logins. The user will overwrites the autocompleted fields and will be created successfully. But when he try to update his username or password in the right menu, if he leaves an empty field, the autocompletion overwrite it, so the user will have an username or password swaped (according on which input he left empty) with the saved loggin in Firefox on this URL.

Actions #6

Updated by Elaad FURREEDAN over 4 years ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from Elaad FURREEDAN to Vincent MEMBRÉ
  • Pull Request set to https://github.com/Normation/rudder-plugins/pull/224
Actions #7

Updated by Anonymous over 4 years ago

  • Status changed from Pending technical review to Pending release
Actions #8

Updated by Vincent MEMBRÉ about 4 years ago

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 6.0-1.6 which was released today.

Actions

Also available in: Atom PDF