URL:
https://github.com/SSSD/sssd/pull/365
Title: #365: PYTHON: Define constants as bytes instead of strings
fidencio commented:
"""
@mzidek-rh: Yes, it's going to change the behaviour for code that uses
pysss_nss_idmap.SID_KEY.encode('utf-8').
Btw, there was some really internal discussion about this before having this PR opened
where IPA developers discussed about "(...) We can have a hack with SID_KEY =
pysss_nss_idmap.SID_KEY.encode('utf-8') or ask to fix in the python bindings.
(...)" and the answer was "(...) we'd better have them [sssd] fix this I
think (...)".
So, in general, it will change the behaviour but seems to be the right approach.
By the way, as I won't be able to touch this code for a week or so (PTO), whoever
review it, please, feel free to push to my branch (as I don't think we should delay
those patch too much).
"""
See the full comment at
https://github.com/SSSD/sssd/pull/365#issuecomment-326287436