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

Duo Does work with latest build #34

Closed
mhwelsh opened this issue Oct 22, 2018 · 1 comment · Fixed by #35
Closed

Duo Does work with latest build #34

mhwelsh opened this issue Oct 22, 2018 · 1 comment · Fixed by #35

Comments

@mhwelsh
Copy link

mhwelsh commented Oct 22, 2018

Thanks for doing this! Was able to get Google auth set up fine, but eventually wanted to switch to Duo.

Looks like when the switch to multi-stage build was made, a bug was introduced. The locations of the pamduo libs is not copied correctly from the builder image, so you get an error saying that module cannot be found for user root.

Put a patch together locally but github is flaky tonight. The prefix command to duo pam doesn't do anything. make install still puts the libs in /lib64/security. This is not copied to the main image.

I added:
COPY --from=duo-builder lib64/ /lib/

after line 98. This works.

@osterman
Copy link
Member

@mhwelsh thanks so much for reporting this and your workaround. That made it quick for @alebabai to address the issue.

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 a pull request may close this issue.

2 participants