/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

  • 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
 
* 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 [#B] Retry a server which has a non-definite reply:
9
 
*** A closed connection during the TLS handshake
10
 
*** A TCP timeout
11
 
** TODO [#B] Use capabilities instead of seteuid().
12
 
** TODO [#A] Retry --connect forever
13
 
 
14
 
* splashy
15
 
** TODO [#B] use scandir(3) instead of readdir(3)
16
 
** TODO [#B] Prefix all debug output with "Mandos plugin " + program_invocation_short_name
17
 
 
18
 
* usplash
19
 
** TODO [#A] Make it work again
20
 
** TODO [#B] use scandir(3) instead of readdir(3)
21
 
** TODO [#B] Prefix all debug output with "Mandos plugin " + program_invocation_short_name
22
 
** TODO Use [[info:libc:Argz%20Functions][argz_extract]]
23
 
 
24
 
* askpass-fifo
25
 
** TODO [#B] Prefix all debug output with "Mandos plugin " + program_invocation_short_name
26
 
** TODO [#B] Drop privileges after opening FIFO.
27
 
 
28
 
* password-prompt
29
 
** TODO [#B] Prefix all debug output with "Mandos plugin " + program_invocation_short_name
30
 
** TODO [#B] lock stdin (with flock()?)
31
 
 
32
 
* TODO [#B] passdev
33
 
 
34
 
* plugin-runner
35
 
** TODO [#B] use scandir(3) instead of readdir(3)
36
 
** TODO [#C] use same file name rules as run-parts(8)
37
 
** kernel command line option for debug info
38
 
** TODO [#B] Use openat()
39
 
 
40
 
* mandos (server)
41
 
** TODO [#B] Log level                                                    :BUGS:
42
 
** TODO Persistent state                                                  :BUGS:
43
 
   /var/lib/mandos/*
44
 
*** TODO /etc/mandos/clients.d/*.conf
45
 
    Watch this directory and add/remove/update clients?
46
 
** TODO [#C] config for TXT record
47
 
** TODO Log level option
48
 
   syslogger.setLevel(logging.WARNING)
49
 
   + SetLogLevel D-Bus call
50
 
** TODO Implement --foreground                                            :BUGS:
51
 
   [[info:standards:Option%20Table][Table of Long Options]]
52
 
** TODO Implement --socket
53
 
   [[info:standards:Option%20Table][Table of Long Options]]
54
 
** 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
55
140
   Is this the default?
56
 
** TODO [#C] DBusServiceObjectUsingSuper
57
 
** TODO [#B] Global enable/disable flag
58
 
** TODO [#B] By-client countdown on secrets given
59
 
** TODO [#B] Fix problem with fsck taking a really long time
60
 
   Whenever a client successfully gets a secret it could get a
61
 
   one-time timeout boost to allow for an fsck-incurred delay
62
 
** TODO [#A] Delay before client receives key
63
 
   This would give an operator opportunity to cancel the request if
64
 
   desired.
65
 
** TODO [#A] Client manual approval mode
66
 
   A client needs manual approval on the server before it gets the
67
 
   secret
68
 
** TODO [#B] Support RFC 3339 time duration syntax
69
 
** More D-Bus methods
70
 
*** NeedsApproval(50, True) -> timeout, default approve
71
 
    Default approval is configurable, but True by default
72
 
    + Approve(True) -> approve sending saved
73
 
    + Approve(False) -> Close client connection immediately
74
 
*** NeedsPassword(50) - Timeout, default disapprove
75
 
    + SetPass(u"gazonk", True) -> Approval, persistent
76
 
    + Approve(False) -> Close client connection immediately
77
 
** TODO [#C] python-parsedatetime
78
 
** TODO [#C] systemd/launchd
79
 
   http://0pointer.de/blog/projects/systemd.html
80
 
** TODO Separate logging logic to own object
81
 
** TODO make clients to a dict!
82
 
** TODO [#A] Limit approval_delay to max gnutls/tls timeout value
83
 
** TODO [#B] break the wait on approval_delay if connection dies
84
 
** TODO Generate Client.runtime_expansions from client options + extra
85
 
** TODO Allow %%(checker)s as a runtime expansion
86
 
 
87
 
* mandos.xml
88
 
** Add mandos contact info in manual pages
89
 
 
90
 
* mandos-ctl
91
 
*** Handle "no D-Bus server" and/or "no Mandos server found" better
92
 
*** [#B] --dump option
93
 
** TODO Support RFC 3339 time duration syntax
94
 
 
95
 
* TODO mandos-dispatch
96
 
  Listens for specified D-Bus signals and spawns shell commands with
97
 
  arguments.
98
 
 
99
 
* mandos-monitor
100
 
** TODO help should be toggable
101
 
** Urwid client data displayer
102
 
   Better view of client data in the listing
103
 
*** Properties popup
104
 
** Nicer crashes. Stack traces Messes up shell.
105
 
*** Print a nice "We are sorry" message, save stack trace to log.
106
 
** Show timeout countdown for approval
107
 
 
108
 
* mandos-keygen
109
 
** TODO Loop until passwords match when run interactively
110
 
** TODO "--secfile" option
111
 
   Using the "secfile" option instead of "secret"
112
 
** TODO [#B] "--test" option
113
 
   For testing decryption before rebooting.
114
 
 
115
 
* Makefile
116
 
** TODO Add "--Xlinker --as-needed"
117
 
   http://udrepper.livejournal.com/19395.html
118
 
** TODO [#C] Implement DEB_BUILD_OPTIONS
119
 
   http://www.debian.org/doc/debian-policy/ch-source.html#s-debianrules-options
120
 
 
121
 
* 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
122
163
** /usr/share/initramfs-tools/hooks/mandos
123
 
*** TODO [#C] use same file name rules as run-parts(8)
124
 
*** TODO [#C] Do not install in initrd.img if configured not to.
125
 
    Use "/etc/initramfs-tools/hooksconf.d/mandos"?
126
 
** TODO [#C] /etc/bash_completion.d/mandos
127
 
   From XML sources directly?
128
 
 
129
 
* Side Stuff
130
 
** TODO Locate which packet move the other bin/sh when busy box is deactivated
131
 
** 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]]
132
180
 
133
181
 
134
182
#+STARTUP: showall