Skip to content

Commit 890eb56

Browse files
committed
Fix mfa_enrollment for AUTHENTICATION_POLICY
1 parent f8d2556 commit 890eb56

File tree

5 files changed

+7
-4
lines changed

5 files changed

+7
-4
lines changed

CHANGELOG.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,9 @@
11
# Changelog
22

3+
## [0.59.1] - 2025-10-25
4+
5+
- Fixed issue with optional `mfa_enrollment` parameter for `AUTHENTICATION_POLICY`.
6+
37
## [0.59.0] - 2025-10-24
48

59
- All `AUTHENTICATION_POLICY` parameters are now optional.

snowddl/parser/authentication_policy.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -75,7 +75,7 @@ def process_authentication_policy(self, f: ParsedFile):
7575
full_name=SchemaObjectIdent(self.env_prefix, f.database, f.schema, f.name),
7676
authentication_methods=self.normalise_params_list(f.params.get("authentication_methods")),
7777
mfa_authentication_methods=self.normalise_params_list(f.params.get("mfa_authentication_methods")),
78-
mfa_enrollment=f.params.get("mfa_enrollment").upper(),
78+
mfa_enrollment=f.params.get("mfa_enrollment").upper() if f.params.get("mfa_enrollment") else None,
7979
mfa_policy=self.normalise_params_dict(f.params.get("mfa_policy")),
8080
client_types=self.normalise_params_list(f.params.get("client_types")),
8181
security_integrations=self.normalise_params_list(f.params.get("security_integrations")),

snowddl/version.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
__version__ = "0.59.0"
1+
__version__ = "0.59.1"
Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,2 +1 @@
11
authentication_methods: [SAML, KEYPAIR]
2-
mfa_enrollment: REQUIRED_PASSWORD_ONLY

test/authentication_policy/aup003.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ def test_step2(helper):
2020
params = helper.desc_authentication_policy("db1", "sc1", "aup003_aup1")
2121

2222
assert params["AUTHENTICATION_METHODS"]["value"] == "[SAML, KEYPAIR]"
23-
assert params["MFA_ENROLLMENT"]["value"] == "REQUIRED_PASSWORD_ONLY"
23+
assert params["MFA_ENROLLMENT"]["value"] == "REQUIRED_SNOWFLAKE_UI_PASSWORD_ONLY"
2424

2525
assert "ALLOWED_METHODS=[ALL]" in params["MFA_POLICY"]["value"]
2626

0 commit comments

Comments
 (0)