/mandos/release

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

« back to all changes in this revision

Viewing changes to TODO

More consistent terminology: Clients are no longer "invalid" - they
are "disabled".  All code and documentation changed to reflect this.

D=Bus API change: The "properties" argument was removed from the
"ClientAdded" signal on interface "se.bsnet.fukt.Mandos".  All code in
both "mandos" and "mandos-monitor" changed to reflect this.

* mandos: Replaced "with closing(F)" with simply "with F" in all
          places where F is a file object.
  (Client.still_valid): Removed.  All callers changed to look at
                        "Client.enabled" instead.
  (dbus_service_property): Check for unsupported signatures with the
                           "byte_arrays" option.
  (DBusObjectWithProperties.Set): - '' -
  (ClientHandler.handle): Use the reverse pipe to receive the
                          "Client.enabled" attribute instead of the
                          now-removed "Client.still_valid()" method.
  (ForkingMixInWithPipe): Renamed to "ForkingMixInWithPipes" (all
                          users changed).  Now also create a reverse
                          pipe for sending data to the child process.
  (ForkingMixInWithPipes.add_pipe): Now takes two pipe fd's as
                                    arguments.  All callers changed.
  (IPv6_TCPServer.handle_ipc): Take an additional "reply_fd" argument
                               (all callers changed).  Close the reply
                               pipe when the child data pipe is
                               closed.  New "GETATTR" IPC method; will
                               pickle client attribute and send it
                               over the reply pipe FD.
  (MandosDBusService.ClientAdded): Removed "properties" argument.  All
                                   emitters changed.
* mandos-clients.conf.xml (DESCRIPTION, OPTIONS): Use
                                                  "enabled/disabled"
                                                  terminology.
* mandos-ctl: Option "--is-valid" renamed to "--is-enabled".
* mandos-monitor: Enable user locale.  Try to log exceptions.
  (MandosClientPropertyCache.__init__): Removed "properties" argument.
                                        All callers changed.
  (UserInterface.add_new_client): Remove "properties" argument.  All
                                  callers changed.  Supply "logger"
                                  argument to MandosClientWidget().
  (UserInterface.add_client): New "logger" argument.  All callers
                              changed.
* mandos.xml (BUGS, SECURITY/CLIENTS): Use "enabled/disabled"
                                       terminology.

Show diffs side-by-side

added added

removed removed

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