/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-30 07:03:04 UTC
  • Revision ID: teddy@recompile.se-20190330070304-dqgch62lsaaygg46
mandos-ctl: Refactor D-Bus operations

* mandos-ctl (dbus): Rename imported module to "dbus_python".
  (main): Only create a bus object and do everything via that object.
  (get_mandos_dbus_object): Remove and move code into dbus or
                            dbus_python_adapter namespaces.
  (if_dbus_exception_log_with_exception_and_exit): - '' -
  (SilenceLogger): - '' -
  (dbus): New; move everything dbus-specific into this module-like
          namespace.
  (dbus_python_adapter): New; move everything specific to the
                         dbus-python D-Bus module into this
                         module-like namespace.
  (command.Base.run): Take only a bus argument; use only that.  Pass
                      "client" argument as a D-Bus object path string,
                      not a dbus-python proxy object.  All derivatives
                      adjusted.
  (command.IsEnabled.is_enabled): Remove.
  (command.Approve, command.Deny, command.Remove,
  command.PropertySetter): Do no logging of D-Bus commands, and use
  only bus, not client, to do D-Bus calls.
  (command.DumpJSON.dbus_boolean_to_bool): Remove; move filtering to
                                           dbus_python_adapter.
  (command.Enable, command.Disable, command.StopChecker,
  command.ApproveByDefault): Use normal Python booleans instead of
  dbus-python's special Boolean types.
  (Unique): New; move here out from inside TestPropertySetterCmd.
  (Test_get_mandos_dbus_object): Remove.
  (Test_get_managed_objects): - '' -
  (Test_dbus_exceptions): New.
  (Test_dbus_MandosBus): - '' -
  (Test_dbus_python_adapter_SystemBus): - '' -
  (Test_dbus_python_adapter_CachingBus): - '' -
  (Test_commands_from_options): Don't create mock client proxy
  objects, define dict of client properties and use a mock dbus to
  verify that the correct D-Bus calls are made.  Also remove any types
  specific to dbus-python.
  (TestEnableCmd, TestDisableCmd, TestStartCheckerCmd,
  TestStopCheckerCmd, TestApproveByDefaultCmd, TestDenyByDefaultCmd):
  Use normal Python booleans instead of dbus-python's special Boolean
  types.
  (TestPropertySetterValueCmd.runTest): Remove; unnecessary.

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]]
 
3
* Testing
 
4
** python-nemu
6
5
 
7
6
* mandos-applet
8
7
 
9
8
* mandos-client
 
9
** TODO A --server option which only adds to the server list.
 
10
   (Unlike --connect, which implicitly disables zeroconf.)
10
11
** TODO [#B] Use capabilities instead of seteuid().
11
 
** TODO [#B] Use struct sockaddr_storage instead of a union
 
12
   https://forums.grsecurity.net/viewtopic.php?f=7&t=2522
12
13
** 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
 
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.
15
18
 
16
19
* splashy
17
20
** TODO [#B] use scandir(3) instead of readdir(3)
 
21
** TODO [#A] Detect partial writes to stdout and exit with EX_TEMPFAIL
18
22
 
19
 
* usplash
20
 
** TODO [#A] Make it work again
 
23
* usplash (Deprecated)
 
24
** TODO [#B] Make it work again
21
25
** TODO [#B] use scandir(3) instead of readdir(3)
22
 
** TODO Use [[info:libc:Argz%20Functions][argz_extract]]
 
26
** TODO [#A] Detect partial writes to stdout and exit with EX_TEMPFAIL
23
27
 
24
28
* askpass-fifo
25
 
** TODO [#B] Drop privileges after opening FIFO.
 
29
** TODO [#A] Detect partial writes to stdout and exit with EX_TEMPFAIL
26
30
 
27
31
* password-prompt
28
32
** TODO [#B] lock stdin (with flock()?)
 
33
** TODO [#A] Detect partial writes to stdout and exit with EX_TEMPFAIL
29
34
 
30
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.
31
38
 
32
39
* TODO [#B] passdev
33
40
 
34
41
* plugin-runner
35
42
** TODO handle printing for errors for plugins
36
 
*** Hook up stderr of plugins, buffer them, and prepend mandos pluig [plugin name]
37
 
** TODO [#B] use scandir(3) instead of readdir(3)
 
43
*** Hook up stderr of plugins, buffer them, and prepend "Mandos Plugin [plugin name]"
38
44
** TODO [#C] use same file name rules as run-parts(8)
39
45
** kernel command line option for debug info
40
 
** TODO [#B] Use openat()
 
46
** TODO [#A] Restart plugins which exit with EX_TEMPFAIL
41
47
 
42
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()
43
54
** TODO [#B] Log level                                                    :BUGS:
44
 
** TODO Persistent state                                                  :BUGS:
45
 
   /var/lib/mandos/*
46
55
*** TODO /etc/mandos/clients.d/*.conf
47
56
    Watch this directory and add/remove/update clients?
48
57
** 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:
53
 
   [[info:standards:Option%20Table][Table of Long Options]]
54
 
** TODO Implement --socket
55
 
   [[info:standards:Option%20Table][Table of Long Options]]
56
 
** TODO Date+time on console log messages                                 :BUGS:
57
 
   Is this the default?
 
58
** TODO Log level dbus option
 
59
   SetLogLevel D-Bus call
58
60
** TODO [#C] DBusServiceObjectUsingSuper
59
61
** 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
 
62
** TODO [#B] By-client countdown on number of secrets given
 
63
** D-Bus Client method NeedsPassword(50) - Timeout, default disapprove
64
64
    + SetPass(u"gazonk", True) -> Approval, persistent
65
65
    + Approve(False) -> Close client connection immediately
66
66
** TODO [#C] python-parsedatetime
67
 
** TODO [#C] systemd/launchd
68
 
   http://0pointer.de/blog/projects/systemd.html
69
67
** 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
 
68
** TODO [#B] Limit approval_delay to max gnutls/tls timeout value
72
69
** TODO [#B] break the wait on approval_delay if connection dies
73
70
** TODO Generate Client.runtime_expansions from client options + extra
74
71
** TODO Allow %%(checker)s as a runtime expansion
75
 
** TODO Use python-tlslite?
76
 
 
77
 
* mandos.xml
78
 
** Add mandos contact info in manual pages
 
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()
79
87
 
80
88
* mandos-ctl
81
 
*** Handle "no D-Bus server" and/or "no Mandos server found" better
82
 
*** [#B] --dump option
83
 
** TODO Support RFC 3339 time duration syntax
 
89
** TODO Remove old string_to_delta format                                 :2:
84
90
 
85
91
* TODO mandos-dispatch
86
92
  Listens for specified D-Bus signals and spawns shell commands with
87
93
  arguments.
88
94
 
89
95
* mandos-monitor
 
96
** TODO --servicename                                                  :BUGS:
90
97
** TODO help should be toggleable
91
98
** Urwid client data displayer
92
99
   Better view of client data in the listing
93
100
*** 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
 
101
** Print a nice "We are sorry" message, save stack trace to log.
97
102
 
98
103
* mandos-keygen
99
104
** TODO "--secfile" option
101
106
** TODO [#B] "--test" option
102
107
   For testing decryption before rebooting.
103
108
 
104
 
* Makefile
105
 
** TODO [#C] Implement DEB_BUILD_OPTIONS
106
 
   http://www.debian.org/doc/debian-policy/ch-source.html#s-debianrules-options
107
 
 
108
109
* Package
109
110
** /usr/share/initramfs-tools/hooks/mandos
110
111
*** TODO [#C] use same file name rules as run-parts(8)
111
112
*** TODO [#C] Do not install in initrd.img if configured not to.
112
113
    Use "/etc/initramfs-tools/hooksconf.d/mandos"?
113
 
** TODO [#C] /etc/bash_completion.d/mandos
 
114
** TODO [#C] $(pkg-config --variable=completionsdir bash-completion)
114
115
   From XML sources directly?
115
116
 
116
117
* Side Stuff
117
 
** TODO Locate which packet move the other bin/sh when busy box is deactivated
118
 
** TODO contact owner of packet, and ask them to have that shell static in position regardless of busybox
 
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]]
119
122
 
120
123
 
121
124
#+STARTUP: showall