/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:
2
2
 
3
3
* [#A] README file
4
4
 
5
 
* plugin-runner
 
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
6
9
** [#B] Add more comments to code
7
10
** [#B] Add more if(debug) calls
8
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
9
34
 
10
 
* password-request
 
35
* Password-request
11
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.
12
43
*** DESCRIPTION
 
44
    Move options to new OPTIONS section.
13
45
    State that this command is not meant to be invoked directly, but
14
46
    is run as a plugin from mandos-client(8) and only run in the
15
47
    initrd environment, not the real system.
16
 
*** PURPOSE
17
 
    As in mandos.xml
18
 
*** OVERVIEW
19
 
    As in mandos.xml
20
48
*** EXIT STATUS
21
 
*** ENVIRONMENT
22
 
    Note that it does *not* currently use cryptsource or crypttarget.
 
49
    Create this section
 
50
*** EXAMPLES
 
51
    Examples of normal usage, debug usage, debugging by connecting
 
52
    directly, etc.
23
53
*** FILES
24
54
    Describe the key files and the key ring files.  Also note that
25
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)
26
62
*** BUGS
27
 
*** EXAMPLE
28
 
    Examples of normal usage, debug usage, debugging by connecting
29
 
    directly, etc.
30
 
*** SECURITY
 
63
    Create this section
31
64
*** SEE ALSO
32
 
    Update from mandos.xml
33
 
** [#B] Temporarily lower kernel log level
34
 
   for less printouts during sucessfull boot.
 
65
    Refer to mandos-client(8mandos) and password-prompt(8mandos)
35
66
** IPv4 support
36
67
** use strsep instead of strtok?
37
68
** Do not depend on GnuPG key rings on disk
38
69
   This would mean creating new GnuPG key rings with GPGME by
39
70
   importing the key files from scratch on every program start.
40
 
** Keydir move: /etc/mandos -> /etc/keys/mandos
41
 
   Must create in preinst if not pre-depending on cryptsetup
42
71
 
43
 
* password-prompt
44
 
** [#C] Use getpass(3)?
 
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)?
45
97
   Man page says "obsolete", but [[info:libc:getpass][GNU LibC Manual: Reading Passwords]]
46
98
   does not.  See also [[http://sources.redhat.com/ml/libc-alpha/2003-05/msg00251.html][Marcus Brinkmann: Re: getpass obsolete?]] and
47
99
   [[http://article.gmane.org/gmane.comp.lib.glibc.alpha/4906][Petter Reinholdtsen: Re: getpass obsolete?]], and especially also
48
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"?
49
102
 
50
 
* mandos (server)
 
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)
51
127
** [#A] /etc/init.d/mandos-server               :teddy:
52
 
** [#B] Log level                               :bugs:
 
128
** Log level
53
129
** /etc/mandos/clients.d/*.conf
54
130
   Watch this directory and add/remove/update clients?
55
131
** config for TXT record
56
 
** [#B] Run-time communication with server      :bugs:
 
132
** Run-time communication with server
57
133
   Probably using D-Bus
58
134
   See also [[*Mandos-tools]]
59
 
** Implement --foreground                       :bugs:
 
135
** Implement --foreground
60
136
   [[info:standards:Option%20Table][Table of Long Options]]
61
137
** Implement --socket
62
138
   [[info:standards:Option%20Table][Table of Long Options]]
63
 
** Date+time on console log messages            :bugs:
 
139
** Date+time on console log messages
64
140
   Is this the default?
65
141
 
66
142
* Mandos-tools/utilities
69
145
** Disable client
70
146
** Enable client
71
147
 
72
 
* Man pages
73
 
** Use xinclude for common sections
74
 
   Like authors, etc.
75
 
 
76
 
 
77
148
* Installer
78
 
** Client-side
 
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
79
153
*** Update initrd.img after installation
80
 
    This seems to use some kind of "trigger" system
81
 
    [[file:/usr/share/doc/dpkg/triggers.txt.gz]]
82
 
    dpkg-trigger(1), deb-triggers(5)
83
 
*** Keydir move: /etc/mandos -> /etc/keys/mandos
84
 
    Must create in preinst if not pre-depending on cryptsetup
85
 
*** mandos-keygen
86
 
**** "--passfile" option
87
 
     Using the "secfile" option instead of "secret"
88
 
**** [#A] "--test" option
89
 
     For testing decryption before rebooting.
90
 
** Server-side
 
154
** Server
91
155
*** [#A] Create mandos user and group for server
92
156
*** [#A] Create /var/run/mandos directory with perm and ownership
93
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
 
94
162
* [#A] Package
95
163
** /usr/share/initramfs-tools/hooks/mandos
96
 
*** Do not install in initrd.img if configured not to.
 
164
*** Do not install in initrd.img if configured not to
97
165
    Use "/etc/initramfs-tools/conf.d/mandos"?  Definitely a debconf
98
166
    question.
99
167
** /etc/bash_completion.d/mandos
100
 
   From XML sources directly?
 
168
*** From xml sources directly?
101
169
** unperish
102
170
** bzr-builddeb
103
171