Merge branch 'bugfix/fix-def-pass-len-check' into 'dev'

Fix default password min length check, crashing crafty on fresh install

See merge request crafty-controller/crafty-4!725
This commit is contained in:
Iain Powrie 2024-03-07 23:49:26 +00:00
commit d9311455eb
2 changed files with 4 additions and 2 deletions

View File

@ -10,7 +10,7 @@ TBD
- Do not allow users at server limit to clone servers ([Merge Request](https://gitlab.com/crafty-controller/crafty-4/-/merge_requests/718))
- Fix bug where you cannot get to config with unloaded server ([Commit](https://gitlab.com/crafty-controller/crafty-4/-/commit/9de08973b6bb2ddf91283c5c6b0e189ff34f7e24))
- Fix forge install v1.20, 1.20.1 and 1.20.2 ([Merge Request](https://gitlab.com/crafty-controller/crafty-4/-/merge_requests/710))
- Fix Sanitisation on Passwords ([Merge Request](https://gitlab.com/crafty-controller/crafty-4/-/merge_requests/715))
- Fix Sanitisation on Passwords ([Merge Request](https://gitlab.com/crafty-controller/crafty-4/-/merge_requests/715) | [Merge Request](https://gitlab.com/crafty-controller/crafty-4/-/merge_requests/725))
- Fix `Upload Imports` on unix systems, that have a space in the root dir name ([Merge Request](https://gitlab.com/crafty-controller/crafty-4/-/merge_requests/722))
- Fix Bedrock downloads, add `www` to download URL ([Merge Request](https://gitlab.com/crafty-controller/crafty-4/-/merge_requests/723))
### Tweaks

View File

@ -24,7 +24,9 @@ class DatabaseBuilder:
"by Crafty. Find it in app/config/default-creds.txt"
)
username = default_data.get("username", "admin")
if self.helper.minimum_password_length > default_data.get("password", password):
if self.helper.minimum_password_length > len(
default_data.get("password", password)
):
Console.critical(
"Default password too short"
" using Crafty's created default."