LnL7 / nix-darwin

nix modules for darwin
MIT License
2.43k stars 405 forks source link

Option to enable touch ID authentication for sudo doesn't work in tmux #985

Open 347Online opened 2 days ago

347Online commented 2 days ago

I really love the out-of-the-box option to enable touch ID for sudo and was surprised to find it doesn't work in tmux

I eventually stumbled across this, which is working for me, but I know it's going to get blown away when macOS does an update

I spent awhile noodling around with getting this working in my personal config to no avail, but it'd be even better to just have this as an option in nix-darwin — programs.pam-reattach.enable perhaps

emilazy commented 2 days ago

Yeah, finally getting this in with the rework of the sudo PAM stuff is on my to‐do list. See also: https://github.com/LnL7/nix-darwin/pull/787, https://github.com/LnL7/nix-darwin/pull/662, https://github.com/LnL7/nix-darwin/pull/606, https://github.com/LnL7/nix-darwin/pull/591. (This is probably the area of nix-darwin that has evolved the slowest for how many PRs it’s had…)

My current plan is as outlined in https://github.com/LnL7/nix-darwin/pull/787#issuecomment-2165246619.

347Online commented 2 days ago

Ahh, thank you that's fairly illuminating! My apologies if I've created extra noise here, for whatever reason I was not able to turn up any of these prior conversations when I searched

emilazy commented 2 days ago

No worries, I just wish I’d made progress on this sooner!