Skip to content

pam: migrate to pam_lastlog2 - #305

Merged
aanderse merged 1 commit into
finix-community:mainfrom
chocoblocko9:lastlog2
Oct 15, 2026
Merged

aanderse merged 1 commit into
finix-community:mainfrom
chocoblocko9:lastlog2

Conversation

@chocoblocko9

Copy link
Copy Markdown
Member

i would be mildly surprised if this works, can test in like 2 hours

@dmkhitaryan

Copy link
Copy Markdown
Contributor

Does anyone else greetd or greetd-adjacent setups? lastlog2 didn't work for me without adding session optional ${pkgs.util-linux.lastlog}/lib/security/pam_lastlog2.so silent to greetd's configuration file. Would like to confirm that's the case or not.

@chocoblocko9

chocoblocko9 commented Oct 14, 2026 •

Copy link
Copy Markdown
Member Author

Does anyone else greetd or greetd-adjacent setups? lastlog2 didn't work for me without adding session optional ${pkgs.util-linux.lastlog}/lib/security/pam_lastlog2.so silent to greetd's configuration file. Would like to confirm that's the case or not.

ah, yeah you're right. right now the greetd module doesnt use lastlog at all it seems so i missed it when i was grepping for it

@aanderse should i add lastlog2 to greetd or is there a reason greetd doesnt use lastlog like everything else?

@aanderse

Copy link
Copy Markdown
Member

ready for merge?

@chocoblocko9

Copy link
Copy Markdown
Member Author

ready for merge?

yeah as far as i know! haven't had problems, guess we'll see if anything bad happens after it's merged

@aanderse
aanderse merged commit 321fa1a into finix-community:main Oct 15, 2026
2 checks passed
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.

3 participants