/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
2
2
 
3
3
* [#A] README file
4
4
 
5
 
* [#A] COPYING file
6
 
  [[file:/usr/share/common-licenses/GPL-3][GPLv3]]
7
 
 
8
 
* Mandos-client
9
 
** [#A] Man page: man8/mandos-client.8mandos
10
 
*** EXIT STATUS
11
 
    Create this section
12
 
*** USAGE
 
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
13
15
    Describe the plus sign syntax for passing options from crypttab
 
16
*** EXIT STATUS
 
17
    Text needed
14
18
*** EXAMPLES
15
19
    Examples of normal usage, debug usage, debugging single or all
16
20
    plugins, examples of crypttab lines with plus syntax, etc.
17
21
*** FILES
18
 
    Describe plugin directory
19
 
*** DIAGNOSTICS
20
 
    Create this section
 
22
    Text needed
21
23
*** SECURITY
22
 
    Create this section
 
24
    Text needed
23
25
*** NOTES
24
 
    Create this section (if needed)
 
26
    Text needed
25
27
*** BUGS
26
 
    Create this section
 
28
    Text needed
27
29
*** SEE ALSO
28
 
    Refer to mandos(8), password-request(8mandos), and
29
 
    password-prompt(8mandos)
30
 
** Use asprintf instead of malloc and strcat?
31
 
** use strsep instead of strtok?
32
 
** use config file in addition to arguments
33
 
** pass things in environment, like device name, etc
34
 
** Fallback
35
 
   As a fallback, if no plugins can be found or if all of them failed,
36
 
   run getpass(3) itself.
 
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
37
34
 
38
35
* Password-request
39
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.
40
43
*** DESCRIPTION
41
44
    Move options to new OPTIONS section.
42
45
    State that this command is not meant to be invoked directly, but
60
63
    Create this section
61
64
*** SEE ALSO
62
65
    Refer to mandos-client(8mandos) and password-prompt(8mandos)
63
 
** Use asprintf instead of malloc and memcpy?
64
66
** IPv4 support
65
67
** use strsep instead of strtok?
66
 
** Do not depend on GPG key rings on disk
67
 
   This would mean creating new GPG key rings with GPGME by importing
68
 
   the key files from scratch on every program start.
 
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.
69
71
 
70
72
* Password-prompt
71
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".
72
77
*** DESCRIPTION
73
78
    Move options to new OPTIONS section.
74
79
*** EXIT STATUS
93
98
   does not.  See also [[http://sources.redhat.com/ml/libc-alpha/2003-05/msg00251.html][Marcus Brinkmann: Re: getpass obsolete?]] and
94
99
   [[http://article.gmane.org/gmane.comp.lib.glibc.alpha/4906][Petter Reinholdtsen: Re: getpass obsolete?]], and especially also
95
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"?
96
102
 
97
103
* Mandos (server)
98
104
** [#A] Command man page: man8/mandos.8
130
136
   [[info:standards:Option%20Table][Table of Long Options]]
131
137
** Implement --socket
132
138
   [[info:standards:Option%20Table][Table of Long Options]]
 
139
** Date+time on console log messages
 
140
   Is this the default?
133
141
 
134
142
* Mandos-tools/utilities
135
143
  All of this probably using D-Bus
138
146
** Enable client
139
147
 
140
148
* Installer
141
 
** DONE [#A] Change initrd.img file to not be publically readable
142
 
   /etc/initramfs-tools/conf.d/mandos
143
 
   UMASK=027
144
 
** Update initrd.img after installation
145
 
** [#A] Create mandos user and group for server
146
 
** [#A] Create /var/run/mandos directory with perm and ownership
 
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.
147
161
 
148
162
* [#A] Package
 
163
** /usr/share/initramfs-tools/hooks/mandos
 
164
*** Do not install in initrd.img if configured not to
 
165
    Use "/etc/initramfs-tools/conf.d/mandos"?  Definitely a debconf
 
166
    question.
149
167
** /etc/bash_completion.d/mandos
150
 
** /etc/initramfs-tools/hooks/mandos
151
 
   [[file:/usr/share/doc/initramfs-tools/examples/example_hook][Example initramfs-tools hook script]]
152
 
*** Create GPG key ring files in initrd
 
168
*** From xml sources directly?
 
169
** unperish
 
170
** bzr-builddeb
153
171
 
154
172
* INSTALL file
155
173