|
 |
c8cd81 |
From de6f074538f6641fd9d84bed204a3d4d50eccbe5 Mon Sep 17 00:00:00 2001
|
|
|
c8cd81 |
From: Rob Crittenden <rcritten@redhat.com>
|
|
|
c8cd81 |
Date: Thu, 4 Aug 2022 12:04:41 -0400
|
|
|
c8cd81 |
Subject: [PATCH] Set default on group pwpolicy with no grace limit in upgrade
|
|
|
c8cd81 |
|
|
|
c8cd81 |
If an existing group policy lacks a password grace limit
|
|
|
c8cd81 |
update it to -1 on upgrade.
|
|
|
c8cd81 |
|
|
|
c8cd81 |
Fixes: https://pagure.io/freeipa/issue/9212
|
|
|
c8cd81 |
|
|
|
c8cd81 |
Signed-off-by: Rob Crittenden <rcritten@redhat.com>
|
|
|
c8cd81 |
Reviewed-By: Florence Blanc-Renaud <frenaud@redhat.com>
|
|
|
c8cd81 |
---
|
|
|
c8cd81 |
.../updates/90-post_upgrade_plugins.update | 1 +
|
|
|
c8cd81 |
ipaserver/install/plugins/update_pwpolicy.py | 66 +++++++++++++++++++
|
|
|
c8cd81 |
2 files changed, 67 insertions(+)
|
|
|
c8cd81 |
|
|
|
c8cd81 |
diff --git a/install/updates/90-post_upgrade_plugins.update b/install/updates/90-post_upgrade_plugins.update
|
|
|
c8cd81 |
index c7ec71d492b0ac0e7641d586b7e7fa7501743bc2..6fe91aa6c6310a69a7f0feb1ad62243945db67f9 100644
|
|
|
c8cd81 |
--- a/install/updates/90-post_upgrade_plugins.update
|
|
|
c8cd81 |
+++ b/install/updates/90-post_upgrade_plugins.update
|
|
|
c8cd81 |
@@ -26,6 +26,7 @@ plugin: update_ra_cert_store
|
|
|
c8cd81 |
plugin: update_mapping_Guests_to_nobody
|
|
|
c8cd81 |
plugin: fix_kra_people_entry
|
|
|
c8cd81 |
plugin: update_pwpolicy
|
|
|
c8cd81 |
+plugin: update_pwpolicy_grace
|
|
|
c8cd81 |
|
|
|
c8cd81 |
# last
|
|
|
c8cd81 |
# DNS version 1
|
|
|
c8cd81 |
diff --git a/ipaserver/install/plugins/update_pwpolicy.py b/ipaserver/install/plugins/update_pwpolicy.py
|
|
|
c8cd81 |
index dca44ce4369dfc11f83a412a1249bb045d46713f..4185f034313bd49ca68e86c620043af6ead5f6d6 100644
|
|
|
c8cd81 |
--- a/ipaserver/install/plugins/update_pwpolicy.py
|
|
|
c8cd81 |
+++ b/ipaserver/install/plugins/update_pwpolicy.py
|
|
|
c8cd81 |
@@ -78,3 +78,69 @@ class update_pwpolicy(Updater):
|
|
|
c8cd81 |
return False, []
|
|
|
c8cd81 |
|
|
|
c8cd81 |
return False, []
|
|
|
c8cd81 |
+
|
|
|
c8cd81 |
+
|
|
|
c8cd81 |
+@register()
|
|
|
c8cd81 |
+class update_pwpolicy_grace(Updater):
|
|
|
c8cd81 |
+ """
|
|
|
c8cd81 |
+ Ensure all group policies have a grace period set.
|
|
|
c8cd81 |
+ """
|
|
|
c8cd81 |
+
|
|
|
c8cd81 |
+ def execute(self, **options):
|
|
|
c8cd81 |
+ ldap = self.api.Backend.ldap2
|
|
|
c8cd81 |
+
|
|
|
c8cd81 |
+ base_dn = DN(('cn', self.api.env.realm), ('cn', 'kerberos'),
|
|
|
c8cd81 |
+ self.api.env.basedn)
|
|
|
c8cd81 |
+ search_filter = (
|
|
|
c8cd81 |
+ "(&(objectClass=krbpwdpolicy)(!(passwordgracelimit=*)))"
|
|
|
c8cd81 |
+ )
|
|
|
c8cd81 |
+
|
|
|
c8cd81 |
+ while True:
|
|
|
c8cd81 |
+ # Run the search in loop to avoid issues when LDAP limits are hit
|
|
|
c8cd81 |
+ # during update
|
|
|
c8cd81 |
+
|
|
|
c8cd81 |
+ try:
|
|
|
c8cd81 |
+ (entries, truncated) = ldap.find_entries(
|
|
|
c8cd81 |
+ search_filter, ['objectclass'], base_dn, time_limit=0,
|
|
|
c8cd81 |
+ size_limit=0)
|
|
|
c8cd81 |
+
|
|
|
c8cd81 |
+ except errors.EmptyResult:
|
|
|
c8cd81 |
+ logger.debug("update_pwpolicy: no policies without "
|
|
|
c8cd81 |
+ "passwordgracelimit set")
|
|
|
c8cd81 |
+ return False, []
|
|
|
c8cd81 |
+
|
|
|
c8cd81 |
+ except errors.ExecutionError as e:
|
|
|
c8cd81 |
+ logger.error("update_pwpolicy: cannot retrieve list "
|
|
|
c8cd81 |
+ "of policies missing passwordgracelimit: %s", e)
|
|
|
c8cd81 |
+ return False, []
|
|
|
c8cd81 |
+
|
|
|
c8cd81 |
+ logger.debug("update_pwpolicy: found %d "
|
|
|
c8cd81 |
+ "policies to update, truncated: %s",
|
|
|
c8cd81 |
+ len(entries), truncated)
|
|
|
c8cd81 |
+
|
|
|
c8cd81 |
+ error = False
|
|
|
c8cd81 |
+
|
|
|
c8cd81 |
+ for entry in entries:
|
|
|
c8cd81 |
+ # Set unlimited BIND by default
|
|
|
c8cd81 |
+ entry['passwordgracelimit'] = -1
|
|
|
c8cd81 |
+ try:
|
|
|
c8cd81 |
+ ldap.update_entry(entry)
|
|
|
c8cd81 |
+ except (errors.EmptyModlist, errors.NotFound):
|
|
|
c8cd81 |
+ pass
|
|
|
c8cd81 |
+ except errors.ExecutionError as e:
|
|
|
c8cd81 |
+ logger.debug("update_pwpolicy: cannot "
|
|
|
c8cd81 |
+ "update policy: %s", e)
|
|
|
c8cd81 |
+ error = True
|
|
|
c8cd81 |
+
|
|
|
c8cd81 |
+ if error:
|
|
|
c8cd81 |
+ # Exit loop to avoid infinite cycles
|
|
|
c8cd81 |
+ logger.error("update_pwpolicy: error(s) "
|
|
|
c8cd81 |
+ "detected during pwpolicy update")
|
|
|
c8cd81 |
+ return False, []
|
|
|
c8cd81 |
+
|
|
|
c8cd81 |
+ elif not truncated:
|
|
|
c8cd81 |
+ # All affected entries updated, exit the loop
|
|
|
c8cd81 |
+ logger.debug("update_pwpolicy: all policies updated")
|
|
|
c8cd81 |
+ return False, []
|
|
|
c8cd81 |
+
|
|
|
c8cd81 |
+ return False, []
|
|
|
c8cd81 |
--
|
|
|
c8cd81 |
2.37.2
|
|
|
c8cd81 |
|