/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
 
* 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
13
11
 
14
12
* splashy
15
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()
16
16
 
17
17
* usplash
18
 
** TODO [#A] Make it work again
19
18
** TODO [#B] use scandir(3) instead of readdir(3)
20
 
** 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()
21
21
 
22
22
* askpass-fifo
 
23
** TODO [#B] Prefix all debug output with "Mandos plugin " + program_invocation_short_name
 
24
** TODO use error() instead of perror()
23
25
** TODO [#B] Drop privileges after opening FIFO.
24
26
 
25
27
* password-prompt
26
 
** TODO [#B] lock stdin (with flock()?)
27
 
 
28
 
* plymouth
29
 
 
30
 
* 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
31
33
 
32
34
* 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
35
** TODO [#B] use scandir(3) instead of readdir(3)
36
36
** TODO [#C] use same file name rules as run-parts(8)
37
 
** kernel command line option for debug info
38
 
** TODO [#B] Use openat()
 
37
** TODO use error() instead of perror()
39
38
 
40
39
* mandos (server)
41
 
** TODO [#A] Make all attributes properties!
42
 
** TODO [#B] Log level                                                    :BUGS:
43
 
** TODO Persistent state                                                  :BUGS:
44
 
   /var/lib/mandos/*
45
 
*** TODO /etc/mandos/clients.d/*.conf
46
 
    Watch this directory and add/remove/update clients?
47
 
** TODO [#C] config for TXT record
48
 
** TODO Log level option
49
 
   syslogger.setLevel(logging.WARNING)
50
 
   + SetLogLevel D-Bus call
51
 
** 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:
52
52
   [[info:standards:Option%20Table][Table of Long Options]]
53
53
** TODO Implement --socket
54
54
   [[info:standards:Option%20Table][Table of Long Options]]
55
 
** TODO Date+time on console log messages                                 :BUGS:
 
55
** TODO Date+time on console log messages       :BUGS:
56
56
   Is this the default?
57
 
** TODO [#C] DBusServiceObjectUsingSuper
58
 
** TODO [#B] Global enable/disable flag
59
 
** TODO [#B] By-client countdown on secrets given
60
 
** TODO [#B] Support RFC 3339 time duration syntax
61
 
** More D-Bus methods
62
 
*** NeedsPassword(50) - Timeout, default disapprove
63
 
    + SetPass(u"gazonk", True) -> Approval, persistent
64
 
    + Approve(False) -> Close client connection immediately
65
 
** TODO [#C] python-parsedatetime
66
 
** TODO [#C] systemd/launchd
67
 
   http://0pointer.de/blog/projects/systemd.html
68
 
** TODO Separate logging logic to own object
69
 
** TODO make clients to a dict!
70
 
** TODO [#A] Limit approval_delay to max gnutls/tls timeout value
71
 
** TODO [#B] break the wait on approval_delay if connection dies
72
 
** TODO Generate Client.runtime_expansions from client options + extra
73
 
** TODO Allow %%(checker)s as a runtime expansion
74
 
** 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
75
72
 
76
73
* mandos.xml
77
 
** 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
78
77
 
79
78
* mandos-ctl
80
79
*** Handle "no D-Bus server" and/or "no Mandos server found" better
86
85
  arguments.
87
86
 
88
87
* mandos-monitor
89
 
** TODO help should be toggleable
90
88
** Urwid client data displayer
91
 
   Better view of client data in the listing
92
89
*** Properties popup
93
 
** Nicer crashes. Stack traces Messes up shell.
94
 
*** Print a nice "We are sorry" message, save stack trace to log.
95
 
** Show timeout countdown for approval
96
90
 
97
91
* mandos-keygen
 
92
** TODO Loop until passwords match when run interactively
98
93
** TODO "--secfile" option
99
94
   Using the "secfile" option instead of "secret"
100
95
** TODO [#B] "--test" option
101
96
   For testing decryption before rebooting.
102
97
 
103
98
* Makefile
104
 
** TODO [#C] Implement DEB_BUILD_OPTIONS
 
99
** Implement DEB_BUILD_OPTIONS
105
100
   http://www.debian.org/doc/debian-policy/ch-source.html#s-debianrules-options
106
101
 
107
102
* Package
112
107
** TODO [#C] /etc/bash_completion.d/mandos
113
108
   From XML sources directly?
114
109
 
115
 
* Side Stuff
116
 
** TODO Locate which packet move the other bin/sh when busy box is deactivated
117
 
** TODO contact owner of packet, and ask them to have that shell static in position regardless of busybox
118
 
 
119
110
 
120
111
#+STARTUP: showall