FabInfra-Mirror/tools.fabaccess-users-toml-validator
1
0
Fork 0
mirror of https://github.com/vmario89/fabaccess-users-toml-validator.git synced 2025-03-12 06:51:42 +01:00
Code Issues Packages Projects Releases Wiki Activity

add check for data types and duplicates of passwords/cardkeys

Browse Source
This commit is contained in:
Mario Voigt 2024-12-26 14:30:35 +01:00
parent eef619899d
commit bd9bfe28fe
1 changed files with 29 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

32
validate.py
View File

@ -46,8 +46,10 @@ def main():
countPassword = 0
countPasswordUnencrypted = 0
countPasswordEncrypted = 0
countPasswordDuplicates = 0
countCardkey = 0
countCardkeyInvalid = 0
countCardkeyDuplicates = 0
countUnknownKeys = 0
countWarnings = 0
@ -82,6 +84,9 @@ def main():
print(str(e))
sys.exit(1)
passwds = []
cardkeys = []
for user in data:
print("--- {}".format(user))
@ -107,19 +112,36 @@ def main():
if "passwd" in data[user]:
passwd = data[user]["passwd"]
countPassword += 1
if passwd.startswith("$argon2") is False:
if type(passwd) != str:
print("Warning: password for user '{}' is not defined as string! BFFH will fail to load".format(user))
countWarnings += 1
elif passwd.startswith("$argon2") is False:
print("Warning: Password for user '{}' is not encrypted!".format(user))
countWarnings += 1
countPasswordUnencrypted += 1
else:
countPasswordEncrypted += 1
if passwd in passwds:
print("Warning: password for user '{}' is already in use by other user(s). That might be insecure".format(user))
countPasswordDuplicates += 1
countWarnings += 1
passwds.append(passwd)
if "cardkey" in data[user]:
cardkey = data[user]["cardkey"]
if is_valid_uuid(cardkey) is False:
print("Warning: Cardkey for user '{}' contains invalid cardkey (no UUID v4)".format(user))
if type(passwd) != str:
print("Warning: cardkey for user '{}' is not defined as string! BFFH will fail to load".format(user))
countWarnings += 1
elif is_valid_uuid(cardkey) is False:
print("Warning: cardkey for user '{}' contains invalid cardkey (no UUID v4)".format(user))
countCardkeyInvalid += 1
countWarnings += 1
if cardkey in cardkeys:
print("Warning: cardkey for user '{}' is already in use by other user(s). That might be insecure".format(user))
countCardkeyDuplicates += 1
countWarnings += 1
cardkeys.append(cardkey)
countCardkey += 1
@ -138,8 +160,8 @@ def main():
print("{} Database statistics {}\n".format("*"*25, "*"*25))
print("- Total users: {}".format(countUsers))
print("- Total unique roles: {}".format(len(uniqueRoles)))
print("- Total passwords: {} (encrypted: {}, unencrypted: {})".format(countPassword, countPasswordEncrypted, countPasswordUnencrypted))
print("- Total cardkeys: {}".format(countCardkey))
print("- Total passwords: {} (encrypted: {}, unencrypted: {}, duplicates: {})".format(countPassword, countPasswordEncrypted, countPasswordUnencrypted, countPasswordDuplicates))
print("- Total cardkeys: {} (duplicates: {})".format(countCardkey, countCardkeyDuplicates))
print("\n")