profiles: merge groups records with [SUCCESS=merge] by poettering · Pull Request #358 · authselect/authselect · GitHub
Skip to content

profiles: merge groups records with [SUCCESS=merge]#358

Merged
pbrezina merged 2 commits into
authselect:masterfrom
poettering:systemd-success-merge
Feb 23, 2024
Merged

profiles: merge groups records with [SUCCESS=merge]#358
pbrezina merged 2 commits into
authselect:masterfrom
poettering:systemd-success-merge

Conversation

@poettering

Copy link
Copy Markdown
Contributor

Services such as systemd-homed would like to advertise users which are part of system groups, such as "wheel". That only works if glibc's [SUCCESS=merge] feature is used in nsswitch.conf, so that group records from multiple sources are merged.

This is documented here:

https://www.freedesktop.org/software/systemd/man/latest/nss-systemd.html#Configuration%20in%20/etc/nsswitch.conf

This hence adds [SUCCESS=merge] expressions to all NSS modules listed in the "groups" lines.

@poettering poettering force-pushed the systemd-success-merge branch from a3c37de to f45baaf Compare December 14, 2023 13:24
@pbrezina

Copy link
Copy Markdown
Member

@pbrezina pbrezina force-pushed the systemd-success-merge branch from f45baaf to 4b49f3b Compare February 23, 2024 13:19
@pbrezina

Copy link
Copy Markdown
Member

Actually, I did this for you. I also added required spec file change in separate commit.

poettering and others added 2 commits February 23, 2024 14:23
Services such as systemd-homed would like to advertise users which are
part of system groups, such as "wheel". That only works if glibc's
[SUCCESS=merge] feature is used in nsswitch.conf, so that group records
from multiple sources are merged.

This is documented here:

https://www.freedesktop.org/software/systemd/man/latest/nss-systemd.html#Configuration%20in%20/etc/nsswitch.conf

This hence adds [SUCCESS=merge] expressions to all NSS modules listed in
the "groups" lines.
@pbrezina pbrezina force-pushed the systemd-success-merge branch from 4b49f3b to 90e11e8 Compare February 23, 2024 13:23
@pbrezina pbrezina merged commit 565d8a7 into authselect:master Feb 23, 2024
@poettering

Copy link
Copy Markdown
Contributor Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants