Rudder plugins » User management

We ignore user from /opt/rudder/etc/rudder-users.xml that are not valid: for example, the one missing a password field.

If we have such an user, we can still add an user with the same name from the plugin UI. Then, there is two users with the same name.
It does not seems to create any problem, but it would be easy to not understand what is happening by looking at the file, and perhaps modify the incorrect entry.

It would be better to at least do an error in that case (but the error would be hard to understand for an user, and not really actionnable) or better, to replace the incorrect user entry by the newly created correct one.

If we delete the user with the duplicate entry, both entries are deleted.

Ex:

Initial content:
<authentication ...>  
  ...
  <user role="admin" name="test"/>     <- this entry is invalid and will be ignore by rudder: the plugin does not display it.
</authentication>

From UI, create a "test" user, new file content: 
<authentication ...>  
  ...
  <user role="admin" name="test"/>     <- this entry is invalid and will be ignore by rudder: the plugin does not display it.
  ...
  <user role="admin" name="test" password="..." />
</authentication>

From UI, delete user "test": both entries are deleted.