Skip to content

RFC changes to the auth-pam plugin.

Tim Small openvpn-devel 2 versions – New

Get series with b4
b4 am [email protected]...

Versions newest first · linked by author and title

  1. v2 Latest6 patches 0 repliesNew
  2. v14 patches 0 repliesSuperseded

Patches in v2 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/6Separate error logging for pam auth and accountsrc/plugins/auth-pam/auth-pam.c+27 -150 replies
2/6Log common name as well as username for pam authsrc/plugins/auth-pam/auth-pam.c+4 -30 replies
3/6Refactor name/value list search and substitution code.src/plugins/auth-pam/auth-pam.c+59 -370 replies
4/6Add default password reply with name_value_listsrc/plugins/auth-pam/auth-pam.c+20 -10 replies
5/6Allow administrator to supply a user to pam_startsrc/plugins/auth-pam/auth-pam.c+17 -10 replies
6/6Improve docs and detail new functionalitysrc/plugins/auth-pam/README.auth-pam+133 -190 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v2 yet.

Read the v2 thread

Cover letter [Openvpn-devel] [PATCH v2 0/6] RFC changes to the auth-pam plugin.

Take 2 - sorry for the noise - this update to the series splits some patches down further, fixes a security bug vs the first iteration, and hopefully makes various improvements to patches.

I wanted to use OpenVPN with PAM whilst enforcing the use of the TLS client cert common name in place of the user-supplied user name. This wouldn't work with the PAM plugin I was using, and in the process of debugging this I've made a few changes to the auth-pam plugin which I hope make it easier to use and less 'brittle' in that it should now be possible to configure customised behaviour whilst minimising the reliance on matching against strings which are only intended to be parsed by humans, not other software.

For my use case, I can now obtain the custom behaviour I wanted without having to rely on any such matching against human-readable messages generated by PAM modules (which may be subject to changes due to software updates, changes in default system modules and PAM config, as well as localisation etc.).

The changes should be backward compatible with existing deployments and I've also made some changes to the README as I felt it was a bit difficult to follow in places...

I write very little C, so please excuse me if the following are a bit rough.

Tim.

Tim Small (6):
Separate error logging for pam auth and account Log common name as well as username for pam auth Refactor name/value list search and substitution code. Add default password reply with name_value_list Allow administrator to supply a user to pam_start Improve docs and detail new functionality

 src/plugins/auth-pam/README.auth-pam | 152 +++++++++++++++++++++++++----
 src/plugins/auth-pam/auth-pam.c      | 182 ++++++++++++++++++++++++-----------
 2 files changed, 259 insertions(+), 75 deletions(-)
Signature · 1 line
2.1.4