URL:
https://github.com/SSSD/sssd/pull/369
Title: #369: pysss_nss_idmap: remove unnecessary python2/3 ifdef
sumit-bose commented:
"""
@lslebodn, I guess it was originally an error on my side to use PyBytes_FromString() when
generating the dicts and not using it for the constants.
Of course strings can be used, but I like bytes more because they better indicate that to
access the values not the string 'name' should be used but
pysss_nss_idmap.NAME_KEY. Ideally in
{'userlogin @domain.com': {b'sid':
'S-1-5-21-4185955025-1922848022-3738247038-1116', b'type': 3}}
b'sid' and b'type' should be returned as SID_KEY and TYPE_KEY
respectively, but I do not know if this is possible.
"""
See the full comment at
https://github.com/SSSD/sssd/pull/369#issuecomment-327223192