-
Notifications
You must be signed in to change notification settings - Fork 183
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Draft: Parse SSSD group listings #3406
Conversation
The SSSD group listings breaks the present implementation by parsing the domain group as the file-size. Resolves: RedHatInsights#3395 Signed-off-by: Sachin Patil <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks like this update could not cover the other example mentioned in the issue #3395
# Example 2
-rw-r--r--. 1 root domain users 40893 Jan 31 16:34 Xorg.1.log
-rw-r--r--. 1 root domain users 36911 Dec 20 09:49 Xorg.1.log.old
In example-2, there is no "@" in the group name field "domain users".
And I have to agree that without this specific character "@", it is a difficult issue to solve based on the current "ls_parser" resolution. Maybe it's possible to consider another overall reconstruction plan, e.g. the insights.parsers.parse_fixed_table
.
It's just my thoughts, you may have a better resolution.
Thanks for pointing that out. I thought I included both the examples (which I did initially) but then I removed it to test each line individually. Finally I forgot to include the second example. My bad! I researched a bit and found that this strange user-group occurs when the sssd/krb5 DOMAIN/REALM is configured. From what I observed the owner is As to not break the current convention I am storing
Yeah! I agree that anything new in the |
Signed-off-by: Sachin Patil <[email protected]>
From this official guide document Adjusting how SSSD interprets full user names, it seems that
and
So, things become more complex ... maybe it's time to think of another resolution. |
Can one of the admins verify this patch? |
Closing due to it won't fix the issue. |
The SSSD group listings breaks the present implementation by parsing the domain
group as the file-size.
Resolves: #3395
Signed-off-by: Sachin Patil [email protected]
All Pull Requests:
Check all that apply:
Complete Description of Additions/Changes:
Add your description here