Skip to content

Conversation

@mrc0mmand
Copy link
Member

@mrc0mmand mrc0mmand commented Oct 9, 2025

Resolves: RHEL-109902

(cherry picked from commit 5072f4268b89a71e47e59c434da0222f722c7f0e)

Resolves: RHEL-109902
(cherry picked from commit 30de5691744781277f992a25afa268518f3fe711)

Resolves: RHEL-109902
… record

This ensures that user names can be specified either in the regular
short syntax or with a realm appended, and both are accepted. (The
latter of course only if the record actually defines a realm)

(cherry picked from commit 8aacf0fee1a8e9503bc071d5557293b0f3af50a4)

Resolves: RHEL-109902
(cherry picked from commit e2e1f38f5a9d442d0a027986024f4ea75ce97d2f)

Resolves: RHEL-109902
(cherry picked from commit 1fb53bb561db11e72bb695d578f8e94042565822)

Resolves: RHEL-109902
…alms

This in particular makes sure that we normalize the user name and update
it in the PAM session, once we acquire it. This means that if you have a
user with name "a" and alias "b", and the user logs in as "b" they end
up properly with "a" as user name set, as intended by the PAM gods.

Moreover, if you have a user "c" in a ralm "d", they may log in by
specifying "c" or "c@d", with equivalent results.

(cherry picked from commit a642f9d2d37945917bf4200a1095a43e6e7b6ea7)

Resolves: RHEL-109902
(cherry picked from commit 40fd0e0423ed9e4ae47d7f1adba83c0487d7decd)

Resolves: RHEL-109902
(cherry picked from commit 5cd7b455e0b2ee5991ff06a885c8bc4fe78c2225)

Resolves: RHEL-109902
(cherry picked from commit 853e9b754a16c58f9fb147376af941ec679e65a6)

Resolves: RHEL-109902
(cherry picked from commit 3d3f27cd9a9d7ba325f0d6acc8526c86b938f09c)

Resolves: RHEL-109902
@github-actions github-actions bot changed the title user-record/home: add concept of alias names for accounts (RHEL-109902) user-record/home: add concept of alias names for accounts Oct 9, 2025
@github-actions
Copy link

github-actions bot commented Oct 9, 2025

Commit validation

Tracker - RHEL-109902

The following commits meet all requirements

commit upstream
342ca91 - strv: add strv_equal_ignore_order() helper systemd/systemd@5072f42
26015b1 - pam: minor coding style tweaks systemd/systemd@30de569
13309b2 - user-record: add helper that checks if a provided user name matches a … systemd/systemd@8aacf0f
864c038 - user-record: add support for alias user names to user record systemd/systemd@e2e1f38
d032a05 - pam_systemd_home: use right field name in error message systemd/systemd@1fb53bb
aef94d1 - pam_systemd_home: support login with alias names + user names with rea… systemd/systemd@a642f9d
5d59365 - homed: support user record aliases systemd/systemd@40fd0e0
fc5963f - homectl: add support for creating users with alias names systemd/systemd@5cd7b45
9edc9cc - test: add test for homed alias and realm user resolution systemd/systemd@853e9b7
34ad5de - update TODO systemd/systemd@3d3f27c

Follow-up detection

Follow-ups

commit follow-up
342ca91 - strv: add strv_equal_ignore_order() helper systemd/systemd@3f23b37
systemd/systemd@c1e0518
26015b1 - pam: minor coding style tweaks systemd/systemd@1970bf7
864c038 - user-record: add support for alias user names to user record systemd/systemd@170d7d4

Success

🟠 Mentions, Follow-ups and Revert commits - Waived


Tracker validation

Success

🟢 Tracker RHEL-109902 has set desired product: rhel-10.2
🟢 Tracker RHEL-109902 has set desired component: systemd
🟢 Tracker RHEL-109902 has been approved
🟢 Tracker RHEL-109902 has set severity


Pull Request validation

Success

🟡 CI - Waived
🟢 Review - Reviewed by a member
🟢 Approval - Changes were approved


Auto Merge

Success

🟢 Pull Request is not marked as draft and it's not blocked by dont-merge label
🟢 Pull Request meet requirements, title has correct form
🟢 Pull Request meet requirements, mergeable is true
🟠 Pull Request meet requirements, mergeable_state is unstable
🟢 Pull Request has correct target branch main
🟢 Pull Request was merged

Copy link
Member

@dtardon dtardon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@dtardon
Copy link
Member

dtardon commented Oct 17, 2025

The follow-ups are trivial except for systemd/systemd@170d7d4, but even that is more like nice-to-have (and not directly relevant to the issue that's being fixed).

@mrc0mmand
Copy link
Member Author

mrc0mmand commented Oct 17, 2025

The CI fail is unrelated (https://issues.redhat.com/browse/RHEL-119135).

@github-actions github-actions bot merged commit 536eac2 into redhat-plumbers:main Oct 17, 2025
29 of 30 checks passed
@mrc0mmand mrc0mmand deleted the RHEL-109902-userdbd branch December 17, 2025 10:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants