Skip to content
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

(#2210145) pam: add call to pam_umask #178

Merged

Conversation

jamacku
Copy link
Member

@jamacku jamacku commented Jun 28, 2023

Setting umask for user sessions via UMASK setting in /etc/login.defs is a well-known feature. Let's make sure that user manager also runs with this umask value.

Follow-up for 5e37d19.

(cherry picked from commit 159f1b78576ce91c3932f4867f07361a530875d3)

Resolves: #2210145

Setting umask for user sessions via UMASK setting in /etc/login.defs is
a well-known feature. Let's make sure that user manager also runs with
this umask value.

Follow-up for 5e37d19.

(cherry picked from commit 159f1b78576ce91c3932f4867f07361a530875d3)

Resolves: #2210145
@jamacku jamacku added this to the RHEL-9.3.0 milestone Jun 28, 2023
@mergify mergify bot added the needs-ci label Jun 28, 2023
@systemd-rhel-bot systemd-rhel-bot changed the title pam: add call to pam_umask (#2210145) pam: add call to pam_umask Jun 28, 2023
@github-actions
Copy link

Tracker - 2210145

The following commits meet all requirements

commit upstream
80d0cef - pam: add call to pam_umask systemd/systemd@159f1b7
systemd/systemd-stable@159f1b7

@mergify mergify bot removed the needs-ci label Jun 28, 2023
Copy link
Contributor

@msekletar msekletar left a comment

Choose a reason for hiding this comment

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

LGTM

@systemd-rhel-bot systemd-rhel-bot merged commit 12894fa into redhat-plumbers:main Jul 13, 2023
31 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants