Skip Menu |
 

To: krb5-bugs@mit.edu
Subject: Delete kadmin/passwd
Date: Wed, 13 Nov 2002 07:44:40 -0500 (EST)
From: hartmans@MIT.EDU (Sam Hartman)

People should use clients/kpasswd instead.

Deleting this may require changing tests depending on whether
kadmin/passwd is used by the tests.
Subject: delete kadmin v1 support
We should get rid of the v1 kadmin support.

Certainly get rid of the support in admin.h if we make it public, to prevent anyone from using it
in new programs. Ideally, kill the support in the library and server as well, if we aren't
supporting anything that needs it.

I think this would mean getting rid of the ancient password-changing service built into
kadmind, for which we haven't installed a client program in a long time. And we can finally kill
the client^H^H^H^H^H^Htest program we use to make sure the server still works (see ticket
1255), and a bunch of ugly code in the library that makes functions take different data
structures depending on the caller's macro definitions, all the ovsec_* wrapper functions, etc.
It'll also make creating an rpcgen input file for the kadmin protocol much easier.
Sent query to krbdev list 2008-08-04 (see http://mailman.mit.edu/pipermail/krbdev/2008-
August/thread.html#6816) asking if anyone is using the ovsec_ or v1 kadm5_ APIs.
The only tests using kadmin/passwd are the kadmin/passwd unit tests. We could easily excise
that subtree, but some of the tests aren't duplicated elsewhere (changing to same password,
password in dictionary, character class requirements).
From: ghudson@mit.edu
Subject: SVN Commit
Download (untitled) / with headers
text/plain 2.7KiB

The kadmin v1 API and the even older ovsec_kadm_* API were legacy when
kadmin was first incorporated in 1996, and compatibility with them is
no longer believed to be necessary.

The uninstalled kadmin/passwd has been removed (since it used the ovsec
API). The test suite has been updated to use the v2 API where
appropriate, and the parts specifically designed to test the old API
have been excised.


https://github.com/krb5/krb5/commit/1ddf7efda0fa665d86431dfc2a57e90e892b81ab
Commit By: ghudson
Revision: 22521
Changed Files:
U trunk/src/config/pre.in
U trunk/src/configure.in
U trunk/src/kadmin/Makefile.in
D trunk/src/kadmin/passwd/
U trunk/src/kadmin/server/Makefile.in
U trunk/src/kadmin/server/misc.h
U trunk/src/kadmin/server/ovsec_kadmd.c
D trunk/src/kadmin/server/server_glue_v1.c
U trunk/src/kadmin/server/server_stubs.c
U trunk/src/kadmin/testing/scripts/env-setup.shin
U trunk/src/kadmin/testing/scripts/init_db
U trunk/src/kadmin/testing/scripts/make-host-keytab.plin
U trunk/src/kadmin/testing/scripts/start_servers_local
U trunk/src/kadmin/testing/util/Makefile.in
U trunk/src/kadmin/testing/util/deps
U trunk/src/kadmin/testing/util/tcl_kadm5.c
U trunk/src/kadmin/testing/util/tcl_kadm5.h
A trunk/src/kadmin/testing/util/tcl_kadm5_syntax
D trunk/src/kadmin/testing/util/tcl_ovsec_kadm.c
D trunk/src/kadmin/testing/util/tcl_ovsec_kadm_syntax
U trunk/src/kadmin/testing/util/test.c
U trunk/src/lib/kadm5/Makefile.in
U trunk/src/lib/kadm5/admin.h
U trunk/src/lib/kadm5/admin_internal.h
U trunk/src/lib/kadm5/chpass_util_strings.et
U trunk/src/lib/kadm5/clnt/client_init.c
U trunk/src/lib/kadm5/clnt/client_principal.c
U trunk/src/lib/kadm5/clnt/clnt_policy.c
U trunk/src/lib/kadm5/clnt/libkadm5clnt.exports
U trunk/src/lib/kadm5/kadm_rpc_xdr.c
U trunk/src/lib/kadm5/misc_free.c
D trunk/src/lib/kadm5/ovsec_glue.c
U trunk/src/lib/kadm5/srv/libkadm5srv.exports
U trunk/src/lib/kadm5/srv/server_init.c
U trunk/src/lib/kadm5/srv/svr_misc_free.c
U trunk/src/lib/kadm5/srv/svr_policy.c
U trunk/src/lib/kadm5/srv/svr_principal.c
U trunk/src/lib/kadm5/unit-test/Makefile.in
D trunk/src/lib/kadm5/unit-test/README.new-tests
D trunk/src/lib/kadm5/unit-test/api.0/
D trunk/src/lib/kadm5/unit-test/api.1/lock.exp
U trunk/src/lib/kadm5/unit-test/config/unix.exp
U trunk/src/lib/kadm5/unit-test/destroy-test.c
U trunk/src/lib/kadm5/unit-test/handle-test.c
U trunk/src/lib/kadm5/unit-test/init-test.c
U trunk/src/lib/kadm5/unit-test/iter-test.c
U trunk/src/lib/kadm5/unit-test/lib/lib.t
U trunk/src/lib/kadm5/unit-test/lock-test.c
U trunk/src/lib/kadm5/unit-test/randkey-test.c
U trunk/src/lib/kadm5/unit-test/site.exp
U trunk/src/lib/rpc/unit-test/lib/helpers.exp
U trunk/src/lib/rpc/unit-test/rpc_test_setup.sh
From: ghudson@mit.edu
Subject: SVN Commit

In doc/Makefile, specify the new location of the kpasswd man page (the
old one was removed in r22521.


https://github.com/krb5/krb5/commit/0aba4a9b363f2264fb5b3409a9aa93a5b5f3c545
Commit By: ghudson
Revision: 22522
Changed Files:
U trunk/doc/Makefile
From: ghudson@mit.edu
Subject: SVN Commit

Remove unused variables resulting from r22521, and also remove the
unused file svr_misc_free.c.


https://github.com/krb5/krb5/commit/f75549596ed93c658805f3df40ca4d9853b28213
Commit By: ghudson
Revision: 22523
Changed Files:
U trunk/src/kadmin/server/server_stubs.c
U trunk/src/lib/kadm5/misc_free.c
D trunk/src/lib/kadm5/srv/svr_misc_free.c
U trunk/src/lib/kadm5/srv/svr_principal.c
From: tlyu@mit.edu
Subject: SVN Commit

Remove bogus conditional from tcl_kadm5_randkey_principal(), which was
causing unparse_keyblocks() to get stack garbage as num_keys when
num_var was "null", thus overrunning the end of the array.

https://github.com/krb5/krb5/commit/e2d100ad59c313807a43162225d049d6797933ff
Commit By: tlyu
Revision: 22645
Changed Files:
U trunk/src/kadmin/testing/util/tcl_kadm5.c
From: ghudson@mit.edu
Subject: SVN Commit

Update a kadm5 testing library function which was calling
kadm5_get_principal without a mask argment. This was causing many
lib/kadm5 tests to fail, but the failures weren't being recorded
properly, so "make check" was still exiting successfully.


https://github.com/krb5/krb5/commit/8ca9abb2d31ece8d1daa7930beab9ac52f7ad23d
Commit By: ghudson
Revision: 22646
Changed Files:
U trunk/src/lib/kadm5/unit-test/lib/lib.t