/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
3
* mandos-client
6
4
** TODO [#B] use scandir(3) instead of readdir(3)
7
5
** TODO [#B] Prefix all debug output with "Mandos plugin " + program_invocation_short_name
 
6
** TODO use error() instead of perror()
8
7
** TODO [#B] Retry a server which has a non-definite reply:
9
8
*** A closed connection during the TLS handshake
10
9
*** A TCP timeout
11
10
** TODO [#B] Use capabilities instead of seteuid().
12
 
** TODO [#A] Retry --connect forever
 
11
** TODO use EX_* from <sysexits.h>
13
12
 
14
13
* splashy
15
14
** TODO [#B] use scandir(3) instead of readdir(3)
16
15
** TODO [#B] Prefix all debug output with "Mandos plugin " + program_invocation_short_name
 
16
** TODO use error() instead of perror()
17
17
 
18
18
* usplash
19
 
** TODO [#A] Make it work again
20
19
** TODO [#B] use scandir(3) instead of readdir(3)
21
20
** TODO [#B] Prefix all debug output with "Mandos plugin " + program_invocation_short_name
22
 
** TODO Use [[info:libc:Argz%20Functions][argz_extract]]
 
21
** TODO use error() instead of perror()
 
22
** TODO use EX_* from <sysexits.h>
23
23
 
24
24
* askpass-fifo
25
25
** TODO [#B] Prefix all debug output with "Mandos plugin " + program_invocation_short_name
 
26
** TODO use error() instead of perror()
26
27
** TODO [#B] Drop privileges after opening FIFO.
27
28
 
28
29
* password-prompt
29
30
** TODO [#B] Prefix all debug output with "Mandos plugin " + program_invocation_short_name
30
 
** TODO [#B] lock stdin (with flock()?)
 
31
** TODO use error() instead of perror()
 
32
** TODO lock stdin (with flock()?)
31
33
 
32
 
* TODO [#B] passdev
 
34
* TODO passdev
33
35
 
34
36
* plugin-runner
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 [#A] Delay before client receives key
 
66
** TODO Delay before client receives key
63
67
   This would give an operator opportunity to cancel the request if
64
68
   desired.
65
 
** TODO [#A] Client manual approval mode
 
69
** TODO Client manual approval mode
66
70
   A client needs manual approval on the server before it gets the
67
71
   secret
68
 
** TODO [#B] Support RFC 3339 time duration syntax
69
 
** More D-Bus methods
70
 
*** NeedsApproval(50, True) -> timeout, default approve
71
 
    Default approval is configurable, but True by default
72
 
    + Approve(True) -> approve sending saved
73
 
    + Approve(False) -> Close client connection immediately
74
 
*** NeedsPassword(50) - Timeout, default disapprove
75
 
    + SetPass(u"gazonk", True) -> Approval, persistent
76
 
    + Approve(False) -> Close client connection immediately
77
 
** TODO [#C] python-parsedatetime
78
 
** TODO [#C] systemd/launchd
79
 
   http://0pointer.de/blog/projects/systemd.html
80
 
** TODO Separate logging logic to own object
81
 
** TODO make clients to a dict!
82
 
** TODO [#A] Limit approval_delay to max gnutls/tls timeout value
83
 
** TODO [#B] break the wait on approval_delay if connection dies
84
 
** TODO Generate Client.runtime_expansions from client options + extra
85
 
** TODO Allow %%(checker)s as a runtime expansion
 
72
** TODO Persistent state
 
73
   /var/lib/mandos/*
86
74
 
87
75
* mandos.xml
88
 
** 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
89
79
 
90
80
* mandos-ctl
91
81
*** Handle "no D-Bus server" and/or "no Mandos server found" better
92
82
*** [#B] --dump option
93
 
** TODO Support RFC 3339 time duration syntax
94
83
 
95
84
* TODO mandos-dispatch
96
85
  Listens for specified D-Bus signals and spawns shell commands with
97
86
  arguments.
98
87
 
99
88
* mandos-monitor
100
 
** TODO help should be toggable
 
89
** D-Bus main loop w/ signal receiver
101
90
** Urwid client data displayer
102
 
   Better view of client data in the listing
 
91
*** Urwid scaffolding
 
92
*** Client Widgets
103
93
*** Properties popup
104
 
** Nicer crashes. Stack traces Messes up shell.
105
 
*** Print a nice "We are sorry" message, save stack trace to log.
106
 
** Show timeout countdown for approval
107
94
 
108
95
* mandos-keygen
109
96
** TODO Loop until passwords match when run interactively
113
100
   For testing decryption before rebooting.
114
101
 
115
102
* Makefile
116
 
** TODO Add "--Xlinker --as-needed"
117
 
   http://udrepper.livejournal.com/19395.html
118
 
** TODO [#C] Implement DEB_BUILD_OPTIONS
 
103
** Implement DEB_BUILD_OPTIONS
119
104
   http://www.debian.org/doc/debian-policy/ch-source.html#s-debianrules-options
120
105
 
121
106
* Package
126
111
** TODO [#C] /etc/bash_completion.d/mandos
127
112
   From XML sources directly?
128
113
 
129
 
* Side Stuff
130
 
** TODO Locate which packet move the other bin/sh when busy box is deactivated
131
 
** TODO contact owner of packet, and ask them to have that shell static in position regardless of busybox
132
 
 
133
114
 
134
115
#+STARTUP: showall