/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:
TODO
1
1
-*- org -*-
2
2
 
3
 
* Testing
4
 
** python-nemu
5
 
 
6
 
* mandos-applet
7
 
 
8
3
* mandos-client
9
 
** TODO A --server option which only adds to the server list.
10
 
   (Unlike --connect, which implicitly disables zeroconf.)
 
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
11
10
** TODO [#B] Use capabilities instead of seteuid().
12
 
   https://forums.grsecurity.net/viewtopic.php?f=7&t=2522
13
 
** TODO [#B] Use getaddrinfo(hints=AI_NUMERICHOST) instead of inet_pton()
14
 
** TODO [#C] Make start_mandos_communication() take "struct server".
15
 
** TODO [#C] --interfaces=regex,eth*,noregex (bridge-utils-interfaces(5))
16
 
** TODO [#A] Detect partial writes to stdout and exit with EX_TEMPFAIL
 
11
** TODO use EX_* from <sysexits.h>
17
12
 
18
13
* splashy
19
14
** TODO [#B] use scandir(3) instead of readdir(3)
20
 
** TODO [#A] Detect partial writes to stdout and exit with EX_TEMPFAIL
 
15
** TODO [#B] Prefix all debug output with "Mandos plugin " + program_invocation_short_name
 
16
** TODO use error() instead of perror()
21
17
 
22
 
* usplash (Deprecated)
23
 
** TODO [#B] Make it work again
 
18
* usplash
24
19
** TODO [#B] use scandir(3) instead of readdir(3)
25
 
** TODO [#A] Detect partial writes to stdout and exit with EX_TEMPFAIL
 
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>
26
23
 
27
24
* askpass-fifo
28
 
** TODO [#A] Detect partial writes to stdout and exit with EX_TEMPFAIL
 
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.
29
28
 
30
29
* password-prompt
31
 
** TODO [#B] lock stdin (with flock()?)
32
 
** TODO [#A] Detect partial writes to stdout and exit with EX_TEMPFAIL
33
 
 
34
 
* plymouth
35
 
** TODO [#A] Detect partial writes to stdout and exit with EX_TEMPFAIL
36
 
 
37
 
* 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
38
35
 
39
36
* plugin-runner
40
 
** TODO handle printing for errors for plugins
41
 
*** Hook up stderr of plugins, buffer them, and prepend "Mandos Plugin [plugin name]"
 
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 [#A] Restart plugins which exit with EX_TEMPFAIL
 
39
** TODO use EX_* from <sysexits.h>
 
40
** TODO use error() instead of perror()
45
41
 
46
42
* mandos (server)
47
 
** TODO [#B] --notify-command
48
 
   This would allow the mandos.service to use
49
 
   --notify-command="systemd-notify --pid --ready"
50
 
** TODO [#B] python-systemd
51
 
*** import systemd.daemon; systemd.daemon.notify()
52
 
** TODO [#B] Log level                                                    :BUGS:
53
 
*** TODO /etc/mandos/clients.d/*.conf
54
 
    Watch this directory and add/remove/update clients?
55
 
** TODO [#C] config for TXT record
56
 
** TODO Log level dbus option
57
 
   SetLogLevel D-Bus call
58
 
** TODO [#C] DBusServiceObjectUsingSuper
59
 
** TODO [#B] Global enable/disable flag
60
 
** TODO [#B] By-client countdown on number of secrets given
61
 
** D-Bus Client method NeedsPassword(50) - Timeout, default disapprove
62
 
    + SetPass(u"gazonk", True) -> Approval, persistent
63
 
    + Approve(False) -> Close client connection immediately
64
 
** TODO [#C] python-parsedatetime
65
 
** TODO Separate logging logic to own object
66
 
** TODO [#B] Limit approval_delay to max gnutls/tls timeout value
67
 
** TODO [#B] break the wait on approval_delay if connection dies
68
 
** TODO Generate Client.runtime_expansions from client options + extra
69
 
** TODO Allow %%(checker)s as a runtime expansion
70
 
** TODO D-Bus AddClient() method on server object
71
 
** TODO Use org.freedesktop.DBus.Method.NoReply annotation on async methods. :2:
72
 
** TODO Save state periodically to recover better from hard shutdowns
73
 
** TODO CheckerCompleted method, deprecate CheckedOK
74
 
** TODO Secret Service API?
75
 
   https://standards.freedesktop.org/secret-service/
76
 
** TODO Remove D-Bus interfaces with old domain name                      :2:
77
 
** TODO Remove old string_to_delta format                                 :2:
78
 
** TODO http://0pointer.de/blog/projects/stateless.html
79
 
*** File in /usr/lib/sysusers.d to create user+group "_mandos"
80
 
** TODO Error handling on error parsing config files
81
 
** TODO init.d script error handling
82
 
** TODO D-Bus server properties; address, port, interface, etc.           :2:
83
 
** Python 3                                                               :2:
84
 
*** TODO [#C] In Python 3.3, use shlex.quote() instead of re.escape()
 
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:
 
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:
 
59
   Is this the default?
 
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
85
79
 
86
80
* mandos-ctl
87
81
*** Handle "no D-Bus server" and/or "no Mandos server found" better
88
 
** TODO Remove old string_to_delta format                                 :2:
 
82
*** [#B] --dump option
89
83
 
90
84
* TODO mandos-dispatch
91
85
  Listens for specified D-Bus signals and spawns shell commands with
92
86
  arguments.
93
87
 
94
88
* mandos-monitor
95
 
** TODO --servicename                                                  :BUGS:
96
 
** TODO help should be toggleable
 
89
** D-Bus main loop w/ signal receiver
97
90
** Urwid client data displayer
98
 
   Better view of client data in the listing
 
91
*** Urwid scaffolding
 
92
*** Client Widgets
99
93
*** Properties popup
100
 
** Print a nice "We are sorry" message, save stack trace to log.
101
94
 
102
95
* mandos-keygen
 
96
** TODO Loop until passwords match when run interactively
103
97
** TODO "--secfile" option
104
98
   Using the "secfile" option instead of "secret"
105
99
** TODO [#B] "--test" option
106
100
   For testing decryption before rebooting.
107
101
 
 
102
* Makefile
 
103
** Implement DEB_BUILD_OPTIONS
 
104
   http://www.debian.org/doc/debian-policy/ch-source.html#s-debianrules-options
 
105
 
108
106
* Package
109
107
** /usr/share/initramfs-tools/hooks/mandos
110
108
*** TODO [#C] use same file name rules as run-parts(8)
111
109
*** TODO [#C] Do not install in initrd.img if configured not to.
112
110
    Use "/etc/initramfs-tools/hooksconf.d/mandos"?
113
 
** TODO [#C] $(pkg-config --variable=completionsdir bash-completion)
 
111
** TODO [#C] /etc/bash_completion.d/mandos
114
112
   From XML sources directly?
115
113
 
116
 
* Side Stuff
117
 
** TODO Locate which package moves the other bin/sh when busybox is deactivated
118
 
** TODO contact owner of package, and ask them to have that shell static in position regardless of busybox
119
 
 
120
 
* [[http://www.undeadly.org/cgi?action=article&sid=20110530221728][OpenBSD]]
121
 
 
122
 
* TODO Use raw public keys (RFC 7250) for TLS communications              :2:
123
 
** Support for this is planned for GnuTLS version 3.6
124
 
   https://gitlab.com/gnutls/gnutls/issues/26
125
 
** Rationale
126
 
*** The client key is used both for communication and encryption
127
 
    Using raw keys in GnuTLS instead uses separate keys for
128
 
    communication and password decryption.
129
 
*** GnuTLS 3.5.9 has deprecated the OpenPGP functions
130
 
    The functions are still available, but deprecated:
131
 
    https://gitlab.com/gnutls/gnutls/issues/102
132
 
 
133
114
 
134
115
#+STARTUP: showall