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

Jump to bottom

add debug message to empty PAM stack file #238

Closed
pbrezina opened this issue Mar 3, 2021 · 0 comments · Fixed by #250
Closed

add debug message to empty PAM stack file #238

pbrezina opened this issue Mar 3, 2021 · 0 comments · Fixed by #250
Assignees
@pbrezina
Labels
pull request available
Milestone
authselect-1.2.4

Comments

@pbrezina
Copy link
Member

pbrezina commented Mar 3, 2021

If a profile does not support specific authentication type (e.g. fingerprint) it just generates empty PAM stack. It would be good to include a pam_debug to make it more visible when someone tries to use it. E.g.

auth required  pam_debug.so auth=authinfo_unavail
@pbrezina pbrezina added this to the authselect-1.2.3 milestone Mar 4, 2021
pbrezina added a commit to pbrezina/authselect that referenced this issue May 13, 2021
@pbrezina
profiles: add pam_debug to empty fingerprint and smartcard stack
ee3a856 
This way if someone tries to use the stack it will return AUTHINFO_UNAVAIL.

Resolves:
authselect#238
pbrezina added a commit to pbrezina/authselect that referenced this issue May 13, 2021
@pbrezina
profiles: add pam_debug to empty fingerprint and smartcard stack
7a0f45f 
This way if someone tries to use the stack it will return AUTHINFO_UNAVAIL.

Resolves:
authselect#238
@pbrezina pbrezina mentioned this issue May 13, 2021
Merged
@pbrezina pbrezina self-assigned this May 13, 2021
pbrezina added a commit that referenced this issue May 31, 2021
@pbrezina
profiles: add pam_debug to empty fingerprint and smartcard stack
85d1142 
This way if someone tries to use the stack it will return AUTHINFO_UNAVAIL.

Resolves:
#238
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pbrezina pbrezina

Labels
pull request available
Projects
None yet
Milestone
authselect-1.2.4
Development

Successfully merging a pull request may close this issue.

profiles: add pam_debug to empty fingerprint and smartcard stack pbrezina/authselect
1 participant
@pbrezina