after some testing, only one member of a group is being mapped, with LDAP as well as w

This is the critical part: <div class="snippet-clipboard-content notranslate posit

as stands of now we can use one of the follwoing: <div class="snippet-clipboard-co

nsscache doesn't map all members of a group about nsscache HOT 2 CLOSED

3c2b2ff5 commented on June 11, 2024

nsscache doesn't map all members of a group

from nsscache.

Comments (2)

3c2b2ff5 commented on June 11, 2024

This is the critical part:

      for record in data:
        # If the dn is requested, return it along with the payload,
        # otherwise ignore it.
        for key in record[1]:
          if isinstance(record[1][key][0], bytes) and key != 'objectSid':
            value = record[1][key][0].decode('utf-8')
            record[1][key] = [value]
        if self._dn_requested:
          merged_records = {'dn': record[0]}
          merged_records.update(record[1])
          yield merged_records
        else:
          yield record[1]

member list is > 1, so it converts the first member only, we need to check the length of the list, as for members it is not always 1, as for the other attributes. My solution would be:

      for record in data:
        # If the dn is requested, return it along with the payload,
        # otherwise ignore it.
        for key in record[1]:
          if len(record[1][key]) > 1:
            for i, item in enumerate(record[1][key]):
              if isinstance(item, bytes) and key != 'objectSid':
                value = item.decode('utf-8')
                record[1][key][i] = value
          else:
            if isinstance(record[1][key][0], bytes) and key != 'objectSid':
              value = record[1][key][0].decode('utf-8')
              record[1][key] = [value]
        if self._dn_requested:
          merged_records = {'dn': record[0]}
          merged_records.update(record[1])
          yield merged_records
        else:
          yield record[1]

I'll create a pull request, please modify as needed.

from nsscache.

3c2b2ff5 commented on June 11, 2024

as stands of now we can use one of the follwoing:

        for key in record[1]:
          for i, item in enumerate(record[1][key]):
            if isinstance(item, bytes) and key != 'objectSid':
              value = item.decode('utf-8')
              record[1][key][i] = value

or:

        for key in record[1]:
          for i in range(len(record[1][key])):
            if isinstance(record[1][key][i], bytes) and key != 'objectSid':
              value = record[1][key][i].decode('utf-8')
              record[1][key][i] = value

How would your suggestions work?

from nsscache.

Recommend Projects