[389-ds-base] branch master updated (575d9e2 -> 0457ea6)

[git repository hosting]

This is an automated email from the git hooks/post-receive script.

firstyear pushed a change to branch master
in repository 389-ds-base.

from  575d9e2   Ticket 49413 - Changelog trimming ignores disabled replica-agreement
       new  0457ea6   Ticket 49474 - purge saslmaps before gssapi test

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "adds" were already present in the repository and have only
been added to this reference.

Summary of changes:
 src/lib389/lib389/topologies.py | 3 +++
 1 file changed, 3 insertions(+)

[389-ds-base] 01/01: Ticket 49474 - purge saslmaps before gssapi test

[git repository hosting]

This is an automated email from the git hooks/post-receive script.

firstyear pushed a commit to branch master
in repository 389-ds-base.

commit 0457ea62dcfaddfe50e5d0d0924524eb55f86a50
Author: William Brown 
Date:   Thu Jan 11 10:45:10 2018 +1000

Ticket 49474 - purge saslmaps before gssapi test

Bug Description:  In the GSSAPI test, depending on enviroment
    some of the default maps can "work" or they "may not". This is
    due to inconsistency in the behaviour of the cyrus-sasl
    library and it's assignment of realm into requests.

Fix Description:  Purge all maps by default, and just use the
    provided test sasl map - this is a single "source of truth"
    and thus allows us to test assertions easier.

https://pagure.io/389-ds-base/issue/49474

Author: wibrown

Review by: spichugi (Thank you!)
---
 src/lib389/lib389/topologies.py | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/src/lib389/lib389/topologies.py b/src/lib389/lib389/topologies.py
index ae64828..7e7bc0c 100644
--- a/src/lib389/lib389/topologies.py
+++ b/src/lib389/lib389/topologies.py
@@ -240,6 +240,9 @@ def topology_st_gssapi(request):

# Add sasl mappings
     saslmappings = SaslMappings(topology.standalone)
+    # First, purge all the default maps.
+    [m.delete() for m in saslmappings.list()]
+    # Now create a single map that works for our case.
     saslmappings.create(properties={
         'cn': 'suffix map',
         # Don't add the realm due to a SASL bug