Shadow Password Support Module

[ellinghaus]

BPO	579435
Nosy	@loewis, @akuchling, @irmen
Files	spwdmodule.c spwd_patch.txt: New patch: the patches needed spwdmodule.c: New patch: Modules/spwdmodule.c libspwd.tex: New patch: Doc/lib/libspwd.tex

^{Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.}

Show more details

GitHub fields:

assignee = 'https://github.com/akuchling'
closed_at = <Date 2005-01-23.09:28:12.000>
created_at = <Date 2002-07-10.03:13:17.000>
labels = ['extension-modules']
title = 'Shadow Password Support Module'
updated_at = <Date 2005-01-23.09:28:12.000>
user = 'https://bugs.python.org/ellinghaus'

bugs.python.org fields:

activity = <Date 2005-01-23.09:28:12.000>
actor = 'loewis'
assignee = 'akuchling'
closed = True
closed_date = None
closer = None
components = ['Extension Modules']
creation = <Date 2002-07-10.03:13:17.000>
creator = 'ellinghaus'
dependencies = []
files = ['4415', '4416', '4417', '4418']
hgrepos = []
issue_num = 579435
keywords = ['patch']
message_count = 9.0
messages = ['40532', '40533', '40534', '40535', '40536', '40537', '40538', '40539', '40540']
nosy_count = 5.0
nosy_names = ['loewis', 'akuchling', 'nnorwitz', 'irmen', 'ellinghaus']
pr_nums = []
priority = 'normal'
resolution = 'accepted'
stage = None
status = 'closed'
superseder = None
type = None
url = 'https://bugs.python.org/issue579435'
versions = ['Python 2.5']

[ellinghaus]

Attached is the spwd module. This module provides
support for Shadow Passwords on Solaris 2.8. This
compliments the nis and pwd modules. This is the only
way to gain access to the encrypted passwords when
using shadow passwords on Solaris.

[nnorwitz]

Logged In: YES
user_id=33168

There doesn't seem to be much interest in this patch. I
don't think it reaches a wide-enough audience, therefore,
I'm rejecting it. If there is broader support (many people
are interested in seeing shadow password support in the
core), we can add it later. I think it would be best to
provide this as a third-party module.

[akuchling]

Logged In: YES
user_id=11375

As it happens, I was hunting for Python shadow password support
and came across this patch. The module isn't Solaris-specific, and
would be necessary on Linux, too.
However, the interface for getting shadow passwords doesn't seem
to be standardized; see http://www.unixpapa.com/incnote/passwd.
html for a list. (The Single Unix spec doesn't seem to mention
shadow passwords.)

One question is whether we want a new module for this, or if the
functions should be in the existing pwd module.

[loewis]

Logged In: YES
user_id=21627

If we eventually want to support all the shadow passwort
APIs, we should make this a separate module - with that
approach, building spwd can might fail without causing a
failure to compile pwd.

Unfortunately, the patch is still incomplete: there is no
change to setup.py, and no documentation change. Are there
any volunteers interesting in completing it? If not, I'm
going to close it again. Anybody hunting for such a module
will still be able to find it.

[irmen]

Logged In: YES
user_id=129426

I would be happy if this module in one form or another makes
it into Python; I recently wanted to do user authentication
based on unix passwords but the regular passwd functions
cannot retrieve the password to check against).
As an aside, how does PAM relate to this? I fooled around a
bit with a python pam module, but simply didn't understand
it and couldn't get it to work properly.

[loewis]

Logged In: YES
user_id=21627

irmen, are you willing to complete this patch?

[irmen]

Logged In: YES
user_id=129426

Martin, yes I will give it a try.

[irmen]

Logged In: YES
user_id=129426

I've attached my new version of the patch (files are marked
New Patch). I've updated the extension module itself to
'modern' standards (i.e. making it look as similar to
pwdmodule.c as possible, including the tuple-like result
object).
I'm not too sure how this module compiles on different
platforms, but I've added preprocessor checks for the
relevant system calls.
This module needs to be tested on different systems (I've
developed it on Linux).
There is no regression test yet.

[loewis]

Logged In: YES
user_id=21627

Thanks for the patch. Applied as

configure 1.468
configure.in 1.481
pyconfig.h.in 1.106
setup.py 1.212
Makefile.deps 1.123
lib.tex 1.237
libpwd.tex 1.15
libspwd.tex 1.1
NEWS 1.1231
Setup.dist 1.49
spwdmodule.c 1.1