#1905 pysss_nss_idmap improvements
Closed: Fixed None Opened 7 years ago by abbra.

  1. Support tuple in addition to list as arguments. IPA Python code
    provides tuples to its methods when an argument is declared as list (+ or * suffix),
    to allow position-significant arguments.

  2. Support also Unicode strings and return them by default. Again, IPA
    Python code by default deals with Unicode strings in arguments (so, in
    upcoming 'trust-resolve' command '--sids' argument will give tuple of
    unicode strings to the Python method).

Additionally, returning unicode strings is important due to the fact
that user/group names are not limited to ASCII. More to that, Windows is
known to localise group and user names to target language, so that for
example, in Russian version a group with RID 498, "enterprise read-only
domain controllers" is actually "Контроллеры домена предприятия — только
чтение".


When this ticket is implemented and the interface stabilizes, we should add some unit tests for the Python interface, too. The patches already include tests for the C API, but not the python wrappers.

There is already a patch for 1. on the list, so I'm assigning the ticket to Sumit.

I split 2. to a separate ticket - https://fedorahosted.org/sssd/ticket/1914

Fields changed

milestone: NEEDS_TRIAGE => SSSD 1.10 beta
rhbz: => 0

Fields changed

owner: somebody => sbose
patch: 0 => 1

resolution: => fixed
status: new => closed

Metadata Update from @abbra:
- Issue assigned to sbose
- Issue set to the milestone: SSSD 1.10 beta

3 years ago

SSSD is moving from Pagure to Github. This means that new issues and pull requests
will be accepted only in SSSD's github repository.

This issue has been cloned to Github and is available here:
- https://github.com/SSSD/sssd/issues/2947

If you want to receive further updates on the issue, please navigate to the github issue
and click on subscribe button.

Thank you for understanding. We apologize for all inconvenience.

Login to comment on this ticket.

Metadata