/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: 2008-08-16 03:29:08 UTC
  • Revision ID: teddy@fukt.bsnet.se-20080816032908-ihw7c05r2mnyk389
Add feature to specify custom environment variables for plugins.

* plugin-runner.c (plugin): New members "environ" and "envc" to
                            contain possible custom environment.
  (getplugin): Return NULL on failure instead of doing exit(); all
               callers changed.
  (add_to_char_array): New helper function for "add_argument" and
                       "add_environment".
  (addargument): Renamed to "add_argument".  Return bool.  Call
                 "add_to_char_array" to actually do things.
  (add_environment): New; analogous to "add_argument".
  (addcustomargument): Renamed to "add_to_argv" to avoid confusion
                       with "add_argument".
  (main): New options "--global-envs" and "--envs-for" to specify
          custom environment for plugins.  Print environment for
          plugins in debug mode.  Use asprintf instead of strcpy and
          strcat.  Use execve() for plugins with custom environments.
          Free environment for plugin when freeing plugin list.

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
 
** TODO Change make-run-server to not include --no-dbus
7
 
 
8
 
* mandos-client
9
 
** TODO [#B] use scandir(3) instead of readdir(3)
10
 
** TODO [#B] Prefix all debug output with "Mandos plugin " + program_invocation_short_name
11
 
** TODO [#B] Retry a server which has a non-definite reply:
12
 
*** A closed connection during the TLS handshake
13
 
*** A TCP timeout
14
 
** TODO [#B] Use capabilities instead of seteuid().
15
 
** TODO [#A] Retry --connect forever
16
 
 
17
 
* splashy
18
 
** TODO [#B] use scandir(3) instead of readdir(3)
19
 
** TODO [#B] Prefix all debug output with "Mandos plugin " + program_invocation_short_name
20
 
 
21
 
* usplash
22
 
** TODO [#A] Make it work again
23
 
** TODO [#B] use scandir(3) instead of readdir(3)
24
 
** TODO [#B] Prefix all debug output with "Mandos plugin " + program_invocation_short_name
25
 
** TODO Use [[info:libc:Argz%20Functions][argz_extract]]
26
 
 
27
 
* askpass-fifo
28
 
** TODO [#B] Prefix all debug output with "Mandos plugin " + program_invocation_short_name
29
 
** TODO [#B] Drop privileges after opening FIFO.
30
 
 
31
 
* password-prompt
32
 
** 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
39
 
 
40
 
* plugin-runner
41
 
** TODO [#B] use scandir(3) instead of readdir(3)
42
 
** TODO [#C] use same file name rules as run-parts(8)
43
 
** kernel command line option for debug info
44
 
** TODO [#B] Use openat()
45
 
 
46
 
* 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:
57
 
   [[info:standards:Option%20Table][Table of Long Options]]
58
 
** TODO Implement --socket
59
 
   [[info:standards:Option%20Table][Table of Long Options]]
60
 
** TODO Date+time on console log messages                                 :BUGS:
 
3
* [#A] README file
 
4
 
 
5
* Plugin-runner
 
6
** [#A] Free plugin name and args
 
7
   [[file:plugin-runner.c::free%20plugin_list][file:plugin-runner.c::free plugin_list]] (both places)
 
8
** [#A] Change syntax for arguments
 
9
** [#B] Add more comments to code
 
10
** [#B] Add more if(debug) calls
 
11
** [#B] Seperate more code to function for more readability
 
12
** [#B] Make free_plugin_list() function
 
13
** [#A] Man page: man8/plugin-runner.8mandos
 
14
*** DESCRIPTION
 
15
    Describe the plus sign syntax for passing options from crypttab
 
16
*** EXIT STATUS
 
17
    Text needed
 
18
*** EXAMPLES
 
19
    Examples of normal usage, debug usage, debugging single or all
 
20
    plugins, examples of crypttab lines with plus syntax, etc.
 
21
*** FILES
 
22
    Text needed
 
23
*** SECURITY
 
24
    Text needed
 
25
*** NOTES
 
26
    Text needed
 
27
*** BUGS
 
28
    Text needed
 
29
*** SEE ALSO
 
30
    Explaining test on what you can read
 
31
** Support in configuration file for environment variables
 
32
** Keydir move: /etc/mandos -> /etc/keys/mandos
 
33
   Must create in preinst if not pre-depending on cryptsetup
 
34
 
 
35
* Password-request
 
36
** [#A] Man page: man8/password-request.8mandos
 
37
** Make prompt exactly like the normal prompt
 
38
   Use environment variables:
 
39
   "Enter passphrase to unlock the disk $cryptsource ($crypttarget): "
 
40
   [[file:plugins.d/password-prompt.c::fprintf%20stderr%20s%20Password%20prefix][Here]]
 
41
** [#B] Temporarily lower kernel log level
 
42
   for less printouts during sucessfull boot.
 
43
*** DESCRIPTION
 
44
    Move options to new OPTIONS section.
 
45
    State that this command is not meant to be invoked directly, but
 
46
    is run as a plugin from mandos-client(8) and only run in the
 
47
    initrd environment, not the real system.
 
48
*** EXIT STATUS
 
49
    Create this section
 
50
*** EXAMPLES
 
51
    Examples of normal usage, debug usage, debugging by connecting
 
52
    directly, etc.
 
53
*** FILES
 
54
    Describe the key files and the key ring files.  Also note that
 
55
    they should normally have been automatically created.
 
56
*** DIAGNOSTICS
 
57
    Create this section
 
58
*** SECURITY
 
59
    Create this section
 
60
*** NOTES
 
61
    Create this section (if needed)
 
62
*** BUGS
 
63
    Create this section
 
64
*** SEE ALSO
 
65
    Refer to mandos-client(8mandos) and password-prompt(8mandos)
 
66
** IPv4 support
 
67
** use strsep instead of strtok?
 
68
** Do not depend on GnuPG key rings on disk
 
69
   This would mean creating new GnuPG key rings with GPGME by
 
70
   importing the key files from scratch on every program start.
 
71
 
 
72
* Password-prompt
 
73
** [#A] Man page: man8/password-prompt.8mandos
 
74
** Show getenv("cryptsource") and getenv("crypttarget") if set
 
75
   cryptsource will be the device, like "/dev/sda3", and crypttarget
 
76
   will be the device mapper name, like "sda3_crypt".
 
77
*** DESCRIPTION
 
78
    Move options to new OPTIONS section.
 
79
*** EXIT STATUS
 
80
    Create this section
 
81
*** EXAMPLES
 
82
    Examples of normal usage, debug usage, with a prefix, etc.
 
83
*** DIAGNOSTICS
 
84
    Create this section
 
85
*** SECURITY
 
86
    Create this section
 
87
    Not much to do here but it is noteworthy to state the danger of
 
88
    not having a fallback option.
 
89
*** NOTES
 
90
    Note that this is more or less a simple getpass(3) wrapper, even
 
91
    though actual use of getpass(3) is not guaranteed.
 
92
*** BUGS
 
93
    Create this section
 
94
*** SEE ALSO
 
95
    Refer to mandos-client(8mandos) and password-request(8mandos)
 
96
** Use getpass(3)?
 
97
   Man page says "obsolete", but [[info:libc:getpass][GNU LibC Manual: Reading Passwords]]
 
98
   does not.  See also [[http://sources.redhat.com/ml/libc-alpha/2003-05/msg00251.html][Marcus Brinkmann: Re: getpass obsolete?]] and
 
99
   [[http://article.gmane.org/gmane.comp.lib.glibc.alpha/4906][Petter Reinholdtsen: Re: getpass obsolete?]], and especially also
 
100
   [[http://www.steve.org.uk/Reference/Unix/faq_4.html#SEC48][Unix Programming FAQ 3.1 How can I make my program not echo input?]]
 
101
** Replace completely with "/lib/cryptsetup/askpass"?
 
102
 
 
103
* Mandos (server)
 
104
** [#A] Command man page: man8/mandos.8
 
105
*** DESCRIPTION
 
106
    Move options to new OPTIONS section
 
107
*** EXIT STATUS
 
108
    Create this section
 
109
*** EXAMPLES
 
110
    Create this section
 
111
*** FILES
 
112
    Describe briefly that the server gets global settings from
 
113
    mandos.conf and clients from clients.conf, but refer to their man
 
114
    pages for more details.
 
115
*** DIAGNOSTICS
 
116
    Create this section
 
117
*** SECURITY
 
118
    Create this section
 
119
*** NOTES
 
120
    Create this section (if needed)
 
121
*** BUGS
 
122
    Create this section
 
123
*** SEE ALSO
 
124
    Refer to the client man page
 
125
** [#A] Config file man page: man5/mandos.conf (mandos.conf)
 
126
** [#A] Config file man page: man5/mandos-clients.conf (clients.conf)
 
127
** [#A] /etc/init.d/mandos-server               :teddy:
 
128
** Log level
 
129
** /etc/mandos/clients.d/*.conf
 
130
   Watch this directory and add/remove/update clients?
 
131
** config for TXT record
 
132
** Run-time communication with server
 
133
   Probably using D-Bus
 
134
   See also [[*Mandos-tools]]
 
135
** Implement --foreground
 
136
   [[info:standards:Option%20Table][Table of Long Options]]
 
137
** Implement --socket
 
138
   [[info:standards:Option%20Table][Table of Long Options]]
 
139
** Date+time on console log messages
61
140
   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
66
 
   Whenever a client successfully gets a secret it could get a
67
 
   one-time timeout boost to allow for an fsck-incurred delay
68
 
** TODO [#A] Delay before client receives key
69
 
   This would give an operator opportunity to cancel the request if
70
 
   desired.
71
 
** TODO [#A] Client manual approval mode
72
 
   A client needs manual approval on the server before it gets the
73
 
   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 approval_delay to max gnutls/tls timeout value
89
 
 
90
 
* mandos.xml
91
 
** Add mandos contact info in manual pages
92
 
 
93
 
* 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
97
 
 
98
 
* TODO mandos-dispatch
99
 
  Listens for specified D-Bus signals and spawns shell commands with
100
 
  arguments.
101
 
 
102
 
* mandos-monitor
103
 
** TODO help should be toggable
104
 
** Urwid client data displayer
105
 
   Better view of client data in the listing
106
 
*** Properties popup
107
 
** Nicer crashes. Stack traces Messes up shell.
108
 
*** Print a nice "We are sorry" message, save stack trace to log.
109
 
 
110
 
* mandos-keygen
111
 
** TODO Loop until passwords match when run interactively
112
 
** TODO "--secfile" option
113
 
   Using the "secfile" option instead of "secret"
114
 
** TODO [#B] "--test" option
115
 
   For testing decryption before rebooting.
116
 
 
117
 
* Makefile
118
 
** TODO Add "--Xlinker --as-needed"
119
 
   http://udrepper.livejournal.com/19395.html
120
 
** TODO [#C] Implement DEB_BUILD_OPTIONS
121
 
   http://www.debian.org/doc/debian-policy/ch-source.html#s-debianrules-options
122
 
 
123
 
* Package
 
141
 
 
142
* Mandos-tools/utilities
 
143
  All of this probably using D-Bus
 
144
** List clients
 
145
** Disable client
 
146
** Enable client
 
147
 
 
148
* Installer
 
149
** Client
 
150
*** DONE [#A] Change initrd.img file to not be publically readable
 
151
    /usr/share/initramfs-tools/conf-hooks.d/mandos
 
152
    UMASK=027
 
153
*** Update initrd.img after installation
 
154
** Server
 
155
*** [#A] Create mandos user and group for server
 
156
*** [#A] Create /var/run/mandos directory with perm and ownership
 
157
 
 
158
** mandos-keygen
 
159
*** [#A] Command man page: man8/mandos-keygen.8
 
160
*** [#A] Output cut-and-paste ready snippet for clients.conf.
 
161
 
 
162
* [#A] Package
124
163
** /usr/share/initramfs-tools/hooks/mandos
125
 
*** TODO [#C] use same file name rules as run-parts(8)
126
 
*** TODO [#C] Do not install in initrd.img if configured not to.
127
 
    Use "/etc/initramfs-tools/hooksconf.d/mandos"?
128
 
** TODO [#C] /etc/bash_completion.d/mandos
129
 
   From XML sources directly?
130
 
 
131
 
* Side Stuff
132
 
** TODO Locate which packet move the other bin/sh when busy box is deactivated
133
 
** TODO contact owner of packet, and ask them to have that shell static in position regardless of busybox
 
164
*** Do not install in initrd.img if configured not to
 
165
    Use "/etc/initramfs-tools/conf.d/mandos"?  Definitely a debconf
 
166
    question.
 
167
** /etc/bash_completion.d/mandos
 
168
*** From xml sources directly?
 
169
** unperish
 
170
** bzr-builddeb
 
171
 
 
172
* INSTALL file
 
173
 
 
174
* Web site
 
175
 
 
176
* Mailing list
 
177
 
 
178
* Announce project on news
 
179
  [[news:comp.os.linux.announce]]
134
180
 
135
181
 
136
182
#+STARTUP: showall