/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: 2019-03-18 22:29:25 UTC
  • Revision ID: teddy@recompile.se-20190318222925-jvhek84dgcfgj6g3
mandos-ctl: Refactor tests

* mandos-ctl: Where the clients names "foo" and "barbar" do not refer
              to the actual mock clients in the TestCommand class,
              change all occurrences of these names to "client1" and
              "client2" (or just "client" when only one is used) .
              Also change all test doubles to use correct terminology;
              some things called mocks are actually stubs or spies,
              and rename all true mocks to have "mock" in their names.
              Also eliminate duplicate values in tests; derive values
              from previously defined values whenever possible.

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
1
-*- org -*-
2
2
 
3
 
* Use _attribute_((nonnull)) wherever possible.
 
3
* Testing
 
4
** python-nemu
 
5
 
 
6
* mandos-applet
4
7
 
5
8
* mandos-client
6
 
** TODO [#B] use scandir(3) instead of readdir(3)
7
 
** TODO [#B] Prefix all debug output with "Mandos plugin " + program_invocation_short_name
8
 
** TODO use error() instead of perror()
9
 
** TODO [#B] Retry a server which has a non-definite reply:
10
 
*** A closed connection during the TLS handshake
11
 
*** A TCP timeout
 
9
** TODO A --server option which only adds to the server list.
 
10
   (Unlike --connect, which implicitly disables zeroconf.)
12
11
** TODO [#B] Use capabilities instead of seteuid().
13
 
** TODO [#A] Retry --connect forever
 
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
 
17
** TODO [#B] Use reallocarray() with GNU LibC 2.29 or later.
14
18
 
15
19
* splashy
16
20
** TODO [#B] use scandir(3) instead of readdir(3)
17
 
** TODO [#B] Prefix all debug output with "Mandos plugin " + program_invocation_short_name
18
 
** TODO [#B] use error() instead of perror()
 
21
** TODO [#A] Detect partial writes to stdout and exit with EX_TEMPFAIL
19
22
 
20
 
* usplash
21
 
** TODO [#A] Make it work again
 
23
* usplash (Deprecated)
 
24
** TODO [#B] Make it work again
22
25
** TODO [#B] use scandir(3) instead of readdir(3)
23
 
** TODO [#B] Prefix all debug output with "Mandos plugin " + program_invocation_short_name
24
 
** TODO [#B] use error() instead of perror()
 
26
** TODO [#A] Detect partial writes to stdout and exit with EX_TEMPFAIL
25
27
 
26
28
* askpass-fifo
27
 
** TODO [#B] Prefix all debug output with "Mandos plugin " + program_invocation_short_name
28
 
** TODO [#B] use error() instead of perror()
29
 
** TODO [#B] Drop privileges after opening FIFO.
 
29
** TODO [#A] Detect partial writes to stdout and exit with EX_TEMPFAIL
30
30
 
31
31
* password-prompt
32
 
** TODO [#B] Prefix all debug output with "Mandos plugin " + program_invocation_short_name
33
 
** TODO [#B] use error() instead of perror()
34
32
** TODO [#B] lock stdin (with flock()?)
 
33
** TODO [#A] Detect partial writes to stdout and exit with EX_TEMPFAIL
 
34
 
 
35
* plymouth
 
36
** TODO [#A] Detect partial writes to stdout and exit with EX_TEMPFAIL
 
37
** TODO [#B] Use reallocarray() with GNU LibC 2.29 or later.
35
38
 
36
39
* TODO [#B] passdev
37
40
 
38
41
* plugin-runner
39
 
** TODO [#B] use scandir(3) instead of readdir(3)
 
42
** TODO handle printing for errors for plugins
 
43
*** Hook up stderr of plugins, buffer them, and prepend "Mandos Plugin [plugin name]"
40
44
** TODO [#C] use same file name rules as run-parts(8)
41
45
** kernel command line option for debug info
42
 
** TODO [#B] use error() instead of perror()
43
 
** TODO [#B] Use openat()
 
46
** TODO [#A] Restart plugins which exit with EX_TEMPFAIL
44
47
 
45
48
* mandos (server)
 
49
** TODO [#B] --notify-command
 
50
   This would allow the mandos.service to use
 
51
   --notify-command="systemd-notify --pid --ready"
 
52
** TODO [#B] python-systemd
 
53
*** import systemd.daemon; systemd.daemon.notify()
46
54
** TODO [#B] Log level                                                    :BUGS:
47
 
** TODO Persistent state                                                  :BUGS:
48
 
   /var/lib/mandos/*
49
55
*** TODO /etc/mandos/clients.d/*.conf
50
56
    Watch this directory and add/remove/update clients?
51
57
** TODO [#C] config for TXT record
52
 
** TODO Log level option
53
 
   syslogger.setLevel(logging.WARNING)
54
 
   + SetLogLevel D-Bus call
55
 
** TODO Implement --foreground                                            :BUGS:
56
 
   [[info:standards:Option%20Table][Table of Long Options]]
57
 
** TODO Implement --socket
58
 
   [[info:standards:Option%20Table][Table of Long Options]]
59
 
** TODO Date+time on console log messages                                 :BUGS:
60
 
   Is this the default?
 
58
** TODO Log level dbus option
 
59
   SetLogLevel D-Bus call
61
60
** TODO [#C] DBusServiceObjectUsingSuper
62
61
** TODO [#B] Global enable/disable flag
63
 
** TODO [#B] By-client countdown on secrets given
64
 
** TODO [#B] Fix problem with fsck taking a really long time
65
 
   Whenever a client successfully gets a secret it could get a
66
 
   one-time timeout boost to allow for an fsck-incurred delay
67
 
** TODO [#A] Delay before client receives key
68
 
   This would give an operator opportunity to cancel the request if
69
 
   desired.
70
 
** TODO [#A] Client manual approval mode
71
 
   A client needs manual approval on the server before it gets the
72
 
   secret
73
 
** TODO [#B] Support RFC 3339 time duration syntax
74
 
** More D-Bus methods
75
 
*** NeedsApproval(50, True) -> timeout, default approve
76
 
    Default approval is configurable, but True by default
77
 
    + Approve(True) -> approve sending saved
78
 
    + Approve(False) -> Close client connection immediately
79
 
*** NeedsPassword(50) - Timeout, default disapprove
 
62
** TODO [#B] By-client countdown on number of secrets given
 
63
** D-Bus Client method NeedsPassword(50) - Timeout, default disapprove
80
64
    + SetPass(u"gazonk", True) -> Approval, persistent
81
65
    + Approve(False) -> Close client connection immediately
82
66
** TODO [#C] python-parsedatetime
83
 
** TODO [#C] systemd/launchd
84
 
   http://0pointer.de/blog/projects/systemd.html
85
 
 
86
 
* mandos.xml
87
 
** [[file:mandos.xml::XXX][Document D-Bus interface]]
88
 
   Remove mention of lack of such interface in BUGS section
89
 
** Add mandos contact info in manual pages
90
 
 
91
 
* TODO [#A] Provide and install /etc/dbus-1/system.d/mandos.conf
 
67
** TODO Separate logging logic to own object
 
68
** TODO [#B] Limit approval_delay to max gnutls/tls timeout value
 
69
** TODO [#B] break the wait on approval_delay if connection dies
 
70
** TODO Generate Client.runtime_expansions from client options + extra
 
71
** TODO Allow %%(checker)s as a runtime expansion
 
72
** TODO D-Bus AddClient() method on server object
 
73
** TODO Use org.freedesktop.DBus.Method.NoReply annotation on async methods. :2:
 
74
** TODO Save state periodically to recover better from hard shutdowns
 
75
** TODO CheckerCompleted method, deprecate CheckedOK
 
76
** TODO Secret Service API?
 
77
   https://standards.freedesktop.org/secret-service/
 
78
** TODO Remove D-Bus interfaces with old domain name                      :2:
 
79
** TODO Remove old string_to_delta format                                 :2:
 
80
** TODO http://0pointer.de/blog/projects/stateless.html
 
81
*** File in /usr/lib/sysusers.d to create user+group "_mandos"
 
82
** TODO Error handling on error parsing config files
 
83
** TODO init.d script error handling
 
84
** TODO D-Bus server properties; address, port, interface, etc.           :2:
 
85
** Python 3                                                               :2:
 
86
*** TODO [#C] In Python 3.3, use shlex.quote() instead of re.escape()
92
87
 
93
88
* mandos-ctl
94
 
*** Handle "no D-Bus server" and/or "no Mandos server found" better
95
 
*** [#B] --dump option
96
 
** TODO Support RFC 3339 time duration syntax
 
89
** TODO Remove old string_to_delta format                                 :2:
97
90
 
98
91
* TODO mandos-dispatch
99
92
  Listens for specified D-Bus signals and spawns shell commands with
100
93
  arguments.
101
94
 
102
95
* mandos-monitor
 
96
** TODO --servicename                                                  :BUGS:
 
97
** TODO help should be toggleable
103
98
** Urwid client data displayer
104
99
   Better view of client data in the listing
105
100
*** Properties popup
 
101
** Print a nice "We are sorry" message, save stack trace to log.
106
102
 
107
103
* mandos-keygen
108
 
** TODO Loop until passwords match when run interactively
109
104
** TODO "--secfile" option
110
105
   Using the "secfile" option instead of "secret"
111
106
** TODO [#B] "--test" option
112
107
   For testing decryption before rebooting.
113
108
 
114
 
* Makefile
115
 
** TODO Add "--Xlinker --as-needed"
116
 
   http://udrepper.livejournal.com/19395.html
117
 
** TODO [#C] Implement DEB_BUILD_OPTIONS
118
 
   http://www.debian.org/doc/debian-policy/ch-source.html#s-debianrules-options
119
 
 
120
109
* Package
121
110
** /usr/share/initramfs-tools/hooks/mandos
122
111
*** TODO [#C] use same file name rules as run-parts(8)
123
112
*** TODO [#C] Do not install in initrd.img if configured not to.
124
113
    Use "/etc/initramfs-tools/hooksconf.d/mandos"?
125
 
** TODO [#C] /etc/bash_completion.d/mandos
 
114
** TODO [#C] $(pkg-config --variable=completionsdir bash-completion)
126
115
   From XML sources directly?
127
116
 
 
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
* [[http://www.undeadly.org/cgi?action=article&sid=20110530221728][OpenBSD]]
 
122
 
128
123
 
129
124
#+STARTUP: showall