Skip to content

Support quiet Flag in pam.d Config#280

Open
DabeDotCom wants to merge 2 commits into
duosecurity:masterfrom
DabeDotCom:pam-quiet
Open

Support quiet Flag in pam.d Config#280
DabeDotCom wants to merge 2 commits into
duosecurity:masterfrom
DabeDotCom:pam-quiet

Conversation

@DabeDotCom
Copy link
Copy Markdown

Example:
/etc/pam.d/ssh

auth   sufficient   pam_duo.so quiet

Summary of the change

A trivial update that allows admins to silence pam_info() messages

Test Plan

I added two simple tests:

  1. Make sure the pam.d parser accepts the quiet flag
  2. Make sure passing the quiet flag actually does silence the output «grin»

PS: I also added the flag to the pam_duo(8) manpage.

Example:
**`/etc/pam.d/ssh`**
```
auth   sufficient   pam_duo.so quiet
```
@DabeDotCom
Copy link
Copy Markdown
Author

Bump? This and #310 should be pretty benign, but they're enough of a "Pebble In My Shoe" that I went ahead and fixed them... It'd be nice if they got reviewed/merged, just so I wouldn't have to maintain my own fork any more. Thanks! :-D

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.

1 participant