/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
 
* Use _attribute_((nonnull)) wherever possible.
4
 
 
5
 
* mandos-applet
6
 
 
7
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
8
10
** TODO [#B] Use capabilities instead of seteuid().
9
 
** TODO [#B] Use struct sockaddr_storage instead of a union
10
 
** TODO [#B] Use getaddrinfo(hints=AI_NUMERICHOST) instead of inet_pton()
11
 
** TODO [#B] Use getnameinfo(serv=NULL, NI_NUMERICHOST) instead of inet_ntop()
12
 
** TODO [#B] Prefer /run/tmp over /tmp, if it exists
 
11
** TODO use EX_* from <sysexits.h>
13
12
 
14
13
* splashy
15
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()
16
17
 
17
18
* usplash
18
 
** TODO [#A] Make it work again
19
19
** TODO [#B] use scandir(3) instead of readdir(3)
20
 
** TODO Use [[info:libc:Argz%20Functions][argz_extract]]
 
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>
21
23
 
22
24
* askpass-fifo
 
25
** TODO [#B] Prefix all debug output with "Mandos plugin " + program_invocation_short_name
 
26
** TODO use error() instead of perror()
23
27
** TODO [#B] Drop privileges after opening FIFO.
24
28
 
25
29
* password-prompt
26
 
** TODO [#B] lock stdin (with flock()?)
27
 
 
28
 
* plymouth
29
 
 
30
 
* TODO [#B] passdev
 
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
31
35
 
32
36
* plugin-runner
33
 
** TODO handle printing for errors for plugins
34
 
*** Hook up stderr of plugins, buffer them, and prepend mandos pluig [plugin name]
35
37
** TODO [#B] use scandir(3) instead of readdir(3)
36
38
** TODO [#C] use same file name rules as run-parts(8)
37
 
** kernel command line option for debug info
38
 
** TODO [#B] Use openat()
 
39
** TODO use EX_* from <sysexits.h>
 
40
** TODO use error() instead of perror()
39
41
 
40
42
* mandos (server)
41
 
** TODO [#B] Log level                                                    :BUGS:
42
 
** TODO Persistent state                                                  :BUGS:
43
 
   /var/lib/mandos/*
44
 
*** TODO /etc/mandos/clients.d/*.conf
45
 
    Watch this directory and add/remove/update clients?
46
 
** TODO [#C] config for TXT record
47
 
** TODO Log level option
48
 
   syslogger.setLevel(logging.WARNING)
49
 
   + SetLogLevel D-Bus call
50
 
** TODO Implement --foreground                                            :BUGS:
 
43
** TODO [#B] Log level                          :BUGS:
 
44
** TODO /etc/mandos/clients.d/*.conf
 
45
   Watch this directory and add/remove/update clients?
 
46
** TODO config for TXT record
 
47
** TODO [#B] Run-time communication with server :BUGS:
 
48
   Probably using D-Bus
 
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:
51
55
   [[info:standards:Option%20Table][Table of Long Options]]
52
56
** TODO Implement --socket
53
57
   [[info:standards:Option%20Table][Table of Long Options]]
54
 
** TODO Date+time on console log messages                                 :BUGS:
 
58
** TODO Date+time on console log messages       :BUGS:
55
59
   Is this the default?
56
 
** TODO [#C] DBusServiceObjectUsingSuper
57
 
** TODO [#B] Global enable/disable flag
58
 
** TODO [#B] By-client countdown on secrets given
59
 
** TODO [#B] Fix problem with fsck taking a really long time
 
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
60
64
   Whenever a client successfully gets a secret it could get a
61
65
   one-time timeout boost to allow for an fsck-incurred delay
62
 
** TODO [#B] Support RFC 3339 time duration syntax
63
 
** More D-Bus methods
64
 
*** NeedsPassword(50) - Timeout, default disapprove
65
 
    + SetPass(u"gazonk", True) -> Approval, persistent
66
 
    + Approve(False) -> Close client connection immediately
67
 
** TODO [#C] python-parsedatetime
68
 
** TODO [#C] systemd/launchd
69
 
   http://0pointer.de/blog/projects/systemd.html
70
 
** TODO Separate logging logic to own object
71
 
** TODO make clients to a dict!
72
 
** TODO [#A] Limit approval_delay to max gnutls/tls timeout value
73
 
** TODO [#B] break the wait on approval_delay if connection dies
74
 
** TODO Generate Client.runtime_expansions from client options + extra
75
 
** TODO Allow %%(checker)s as a runtime expansion
76
 
** TODO Use python-tlslite?
 
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/*
77
74
 
78
75
* mandos.xml
79
 
** Add mandos contact info in manual pages
 
76
** [[file:mandos.xml::XXX][Document D-Bus interface]]
 
77
 
 
78
* Provide and install /etc/dbus-1/system.d/mandos.conf
80
79
 
81
80
* mandos-ctl
82
81
*** Handle "no D-Bus server" and/or "no Mandos server found" better
83
82
*** [#B] --dump option
84
 
** TODO Support RFC 3339 time duration syntax
85
83
 
86
84
* TODO mandos-dispatch
87
85
  Listens for specified D-Bus signals and spawns shell commands with
88
86
  arguments.
89
87
 
90
88
* mandos-monitor
91
 
** TODO help should be toggleable
 
89
** D-Bus main loop w/ signal receiver
92
90
** Urwid client data displayer
93
 
   Better view of client data in the listing
 
91
*** Urwid scaffolding
 
92
*** Client Widgets
94
93
*** Properties popup
95
 
** Nicer crashes. Stack traces Messes up shell.
96
 
*** Print a nice "We are sorry" message, save stack trace to log.
97
 
** Show timeout countdown for approval
98
94
 
99
95
* mandos-keygen
 
96
** TODO Loop until passwords match when run interactively
100
97
** TODO "--secfile" option
101
98
   Using the "secfile" option instead of "secret"
102
99
** TODO [#B] "--test" option
103
100
   For testing decryption before rebooting.
104
101
 
105
102
* Makefile
106
 
** TODO [#C] Implement DEB_BUILD_OPTIONS
 
103
** Implement DEB_BUILD_OPTIONS
107
104
   http://www.debian.org/doc/debian-policy/ch-source.html#s-debianrules-options
108
105
 
109
106
* Package
114
111
** TODO [#C] /etc/bash_completion.d/mandos
115
112
   From XML sources directly?
116
113
 
117
 
* Side Stuff
118
 
** TODO Locate which packet move the other bin/sh when busy box is deactivated
119
 
** TODO contact owner of packet, and ask them to have that shell static in position regardless of busybox
120
 
 
121
114
 
122
115
#+STARTUP: showall