Title: #365: PYTHON: Define constants as bytes instead of strings
@mzidek-rh: Yes, it's going to change the behaviour for code that uses
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
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