/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-12-25 23:13:47 UTC
  • Revision ID: teddy@fukt.bsnet.se-20091225231347-gg9u9ru0wj0f24hh
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
 
* Release critical
6
 
** Make mandos-ctl fail nicer to permission deny (runing as non-root for example)
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
16
11
 
17
12
* splashy
18
13
** TODO [#B] use scandir(3) instead of readdir(3)
19
14
** TODO [#B] Prefix all debug output with "Mandos plugin " + program_invocation_short_name
 
15
** TODO use error() instead of perror()
20
16
 
21
17
* usplash
22
 
** TODO [#A] Make it work again
23
18
** TODO [#B] use scandir(3) instead of readdir(3)
24
19
** TODO [#B] Prefix all debug output with "Mandos plugin " + program_invocation_short_name
25
 
** TODO Use [[info:libc:Argz%20Functions][argz_extract]]
 
20
** TODO use error() instead of perror()
26
21
 
27
22
* askpass-fifo
28
23
** TODO [#B] Prefix all debug output with "Mandos plugin " + program_invocation_short_name
 
24
** TODO use error() instead of perror()
29
25
** TODO [#B] Drop privileges after opening FIFO.
30
26
 
31
27
* password-prompt
32
28
** 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
 
29
** TODO use error() instead of perror()
 
30
** TODO lock stdin (with flock()?)
 
31
 
 
32
* TODO passdev
39
33
 
40
34
* plugin-runner
41
35
** TODO [#B] use scandir(3) instead of readdir(3)
42
36
** TODO [#C] use same file name rules as run-parts(8)
43
 
** kernel command line option for debug info
44
 
** TODO [#B] Use openat()
 
37
** TODO use error() instead of perror()
45
38
 
46
39
* 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:
 
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:
57
52
   [[info:standards:Option%20Table][Table of Long Options]]
58
53
** TODO Implement --socket
59
54
   [[info:standards:Option%20Table][Table of Long Options]]
60
 
** TODO Date+time on console log messages                                 :BUGS:
 
55
** TODO Date+time on console log messages       :BUGS:
61
56
   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
 
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
66
61
   Whenever a client successfully gets a secret it could get a
67
62
   one-time timeout boost to allow for an fsck-incurred delay
68
 
** TODO [#A] Delay before client receives key
 
63
** TODO Delay before client receives key
69
64
   This would give an operator opportunity to cancel the request if
70
65
   desired.
71
 
** TODO [#A] Client manual approval mode
 
66
** TODO Client manual approval mode
72
67
   A client needs manual approval on the server before it gets the
73
68
   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 approved_delay to max gnutls/tls timeout value
89
 
** TODO [#B] break the wait on approved_delay if connection dies
 
69
** TODO Persistent state
 
70
   /var/lib/mandos/*
 
71
** TODO Support RFC 3339 time duration syntax
90
72
 
91
73
* mandos.xml
92
74
** [[file:mandos.xml::XXX][Document D-Bus interface]]
93
 
   Remove mention of lack of such interface in BUGS section
94
 
** Add mandos contact info in manual pages
95
75
 
96
 
* TODO [#A] Provide and install /etc/dbus-1/system.d/mandos.conf
 
76
* Provide and install /etc/dbus-1/system.d/mandos.conf
97
77
 
98
78
* mandos-ctl
99
79
*** Handle "no D-Bus server" and/or "no Mandos server found" better
105
85
  arguments.
106
86
 
107
87
* mandos-monitor
108
 
** TODO help should be toggable
109
88
** Urwid client data displayer
110
 
   Better view of client data in the listing
111
89
*** Properties popup
112
 
** Nicer crashes. Stack traces Messes up shell.
113
 
*** Print a nice "We are sorry" message, save stack trace to log.
114
90
 
115
91
* mandos-keygen
116
92
** TODO Loop until passwords match when run interactively
120
96
   For testing decryption before rebooting.
121
97
 
122
98
* Makefile
123
 
** TODO Add "--Xlinker --as-needed"
124
 
   http://udrepper.livejournal.com/19395.html
125
 
** TODO [#C] Implement DEB_BUILD_OPTIONS
 
99
** Implement DEB_BUILD_OPTIONS
126
100
   http://www.debian.org/doc/debian-policy/ch-source.html#s-debianrules-options
127
101
 
128
102
* Package
133
107
** TODO [#C] /etc/bash_completion.d/mandos
134
108
   From XML sources directly?
135
109
 
136
 
* Side Stuff
137
 
** TODO Locate which packet move the other bin/sh when busy box is deactivated
138
 
** TODO contact owner of packet, and ask them to have that shell static in position regardless of busybox
139
 
 
140
110
 
141
111
#+STARTUP: showall