nixos/pgadmin: apply review suggestions
Signed-off-by: Florian Brandes <florian.brandes@posteo.de>
This commit is contained in:
parent
010a6250db
commit
bc21d288f4
@ -44,13 +44,13 @@ in
|
||||
initialPasswordFile = mkOption {
|
||||
description = lib.mdDoc ''
|
||||
Initial password file for the pgAdmin account. Minimum length by default is 6.
|
||||
Please see services.pgadmin.passwordLength.
|
||||
Please see `services.pgadmin.minimumPasswordLength`.
|
||||
NOTE: Should be string not a store path, to prevent the password from being world readable
|
||||
'';
|
||||
type = types.path;
|
||||
};
|
||||
|
||||
passwordLength = mkOption {
|
||||
minimumPasswordLength = mkOption {
|
||||
description = lib.mdDoc "Minimum length of the password";
|
||||
type = types.int;
|
||||
default = 6;
|
||||
@ -122,7 +122,7 @@ in
|
||||
|
||||
services.pgadmin.settings = {
|
||||
DEFAULT_SERVER_PORT = cfg.port;
|
||||
PASSWORD_LENGTH_MIN = cfg.passwordLength;
|
||||
PASSWORD_LENGTH_MIN = cfg.minimumPasswordLength;
|
||||
SERVER_MODE = true;
|
||||
} // (optionalAttrs cfg.openFirewall {
|
||||
DEFAULT_SERVER = mkDefault "::";
|
||||
@ -151,9 +151,8 @@ in
|
||||
# and presenting a hard to find error message
|
||||
# see https://github.com/NixOS/nixpkgs/issues/270624
|
||||
PW_LENGTH=$(wc -m < ${escapeShellArg cfg.initialPasswordFile})
|
||||
if [ $PW_LENGTH -lt ${toString cfg.passwordLength} ]
|
||||
then
|
||||
echo "Password must be at least ${toString cfg.passwordLength} characters long"
|
||||
if [ $PW_LENGTH -lt ${toString cfg.minimumPasswordLength} ]; then
|
||||
echo "Password must be at least ${toString cfg.minimumPasswordLength} characters long"
|
||||
exit 1
|
||||
fi
|
||||
(
|
||||
|
@ -41,7 +41,7 @@ import ./make-test-python.nix ({ pkgs, lib, ... }:
|
||||
enable = true;
|
||||
initialEmail = "bruh@localhost.de";
|
||||
initialPasswordFile = pkgs.writeText "pw" "bruh2012!";
|
||||
passwordLength = 12;
|
||||
minimumPasswordLength = 12;
|
||||
};
|
||||
};
|
||||
};
|
||||
|
Loading…
Reference in New Issue
Block a user