diff --git a/authentik/blueprints/tests/test_packaged.py b/authentik/blueprints/tests/test_packaged.py index 443173bac2..32d392447f 100644 --- a/authentik/blueprints/tests/test_packaged.py +++ b/authentik/blueprints/tests/test_packaged.py @@ -27,7 +27,8 @@ def blueprint_tester(file_name: Path) -> Callable: base = Path("blueprints/") rel_path = Path(file_name).relative_to(base) importer = Importer.from_string(BlueprintInstance(path=str(rel_path)).retrieve()) - self.assertTrue(importer.validate()[0]) + validation, logs = importer.validate() + self.assertTrue(validation, logs) self.assertTrue(importer.apply()) return tester diff --git a/authentik/policies/password/models.py b/authentik/policies/password/models.py index d8584617bc..ede76c9f3a 100644 --- a/authentik/policies/password/models.py +++ b/authentik/policies/password/models.py @@ -89,6 +89,10 @@ class PasswordPolicy(Policy): def passes_static(self, password: str, request: PolicyRequest) -> PolicyResult: """Check static rules""" + error_message = self.error_message + if error_message == "": + error_message = _("Invalid password.") + if len(password) < self.length_min: LOGGER.debug("password failed", check="static", reason="length") return PolicyResult(False, self.error_message) diff --git a/blueprints/default/flow-oobe.yaml b/blueprints/default/flow-oobe.yaml index 97b0258f10..5264cb7d9b 100644 --- a/blueprints/default/flow-oobe.yaml +++ b/blueprints/default/flow-oobe.yaml @@ -101,7 +101,6 @@ entries: - !KeyOf prompt-field-email - !KeyOf prompt-field-password - !KeyOf prompt-field-password-repeat - validation_policies: [] id: stage-default-oobe-password identifiers: name: stage-default-oobe-password diff --git a/blueprints/default/flow-password-change.yaml b/blueprints/default/flow-password-change.yaml index 2c0f8db217..31bb657c29 100644 --- a/blueprints/default/flow-password-change.yaml +++ b/blueprints/default/flow-password-change.yaml @@ -2,6 +2,17 @@ version: 1 metadata: name: Default - Password change flow entries: +- attrs: + check_static_rules: true + check_zxcvbn: true + length_min: 8 + password_field: password + zxcvbn_score_threshold: 2 + error_message: Password needs to be 8 characters or longer. + identifiers: + name: default-password-change-password-policy + model: authentik_policies_password.passwordpolicy + id: default-password-change-password-policy - attrs: designation: stage_configuration name: Change Password @@ -39,6 +50,8 @@ entries: fields: - !KeyOf prompt-field-password - !KeyOf prompt-field-password-repeat + validation_policies: + - !KeyOf default-password-change-password-policy identifiers: name: default-password-change-prompt id: default-password-change-prompt diff --git a/website/docs/customize/policies/index.md b/website/docs/customize/policies/index.md index 95ecff8ef3..39169ba9c6 100644 --- a/website/docs/customize/policies/index.md +++ b/website/docs/customize/policies/index.md @@ -32,6 +32,10 @@ This policy can enforce regular password rotation by expiring set passwords afte ### Password Policy +:::warning +By default, authentik's Password policy is compliant with [NIST's recommendations](https://pages.nist.gov/800-63-4/sp800-63b.html#password) for passwords. To remain compliant with NIST, be cautious when editing the default values. For additional hardening configuration settings, refer to [Hardening authentik](../../security/security-hardening.md#password-policy). +::: + This policy allows you to specify password rules, such as length and required characters. The following rules can be set: diff --git a/website/docs/security/security-hardening.md b/website/docs/security/security-hardening.md index 0c6bc18ee8..a705e4ca2d 100644 --- a/website/docs/security/security-hardening.md +++ b/website/docs/security/security-hardening.md @@ -4,6 +4,17 @@ title: Hardening authentik While authentik is secure out of the box, you can take steps to further increase the security of an authentik instance. As everyone knows, there is a consequential tradeoff between security and convenience. All of these hardening practices have an impact on the user experience and should only be applied knowing this tradeoff. +### Password policy + +authentik's default Password policy complies with the [NIST SP 800-63 Digital Identity Guidelines](https://pages.nist.gov/800-63-4/sp800-63b.html#password). + +However, for further hardening compliant to the NIST Guidelines, consider + +- setting the length of the password to a minimum of 15 characters, and +- enabling the "Check haveibeenpwned.com" blocklist comparison (note that this cannot be used on Air-gapped instances) + +For further options, see [Password policy](../customize/policies/index.md#password-policy). + ### Expressions [Expressions](../customize/policies/expression.mdx) allow super-users and other highly privileged users to create custom logic within authentik to modify its behaviour. Editing/creating these expressions is, by default, limited to super-users and any related events are fully logged.