/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
10
 
** TODO [#B] Flag fprintf_plus so compiler checks the format string.
11
 
** TODO [#A] OpenVPN network hook
12
 
** TODO [#A] Wireless network hook
 
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
13
10
** TODO [#B] Use capabilities instead of seteuid().
14
 
** TODO [#B] Use struct sockaddr_storage instead of a union
15
 
** TODO [#B] Use getaddrinfo(hints=AI_NUMERICHOST) instead of inet_pton()
16
 
** TODO [#B] Use getnameinfo(serv=NULL, NI_NUMERICHOST) instead of inet_ntop()
17
 
** TODO [#B] Prefer /run/tmp over /tmp, if it exists
18
11
 
19
12
* splashy
20
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()
21
16
 
22
 
* usplash (Deprecated)
23
 
** TODO [#A] Make it work again
 
17
* usplash
24
18
** TODO [#B] use scandir(3) instead of readdir(3)
25
 
** 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()
26
21
 
27
22
* askpass-fifo
 
23
** TODO [#B] Prefix all debug output with "Mandos plugin " + program_invocation_short_name
 
24
** TODO use error() instead of perror()
28
25
** TODO [#B] Drop privileges after opening FIFO.
29
26
 
30
27
* password-prompt
31
 
** TODO [#B] lock stdin (with flock()?)
32
 
 
33
 
* plymouth
34
 
 
35
 
* 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
36
33
 
37
34
* plugin-runner
38
 
** TODO handle printing for errors for plugins
39
 
*** Hook up stderr of plugins, buffer them, and prepend mandos pluig [plugin name]
40
35
** TODO [#B] use scandir(3) instead of readdir(3)
41
36
** TODO [#C] use same file name rules as run-parts(8)
42
 
** kernel command line option for debug info
43
 
** TODO [#B] Use openat()
 
37
** TODO use error() instead of perror()
44
38
 
45
39
* mandos (server)
46
 
** TODO Document why we ignore sigint
47
 
** TODO [#B] Log level                                                    :BUGS:
48
 
*** TODO /etc/mandos/clients.d/*.conf
49
 
    Watch this directory and add/remove/update clients?
50
 
** TODO [#C] config for TXT record
51
 
** TODO Log level dbus option
52
 
   SetLogLevel D-Bus call
53
 
** 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:
54
52
   [[info:standards:Option%20Table][Table of Long Options]]
55
53
** TODO Implement --socket
56
54
   [[info:standards:Option%20Table][Table of Long Options]]
57
 
** TODO [#C] DBusServiceObjectUsingSuper
58
 
** TODO [#B] Global enable/disable flag
59
 
** TODO [#B] By-client countdown on number of 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
 
   http://wiki.debian.org/systemd
69
 
** TODO Separate logging logic to own object
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?
75
 
** TODO D-Bus AddClient() method on server object
 
55
** TODO Date+time on console log messages       :BUGS:
 
56
   Is this the default?
 
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
82
80
*** [#B] --dump option
83
81
** TODO Support RFC 3339 time duration syntax
84
 
** TODO Send milliseconds if bare integer is passed as time duration
85
82
 
86
83
* TODO mandos-dispatch
87
84
  Listens for specified D-Bus signals and spawns shell commands with
88
85
  arguments.
89
86
 
90
87
* mandos-monitor
91
 
** TODO help should be toggleable
92
88
** Urwid client data displayer
93
 
   Better view of client data in the listing
94
89
*** Properties popup
95
 
** Nicer crashes. Stack traces Messes up shell.
96
 
*** Print a nice "We are sorry" message, save stack trace to log.
97
 
** Show timeout countdown for approval
98
90
 
99
91
* mandos-keygen
 
92
** TODO Loop until passwords match when run interactively
100
93
** TODO "--secfile" option
101
94
   Using the "secfile" option instead of "secret"
102
95
** TODO [#B] "--test" option
103
96
   For testing decryption before rebooting.
104
97
 
105
98
* Makefile
106
 
** TODO [#C] Implement DEB_BUILD_OPTIONS
 
99
** Implement DEB_BUILD_OPTIONS
107
100
   http://www.debian.org/doc/debian-policy/ch-source.html#s-debianrules-options
108
101
 
109
102
* Package
114
107
** TODO [#C] /etc/bash_completion.d/mandos
115
108
   From XML sources directly?
116
109
 
117
 
* Side Stuff
118
 
** TODO Locate which package moves the other bin/sh when busybox is deactivated
119
 
** TODO contact owner of package, and ask them to have that shell static in position regardless of busybox
120
 
 
121
110
 
122
111
#+STARTUP: showall