URL: https://github.com/freeipa/freeipa/pull/1454
Author: tiran
 Title: #1454: [Backport][ipa-4-5] 10-config.update: remove 
nsslapd-sasl-max-buffer-size override as htt…
Action: opened

PR body:
"""
…ps://pagure.io/389-ds-base/issue/47457 was fixed directly in 389 Directory 
Server.

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 )"

Fixes: https://pagure.io/freeipa/issue/7341

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>
Reviewed-By: Florence Blanc-Renaud <fren...@redhat.com>

Manual backport of PR #1422 
"""

To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/1454/head:pr1454
git checkout pr1454
From 4b4d6116bfd98ac82f8594776e776aec06df8956 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 )"

Fixes: https://pagure.io/freeipa/issue/7341

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>
Reviewed-By: Florence Blanc-Renaud <fren...@redhat.com>
---
 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