SSSD / sssd

#398 sss_showgroup member lists should be comma separated
Closed: Fixed Opened by jgalipea.

Closed: Fixed
Reopen Issue

Description[[BR]]
When showing a group that has multiple child group members, the list should be comma separated because group names can contain spaces. To be consistent the member users list should also be comma separated.[[BR]]
[[BR]]
EXAMPLE:[[BR]]

[root@sssd-f12-64 ~]# sss_groupshow "MY PARENT"
Group: MY PARENT
GID number: 3007
Member users: 
Member groups: MY GROUP MY GROUP2

Groups are "MY GROUP" and "MY GROUP2"

Fields changed

component: SSSD => sss_tools
owner: somebody => jhrozek

Fields changed

milestone: NEEDS_TRIAGE => SSSD 1.1
priority: major => minor

Is there any reason why you didn't use comma's to separate the list? It looks odd because the title and values are delimited with a colon.[[BR]]

Member groups: Engineering:Sales:

This would look better:[[BR]]

Member groups: Engineering,Sales

I honestly don't have an answer why I chose colon..but you are absolutely correct, patch 61f4ae23334073bb989469b8c9ef168389db24b3 fixed the behaviour to use commas.

fixedin: => 1.1.0
resolution: => fixed
status: new => closed

Fields changed

rhbz: => 0

Metadata Update from @jgalipea:
- Issue assigned to jhrozek
- Issue set to the milestone: SSSD 1.1

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/1440

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.
Metadata
None
None
None
High
defect
sss_tools
1.0.4
0
0
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.