Your patch is good (for pam and other methods that allow the app to handle the password) and it is in trunk
On Nov 21, 3:35 pm, Matt Broadstone <mbroa...@gmail.com> wrote: > On Mon, Nov 21, 2011 at 3:25 PM, Massimo Di > Pierro<massimo.dipie...@gmail.com> wrote: > > Good point. Basic auth may not work with PAM. Please open a ticket. > > OK, issue created:http://code.google.com/p/web2py/issues/detail?id=532 > > Also, in the comments section I've included a patch to allow PAM > authentication. As I note in the comment I don't have time to check > that this works for all auth cases, but we only allow PAM and form > based logins so this solution suffices for our needs. > > Matt > > > > > > > > > On Nov 21, 1:26 pm, Matt Broadstone <mbroa...@gmail.com> wrote: > >> Is it possible to use PAM for basic auth? I have basic auth working > >> (thanks to Hong-Khoan Quach's patch), however I can't seem to link it > >> to PAM. We are using PAM successfully for web based user > >> authentication, but basic auth seems to be ignoring this. > > >> Thanks, > >> Matt