/mandos/trunk

To get this branch, use:
bzr branch http://bzr.recompile.se/loggerhead/mandos/trunk

« back to all changes in this revision

Viewing changes to TODO

  • Committer: Teddy Hogeborn
  • Date: 2009-10-24 19:17:52 UTC
  • Revision ID: teddy@fukt.bsnet.se-20091024191752-7g6xlf6wpp2pdexb
Convert some programs to use the exit codes from <sysexits.h>.  Change
all programs using the "argp" parsing functions to use them correctly;
checking return value, using argp_error() to report parse errors etc.

* plugin-runner.c: Use <sysexits.h> exit codes.  Always use fallback,
                   even on option errors, except for "--help", etc.
  (getplugin): Make sure "errno" is set correctly on return.
  (main): Declare our own "--help", "--usage", and "--version"
          options which do not cause the fallback to be invoked.
          In all other options, use fallback on any error.
  (parse_opt, parse_opt_config_file): Reset errno at start and return
                                      errno.  No need to check "arg"
                                      for NULL.  New "--help",
                                      "--usage", and "--version"
                                      options.
  (parse_opt): Accept empty string as global option.  Do not print
               errors which will be detected and reported later.  Do
               "argp_error()" on parse error or empty plugin names.
* plugins.d/mandos-client.c: Use <sysexits.h> exit codes.  Do not
                             return successful exit code on "--help",
                             etc. since this would give the wrong
                             message to "plugin-runner".
  (main): Declare our own "--help", "--usage", and "--version"
          options which do not return a successful exit code.
  (parse_opt): Reset errno at start and return errno.  Do
               "argp_error()" on parse errors.  New "--help",
               "--usage", and "--version" options.
* plugins.d/password-prompt.c: Use exit codes from <sysexits.h>.  Do
                               not return successful exit code on
                               "--help", etc. since this would give
                               the wrong message to "plugin-runner".
  (main): Declare our own "--help", "--usage", and "--version" options
          which do not return a successful exit code.  Do
          close(STDOUT_FILENO) after writing to check its return code.
  (parse_opt): Reset errno at start and return errno.

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
1
-*- org -*-
2
2
 
3
 
