Merge pull request from GHSA-4w46-w44m-3jq3

* strip password after authentication to prevent cleartext password storage

* fixed forgotten testcase forcing ;-/

* added test to check if password is not stored in user record

Co-authored-by: Fabian Strachanski <fabian@fastr.de>
This commit is contained in:
Antonio Davi Macedo Coelho de Castro
2020-12-02 13:08:02 -08:00
committed by GitHub
parent 4dee0bc61e
commit da905a357d
2 changed files with 47 additions and 0 deletions

View File

@@ -23,6 +23,7 @@ function validateAuthData(authData, options) {
return new Promise((resolve, reject) => {
client.bind(userCn, authData.password, ldapError => {
delete(authData.password);
if (ldapError) {
let error;
switch (ldapError.code) {