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