URL: https://github.com/freeipa/freeipa/pull/1422
Author: fcami
 Title: #1422: remove nsslapd-sasl-max-buffer-size override from 
10-config.update
Action: opened

PR body:
"""
Since https://pagure.io/389-ds-base/issue/47457 was fixed directly in 389 
Directory Server, overriding  nsslapd-sasl-max-buffer-size at install time is 
not necessary anymore and it breaks overriding its value with 
--dirsrv-config-file.

The patch addresses:
https://bugzilla.redhat.com/show_bug.cgi?id=1527020
"nsslapd-sasl-max-buffer-size is hardcoded to '2097152' during
install even if another value was provided in an LDIF
( --dirsrv-config-file )"

Tested against RHEL 7.4, the nsslapd-sasl-max-buffer-size parameter
is still 2097152 after this change and the change allows overriding
its value using --dirsrv-config-file properly.

Fix suggested by Florence Blanc-Renaud.

Signed-off-by: François Cami <fc...@fedoraproject.org>
"""

To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/1422/head:pr1422
git checkout pr1422
From 802ed23f47b0cda49eded3582b406b37d4211b73 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Fran=C3=A7ois=20Cami?= <fc...@fedoraproject.org>
Date: Tue, 26 Dec 2017 16:05:22 +0100
Subject: [PATCH] 10-config.update: remove nsslapd-sasl-max-buffer-size
 override as https://pagure.io/389-ds-base/issue/47457 was fixed directly in
 389 Directory Server.
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

The patch addresses:
https://bugzilla.redhat.com/show_bug.cgi?id=1527020
"nsslapd-sasl-max-buffer-size is hardcoded to '2097152' during
install even if another value was provided in an LDIF
( --dirsrv-config-file )"

Tested against RHEL 7.4, the nsslapd-sasl-max-buffer-size parameter
is still 2097152 after this change and the change allows overriding
its value using --dirsrv-config-file properly.

Fix suggested by Florence Blanc-Renaud.

Signed-off-by: François Cami <fc...@fedoraproject.org>
---
 install/updates/10-config.update | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/install/updates/10-config.update b/install/updates/10-config.update
index f725bbfdcd..b24bf60319 100644
--- a/install/updates/10-config.update
+++ b/install/updates/10-config.update
@@ -58,12 +58,6 @@ addifnew:nsSaslMapPriority: 10
 dn: cn=Name Only,cn=mapping,cn=sasl,cn=config
 addifnew:nsSaslMapPriority: 10
 
-# Default SASL buffer size was too small and could lead for example to
-# migration errors
-# Can be removed when https://fedorahosted.org/389/ticket/47457 is fixed
-dn: cn=config
-only:nsslapd-sasl-max-buffer-size:2097152
-
 # Allow hashed passwords to be added by non-DM users. Without this
 # setting, password migration fails
 dn: cn=config
_______________________________________________
FreeIPA-devel mailing list -- freeipa-devel@lists.fedorahosted.org
To unsubscribe send an email to freeipa-devel-le...@lists.fedorahosted.org

Reply via email to