/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
 
* Release critical
6
 
** plymouth.8mandos
7
 
 
8
3
* mandos-client
9
4
** TODO [#B] use scandir(3) instead of readdir(3)
10
5
** TODO [#B] Prefix all debug output with "Mandos plugin " + program_invocation_short_name
 
6
** TODO use error() instead of perror()
11
7
** TODO [#B] Retry a server which has a non-definite reply:
12
8
*** A closed connection during the TLS handshake
13
9
*** A TCP timeout
14
10
** TODO [#B] Use capabilities instead of seteuid().
15
 
** TODO [#A] Retry --connect forever
 
11
** TODO use EX_* from <sysexits.h>
16
12
 
17
13
* splashy
18
14
** TODO [#B] use scandir(3) instead of readdir(3)
19
15
** TODO [#B] Prefix all debug output with "Mandos plugin " + program_invocation_short_name
 
16
** TODO use error() instead of perror()
20
17
 
21
18
* usplash
22
 
** TODO [#A] Make it work again
23
19
** TODO [#B] use scandir(3) instead of readdir(3)
24
20
** TODO [#B] Prefix all debug output with "Mandos plugin " + program_invocation_short_name
25
 
** TODO Use [[info:libc:Argz%20Functions][argz_extract]]
 
21
** TODO use error() instead of perror()
 
22
** TODO use EX_* from <sysexits.h>
26
23
 
27
24
* askpass-fifo
28
25
** TODO [#B] Prefix all debug output with "Mandos plugin " + program_invocation_short_name
 
26
** TODO use error() instead of perror()
29
27
** TODO [#B] Drop privileges after opening FIFO.
30
28
 
31
29
* password-prompt
32
30
** TODO [#B] Prefix all debug output with "Mandos plugin " + program_invocation_short_name
33
 
** TODO [#B] lock stdin (with flock()?)
34
 
 
35
 
* plymouth
36
 
** TODO Use [[info:libc:Argz%20Functions][argz_extract]]
37
 
 
38
 
* TODO [#B] passdev
 
31
** TODO use error() instead of perror()
 
32
** TODO lock stdin (with flock()?)
 
33
 
 
34
* TODO passdev
39
35
 
40
36
* plugin-runner
41
37
** TODO [#B] use scandir(3) instead of readdir(3)
42
38
** TODO [#C] use same file name rules as run-parts(8)
43
 
** kernel command line option for debug info
44
 
** TODO [#B] Use openat()
 
39
** TODO use EX_* from <sysexits.h>
 
40
** TODO use error() instead of perror()
45
41
 
46
42
* mandos (server)
47
 
** TODO [#B] Log level                                                    :BUGS:
48
 
** TODO Persistent state                                                  :BUGS:
49
 
   /var/lib/mandos/*
50
 
*** TODO /etc/mandos/clients.d/*.conf
51
 
    Watch this directory and add/remove/update clients?
52
 
** TODO [#C] config for TXT record
53
 
** TODO Log level option
54
 
   syslogger.setLevel(logging.WARNING)
55
 
   + SetLogLevel D-Bus call
56
 
** 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:
57
55
   [[info:standards:Option%20Table][Table of Long Options]]
58
56
** TODO Implement --socket
59
57
   [[info:standards:Option%20Table][Table of Long Options]]
60
 
** TODO Date+time on console log messages                                 :BUGS:
 
58
** TODO Date+time on console log messages       :BUGS:
61
59
   Is this the default?
62
 
** TODO [#C] DBusServiceObjectUsingSuper
63
 
** TODO [#B] Global enable/disable flag
64
 
** TODO [#B] By-client countdown on secrets given
65
 
** 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
66
64
   Whenever a client successfully gets a secret it could get a
67
65
   one-time timeout boost to allow for an fsck-incurred delay
68
 
** TODO [#A] Delay before client receives key
 
66
** TODO Delay before client receives key
69
67
   This would give an operator opportunity to cancel the request if
70
68
   desired.
71
 
** TODO [#A] Client manual approval mode
 
69
** TODO Client manual approval mode
72
70
   A client needs manual approval on the server before it gets the
73
71
   secret
74
 
** TODO [#B] Support RFC 3339 time duration syntax
75
 
** More D-Bus methods
76
 
*** NeedsApproval(50, True) -> timeout, default approve
77
 
    Default approval is configurable, but True by default
78
 
    + Approve(True) -> approve sending saved
79
 
    + Approve(False) -> Close client connection immediately
80
 
*** NeedsPassword(50) - Timeout, default disapprove
81
 
    + SetPass(u"gazonk", True) -> Approval, persistent
82
 
    + Approve(False) -> Close client connection immediately
83
 
** TODO [#C] python-parsedatetime
84
 
** TODO [#C] systemd/launchd
85
 
   http://0pointer.de/blog/projects/systemd.html
86
 
** TODO Separate logging logic to own object
87
 
** TODO make clients to a dict!
88
 
** TODO [#A] Limit approval_delay to max gnutls/tls timeout value
89
 
** TODO [#B] break the wait on approval_delay if connection dies
 
72
** TODO Persistent state
 
73
   /var/lib/mandos/*
90
74
 
91
75
* mandos.xml
92
 
** 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
93
79
 
94
80
* mandos-ctl
95
81
*** Handle "no D-Bus server" and/or "no Mandos server found" better
96
82
*** [#B] --dump option
97
 
** TODO Support RFC 3339 time duration syntax
98
83
 
99
84
* TODO mandos-dispatch
100
85
  Listens for specified D-Bus signals and spawns shell commands with
101
86
  arguments.
102
87
 
103
88
* mandos-monitor
104
 
** TODO help should be toggable
 
89
** D-Bus main loop w/ signal receiver
105
90
** Urwid client data displayer
106
 
   Better view of client data in the listing
 
91
*** Urwid scaffolding
 
92
*** Client Widgets
107
93
*** Properties popup
108
 
** Nicer crashes. Stack traces Messes up shell.
109
 
*** Print a nice "We are sorry" message, save stack trace to log.
110
94
 
111
95
* mandos-keygen
112
96
** TODO Loop until passwords match when run interactively
116
100
   For testing decryption before rebooting.
117
101
 
118
102
* Makefile
119
 
** TODO Add "--Xlinker --as-needed"
120
 
   http://udrepper.livejournal.com/19395.html
121
 
** TODO [#C] Implement DEB_BUILD_OPTIONS
 
103
** Implement DEB_BUILD_OPTIONS
122
104
   http://www.debian.org/doc/debian-policy/ch-source.html#s-debianrules-options
123
105
 
124
106
* Package
129
111
** TODO [#C] /etc/bash_completion.d/mandos
130
112
   From XML sources directly?
131
113
 
132
 
* Side Stuff
133
 
** TODO Locate which packet move the other bin/sh when busy box is deactivated
134
 
** TODO contact owner of packet, and ask them to have that shell static in position regardless of busybox
135
 
 
136
114
 
137
115
#+STARTUP: showall