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

modify control value of pam_systemd #375

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

kkzhsh
Copy link

@kkzhsh kkzhsh commented Aug 28, 2024

I am debugging a display session related issue.
The scene is as follow:

  1. Type in username and password
  2. Lightdm greeter session starts and pam_systemd.so is called
  3. pam_systemd.so returns with some err code, then the X session can not run

I guess the control value of pam_systemd.so shall be modified as “required”, and which shall result lightdm fail, and lightdm has chance to be restarted automatically by systemd.

Could U help to review this PR? Thanks

Copy link

github-actions bot commented Aug 28, 2024

Everyone contributing to this PR have now signed the CLA. Thanks!

@kkzhsh kkzhsh closed this Aug 29, 2024
@kkzhsh kkzhsh reopened this Aug 29, 2024
@kkzhsh kkzhsh closed this Aug 29, 2024
@kkzhsh kkzhsh reopened this Aug 29, 2024
@kkzhsh kkzhsh closed this Aug 29, 2024
@kkzhsh kkzhsh reopened this Aug 29, 2024
@kkzhsh kkzhsh closed this Aug 29, 2024
@kkzhsh kkzhsh reopened this Aug 29, 2024
@kkzhsh
Copy link
Author

kkzhsh commented Aug 29, 2024

Dear @robert-ancell
I have signed CLA from URL https://ubuntu.com/legal/contributors
Why the CLA-check always fail?

@robert-ancell
Copy link
Collaborator

Dear @robert-ancell I have signed CLA from URL https://ubuntu.com/legal/contributors Why the CLA-check always fail?

It will just take time for the system to update, no problem!

@kkzhsh kkzhsh closed this Aug 30, 2024
@kkzhsh kkzhsh reopened this Aug 30, 2024
@kkzhsh kkzhsh closed this Sep 2, 2024
@kkzhsh kkzhsh reopened this Sep 2, 2024
@kkzhsh
Copy link
Author

kkzhsh commented Sep 19, 2024

@robert-ancell
Could u help to review it? thanks

@kkzhsh kkzhsh closed this Oct 18, 2024
@kkzhsh kkzhsh reopened this Oct 18, 2024
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