URL: https://github.com/freeipa/freeipa/pull/761
Title: #761: Fixing adding authenticator indicators to host

pvoborni commented:
"""
I'd fix it on all places in host-mod:
```
  885                  raise errors.ACIError(info=msg)
  886              obj_classes = entry_attrs_old['objectclass']
  887:             if 'krbprincipalaux' not in obj_classes:
  888                  obj_classes.append('krbprincipalaux')
  889                  entry_attrs['objectclass'] = obj_classes
  ...
  921                  _entry_attrs = ldap.get_entry(dn, ['objectclass'])
  922                  obj_classes = _entry_attrs['objectclass']
  923:             if 'ieee802device' not in obj_classes:
  924                  obj_classes.append('ieee802device')
  925                  entry_attrs['objectclass'] = obj_classes
  ...
  941                  _entry_attrs = ldap.get_entry(dn, ['objectclass'])
  942                  obj_classes = entry_attrs['objectclass'] = 
_entry_attrs['objectclass']
  943:             if 'ipasshhost' not in obj_classes:
  944                  obj_classes.append('ipasshhost')
```

so that the plugin would be consistent. Rest of framework can be fixed other 
time.
"""

See the full comment at 
https://github.com/freeipa/freeipa/pull/761#issuecomment-299172235
-- 
Manage your subscription for the Freeipa-devel mailing list:
https://www.redhat.com/mailman/listinfo/freeipa-devel
Contribute to FreeIPA: http://www.freeipa.org/page/Contribute/Code

Reply via email to