andrewg
committed
on 08 Apr 05
Bugfix for an edge condition. User would get two notifications if: a) notify is set to true, b) password column exists but value was set to… Show more
Bugfix for an edge condition.  User would get two notifications if: a) notify is set to true, b) password column exists but value was set to empty.  The first notification, generated out of the local authentication driver, would contain the correct password.  The second notification, generated from this script, would contain an empty string as the password.  With the fix, we always generate the random password from this script.  A byproduct of this bugfix is that it's now possible for the user to get *no* notification even if a random password is generated.  This would happen if the password was empty or nonexistent and the notify column is set to false.  This seems undesirable but not necessarily worse than sending a notification when the CSV said not to.  And perhaps there are situations where the admin wants to create the accounts but not make them available yet.

Show less