To handle "Internal server error 500" save and load passwordHash to u…

…ser.yaml

passwordHash in user.yaml is updated after every successful check_password via ldap with forced save.

octoprint_auth_ldap/user_manager.py

@@ -137,6 +137,7 @@ def check_password(self, username, password):
                 self.logger.debug("%s was %sauthenticated" % (user.get_name(), "" if authenticated else "not "))
                 if authenticated:
                     user._passwordHash = LDAPUserManager.create_password_hash(password, settings=self._settings)
+                    self._save(force=True)
                 return authenticated
             else:
                 self.logger.debug("%s is inactive or no longer a member of required groups" % user.get_id())
@@ -187,6 +188,7 @@ def _load(self):
                         self.logger.debug("Loading %s as %s" % (name, LDAPUser.__name__))
                         self._users[name] = LDAPUser(
                             username=name,
+                            passwordHash=attributes["password"],
                             active=attributes["active"],
                             permissions=permissions,
                             groups=groups,
@@ -234,7 +236,7 @@ def _save(self, force=False):
                     # password field has to exist because of how FilebasedUserManager processes
                     # data, but an empty password hash cannot match any entered password (as
                     # whatever the user enters will be hashed... even an empty password.
-                    "password": None,
+                    "password": user._passwordHash,
 
                     "active": user._active,
                     "groups": self._from_groups(*user._groups),