* [#A] README file
 
3
* mandos-client
 
4
** TODO [#B] use scandir(3) instead of readdir(3)
 
5
** TODO [#B] Prefix all debug output with "Mandos plugin " + program_invocation_short_name
 
6
** TODO use error() instead of perror()
 
7
** TODO [#B] Retry a server which has a non-definite reply:
 
8
*** A closed connection during the TLS handshake
 
9
*** A TCP timeout
 
10
** TODO [#B] Use capabilities instead of seteuid().
 
11
** TODO use EX_* from <sysexits.h>
 
12
 
 
13
* splashy
 
14
** TODO [#B] use scandir(3) instead of readdir(3)
 
15
** TODO [#B] Prefix all debug output with "Mandos plugin " + program_invocation_short_name
 
16
** TODO use error() instead of perror()
 
17
 
 
18
* usplash
 
19
** TODO [#B] use scandir(3) instead of readdir(3)
 
20
** TODO [#B] Prefix all debug output with "Mandos plugin " + program_invocation_short_name
 
21
** TODO use error() instead of perror()
 
22
** TODO use EX_* from <sysexits.h>
 
23
 
 
24
* askpass-fifo
 
25
** TODO [#B] Prefix all debug output with "Mandos plugin " + program_invocation_short_name
 
26
** TODO use error() instead of perror()
 
27
** TODO [#B] Drop privileges after opening FIFO.
 
28
 
 
29
* password-prompt
 
30
** TODO [#B] Prefix all debug output with "Mandos plugin " + program_invocation_short_name
 
31
** TODO use error() instead of perror()
 
32
** TODO lock stdin (with flock()?)
 
33
 
 
34
* TODO passdev
4
35
 
5
36
* plugin-runner
6
 
** [#B] Add more comments to code
7
 
** [#B] Add more if(debug) calls
8
 
** [#B] Seperate more code to function for more readability
9
 
 
10
 
* password-request
11
 
** [#A] Man page: man8/password-request.8mandos
12
 
*** DESCRIPTION
13
 
    State that this command is not meant to be invoked directly, but
14
 
    is run as a plugin from mandos-client(8) and only run in the
15
 
    initrd environment, not the real system.
16
 
*** PURPOSE
17
 
    As in mandos.xml
18
 
*** OVERVIEW
19
 
    As in mandos.xml
20
 
*** EXIT STATUS
21
 
*** ENVIRONMENT
22
 
    Note that it does *not* currently use cryptsource or crypttarget.
23
 
*** FILES
24
 
    Describe the key files and the key ring files.  Also note that
25
 
    they should normally have been automatically created.
26
 
*** BUGS
27
 
*** EXAMPLE
28
 
    Examples of normal usage, debug usage, debugging by connecting
29
 
    directly, etc.
30
 
*** SECURITY
31
 
*** SEE ALSO
32
 
    Update from mandos.xml
33
 
** [#B] Temporarily lower kernel log level
34
 
   for less printouts during sucessfull boot.
35
 
** IPv4 support
36
 
** use strsep instead of strtok?
37
 
** Do not depend on GnuPG key rings on disk
38
 
   This would mean creating new GnuPG key rings with GPGME by
39
 
   importing the key files from scratch on every program start.
40
 
** Keydir move: /etc/mandos -> /etc/keys/mandos
41
 
   Must create in preinst if not pre-depending on cryptsetup
42
 
 
43
 
* password-prompt
44
 
** [#C] Use getpass(3)?
45
 
   Man page says "obsolete", but [[info:libc:getpass][GNU LibC Manual: Reading Passwords]]
46
 
   does not.  See also [[http://sources.redhat.com/ml/libc-alpha/2003-05/msg00251.html][Marcus Brinkmann: Re: getpass obsolete?]] and
47
 
   [[http://article.gmane.org/gmane.comp.lib.glibc.alpha/4906][Petter Reinholdtsen: Re: getpass obsolete?]], and especially also
48
 
   [[http://www.steve.org.uk/Reference/Unix/faq_4.html#SEC48][Unix Programming FAQ 3.1 How can I make my program not echo input?]]
 
37
** TODO [#B] use scandir(3) instead of readdir(3)
 
38
** TODO [#C] use same file name rules as run-parts(8)
 
39
** TODO use EX_* from <sysexits.h>
 
40
** TODO use error() instead of perror()
49
41
 
50
42
* mandos (server)
51
 
** [#A] /etc/init.d/mandos-server               :teddy:
52
 
** [#B] Log level                               :bugs:
53
 
** /etc/mandos/clients.d/*.conf
 
43
** TODO [#B] Log level                          :BUGS:
 
44
** TODO /etc/mandos/clients.d/*.conf
54
45
   Watch this directory and add/remove/update clients?
55
 
** config for TXT record
56
 
** [#B] Run-time communication with server      :bugs:
 
46
** TODO config for TXT record
 
47
** TODO [#B] Run-time communication with server :BUGS:
57
48
   Probably using D-Bus
58
 
   See also [[*Mandos-tools]]
59
 
** Implement --foreground                       :bugs:
60
 
   [[info:standards:Option%20Table][Table of Long Options]]
61
 
** Implement --socket
62
 
   [[info:standards:Option%20Table][Table of Long Options]]
63
 
** Date+time on console log messages            :bugs:
 
49
*** Client class
 
50
*** Main server
 
51
    + SetLogLevel
 
52
      syslogger.setLevel(logging.WARNING)
 
53
    + [[http://log.ometer.com/2007-05.html][Best D-Bus practices]]
 
54
** TODO Implement --foreground                  :BUGS:
 
55
   [[info:standards:Option%20Table][Table of Long Options]]
 
56
** TODO Implement --socket
 
57
   [[info:standards:Option%20Table][Table of Long Options]]
 
58
** TODO Date+time on console log messages       :BUGS:
64
59
   Is this the default?
65
 
 
66
 
* Mandos-tools/utilities
67
 
  All of this probably using D-Bus
68
 
** List clients
69
 
** Disable client
70
 
** Enable client
71
 
 
72
 
* Man pages
73
 
** Use xinclude for common sections
74
 
   Like authors, etc.
75
 
 
76
 
 
77
 
* Installer
78
 
** Client-side
79
 
*** Update initrd.img after installation
80
 
    This seems to use some kind of "trigger" system
81
 
    [[file:/usr/share/doc/dpkg/triggers.txt.gz]]
82
 
    dpkg-trigger(1), deb-triggers(5)
83
 
*** Keydir move: /etc/mandos -> /etc/keys/mandos
84
 
    Must create in preinst if not pre-depending on cryptsetup
85
 
*** mandos-keygen
86
 
**** "--passfile" option
87
 
     Using the "secfile" option instead of "secret"
88
 
**** [#A] "--test" option
89
 
     For testing decryption before rebooting.
90
 
** Server-side
91
 
*** [#A] Create mandos user and group for server
92
 
*** [#A] Create /var/run/mandos directory with perm and ownership
93
 
 
94
 
* [#A] Package
 
60
** TODO DBusServiceObjectUsingSuper
 
61
** TODO Global enable/disable flag
 
62
** TODO By-client countdown on secrets given
 
63
** TODO Fix problem with fsck taking a really long time
 
64
   Whenever a client successfully gets a secret it could get a
 
65
   one-time timeout boost to allow for an fsck-incurred delay
 
66
** TODO Delay before client receives key
 
67
   This would give an operator opportunity to cancel the request if
 
68
   desired.
 
69
** TODO Client manual approval mode
 
70
   A client needs manual approval on the server before it gets the
 
71
   secret
 
72
** TODO Persistent state
 
73
   /var/lib/mandos/*
 
74
 
 
75
* mandos.xml
 
76
** [[file:mandos.xml::XXX][Document D-Bus interface]]
 
77
 
 
78
* Provide and install /etc/dbus-1/system.d/mandos.conf
 
79
 
 
80
* mandos-ctl
 
81
*** Handle "no D-Bus server" and/or "no Mandos server found" better
 
82
*** [#B] --dump option
 
83
 
 
84
* TODO mandos-dispatch
 
85
  Listens for specified D-Bus signals and spawns shell commands with
 
86
  arguments.
 
87
 
 
88
* mandos-monitor
 
89
** D-Bus main loop w/ signal receiver
 
90
** Urwid client data displayer
 
91
*** Urwid scaffolding
 
92
*** Client Widgets
 
93
*** Properties popup
 
94
 
 
95
* mandos-keygen
 
96
** TODO Loop until passwords match when run interactively
 
97
** TODO "--secfile" option
 
98
   Using the "secfile" option instead of "secret"
 
99
** TODO [#B] "--test" option
 
100
   For testing decryption before rebooting.
 
101
 
 
102
* Makefile
 
103
** Implement DEB_BUILD_OPTIONS
 
104
   http://www.debian.org/doc/debian-policy/ch-source.html#s-debianrules-options
 
105
 
 
106
* Package
95
107
** /usr/share/initramfs-tools/hooks/mandos
96
 
*** Do not install in initrd.img if configured not to.
97
 
    Use "/etc/initramfs-tools/conf.d/mandos"?  Definitely a debconf
98
 
    question.
99
 
** /etc/bash_completion.d/mandos
 
108
*** TODO [#C] use same file name rules as run-parts(8)
 
109
*** TODO [#C] Do not install in initrd.img if configured not to.
 
110
    Use "/etc/initramfs-tools/hooksconf.d/mandos"?
 
111
** TODO [#C] /etc/bash_completion.d/mandos
100
112
   From XML sources directly?
101
 
** unperish
102
 
** bzr-builddeb
103
 
 
104
 
* INSTALL file
105
 
 
106
 
* Web site
107
 
 
108
 
* Mailing list
109
 
 
110
 
* Announce project on news
111
 
  [[news:comp.os.linux.announce]]
112
113
 
113
114
 
114
115
#+STARTUP: showall