#49887 Suffix for sasl fails in some cases during setup
Closed: wontfix 3 years ago Opened 3 years ago by firstyear.

Issue Description

``` # Get suffix for sasl map entries (template-sasl.ldif)
if len(backends) > 0:

      ds_suffix = backends[0]['suffix']

E KeyError: 'suffix'

/usr/lib/python3.6/site-packages/lib389-1.4.0.1-py3.6.egg/lib389/instance/setup.py:657: KeyError

This is with the basic test suite.

The issue is that there ARE no suffixes for this install, so it's not able to fix one at the index.

First, this seems to be incorrectly handled. I think it should be:

for be in backends:
# Do stuff

It also is incorrect as it's doing template replacement. Rather than template replacement, the backends should be iterated over near line 767 and then:

for backend in backends:
saslmap = SaslMap() ...
```

There are lib389 objects for this. We should never be doing template and text substitution ever again in the installer. Even dse.ldif is an odd one out that should be removed eventually.


See also src/lib389/lib389/saslmap.py

Metadata Update from @firstyear:
- Custom field component adjusted to None
- Custom field origin adjusted to None
- Custom field reviewstatus adjusted to None
- Custom field type adjusted to None
- Custom field version adjusted to None

3 years ago

Metadata Update from @mreynolds:
- Issue set to the milestone: 1.4.0

3 years ago

Metadata Update from @spichugi:
- Custom field reviewstatus adjusted to ack (was: None)

3 years ago

Metadata Update from @firstyear:
- Custom field reviewstatus adjusted to None (was: ack)
- Issue close_status updated to: fixed
- Issue status updated to: Closed (was: Open)

3 years ago

@spichugi, should we record the related PR when then fix is merged (https://pagure.io/389-ds-base/pull-request/49920) ?
I only know 'git log --oneline' to retrieve the changes, is there an other way ?

389-ds-base is moving from Pagure to Github. This means that new issues and pull requests
will be accepted only in 389-ds-base's github repository.

This issue has been cloned to Github and is available here:
- https://github.com/389ds/389-ds-base/issues/2946

If you want to receive further updates on the issue, please navigate to the github issue
and click on subscribe button.

Thank you for understanding. We apologize for all inconvenience.

Metadata Update from @spichugi:
- Issue close_status updated to: wontfix (was: fixed)

2 years ago

Login to comment on this ticket.

Metadata