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

Consider adding environment variables from PAM. #2

Open
Duncaen opened this issue May 8, 2016 · 3 comments
Open

Consider adding environment variables from PAM. #2

Duncaen opened this issue May 8, 2016 · 3 comments

Comments

@Duncaen
Copy link
Owner

Duncaen commented May 8, 2016

Get environment variables from pam modules with pam_getenvlist or pam_getenv.
Should doas keep all variables or only those that are specified in doas.conf by keepenv?

@escondida
Copy link

Personally, I'd argue that keepenv is probably better, to prevent accidentally leaking variables that could have unintended side effects.

@PaulGrandperrin
Copy link

Hi, I just wrote about problems that happen on NixOS because of this issue: NixOS/nixpkgs#158988

Would it be possible to reconsider ?

@PaulGrandperrin
Copy link

Maybe I can try writing a PR if you want :)

PaulGrandperrin added a commit to PaulGrandperrin/nix-systems that referenced this issue Nov 3, 2023
ennoying hack to go around Duncaen/OpenDoas#2
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

No branches or pull requests

3 participants