Mercurial > dovecot > original-hg > dovecot-1.2
annotate TODO @ 7603:58c7f5c31db1 HEAD 1.1.rc8
Released v1.1.rc8.
author | Timo Sirainen <tss@iki.fi> |
---|---|
date | Tue, 03 Jun 2008 01:02:31 +0300 |
parents | 79857a116d28 |
children | 8e206e25a142 |
rev | line source |
---|---|
7575 | 1 - recent assert. both with mbox and maildir. |
7502 | 2 - sieve-cmu.c crash: i_assert(buf->used - 1 == part->body_size.physical_size); |
7449 | 3 - convert plugin: Create a r/w lock for a file. It's read-locked if |
4 conversion isn't wanted and released when process dies. If conversion is | |
5 wanted and write-lock succeeds, conversion is done, if write-lock doesn't | |
6 succeed it fallbacks to using the old storage. When process is exiting it | |
7 again tries to write-lock and do the conversion. Add a parameter that | |
8 specifies if conversion should be done. | |
7259 | 9 - lucene: handle replacement chars? |
10 - squat: | |
7380 | 11 - wrong indexid |
12 - fts_build_init() assertion failed: (last_uid < last_uid_locked) | |
7259 | 13 - is locking done right? it reads header without file being locked? |
14 - split after ~8 bytes? | |
7391 | 15 - expunges are delayed until more mails are added |
7259 | 16 - test replacement chars (SEARCH / SORT / Squat) |
6987 | 17 |
7575 | 18 - dbox: |
19 - "File unexpectedly lost" doesn't get fixed by itself | |
20 - Fix support for multi-message files | |
7603 | 21 - Delete dovecot-keywords and dovecot-uidlist after all maildir files |
22 have been converted to native dbox | |
7449 | 23 - DEBUG: buffer overflow checking code probably doesn't handle a successful |
24 t_try_realloc() or pool_alloconly_realloc() properly | |
6987 | 25 - cache: compress when we can drop temporary fields. |
26 - POP3 UIDL caching | |
27 - ACL: "foo/bar" in public namespace -> LIST "" % doesn't show "foo" | |
28 - new %modifier for reverse DNS lookups with a cache | |
29 - auth_gssapi_hostname = %Xl | |
30 - proxying would also want DNS lookups, but not reverse.. | |
7072 | 31 - ldap domain lookups which set the base for user lookup |
7502 | 32 - ldap: same attribute can't be used for multiple values. |
33 - ldap: multiple attributes can't be merged to a single value. | |
5621 | 34 |
7575 | 35 - Per-user options: |
36 - Deny deleting non-empty mailboxes | |
37 - Disable IDLE "still here" notifications | |
38 | |
6105 | 39 - maildir+pop3/deliver fast updates: |
40 - with locking enabled, pop3 could just keep the one and same sync lock and | |
41 do the whole thing using sync transaction | |
42 - don't update dovecot-uidlist if dovecot.index.cache doesn't exist / | |
43 there's nothing to cache | |
44 - if all messages are expunged and there are no unknown extensions in index, | |
45 unlink dovecot.index and rotate log and add some initial useful info to | |
46 the log (uidvalidity, nextuid) | |
5621 | 47 |
6105 | 48 - maildir |
6987 | 49 - don't allow more than 26 keywords |
6105 | 50 - physical separator could be configurable |
51 - setting flags to same as they already are causes unnecessary | |
52 rename()s with identical parameters | |
53 - deliver+maildir: if new mails are in new/ or cur/ they're not added to | |
54 dovecot-uidlist but newly saved mails are, so UIDs will be in wrong order | |
55 - maildir_copy_with_hardlinks: We're currently first hardlinking to tmp/ and | |
56 then rename()ing. This wouldn't be necessary if uidlist syncing noticed | |
57 that someone else already had added them to uidlist, and the existing UIDs | |
58 could be assigned to them in the index. | |
6754 | 59 - copying should copy already-cached data |
6105 | 60 - maildir_copy_preserve_filename=yes has a race condition causing "Append with |
61 UID n, but next_uid = y" errors when quota plugin is loaded. Practically | |
62 won't happen except in stress testing. | |
5621 | 63 |
6105 | 64 - mbox |
65 - UID renumbering doesn't really work after all? | |
66 - still problems with CRLF mboxes.. | |
67 | |
6987 | 68 - mail_log: log appends. support from and subject fields |
69 - proxy: If remote server disconnects on login: | |
70 login: tried to change state 2 -> 2 | |
6105 | 71 - logging consistency: |
72 http://www.dovecot.org/list/dovecot/2007-April/021532.html | |
73 - EXPUNGE command in read-only mailbox should give an error message if | |
74 there are messages marked as \Deleted? | |
75 - dovecot -o setting=something overriding | |
5557 | 76 - file_cache: we're growing the mmap in page size blocks, which is horribly |
77 slow if mremap() doesn't exist. | |
78 - login_max_processes_count shouldn't count proxying processes | |
79 | |
80 - Allow %variables in mail_chroot setting | |
81 - something should be done to umask setting. we should be creating files with | |
82 0666 or 0777 and rely on umask, but we shouldn't do that unless we're sure | |
83 that the umask is the wanted one (ie. imap/pop3 process wasn't started | |
84 directly) | |
85 - ssl_verify_client_cert isn't working if the SSL cert doesn't have CRL | |
86 | |
3087 | 87 - keywords: |
88 - add some limits to how many there can be | |
89 - don't return \* in PERMANENTFLAGS when we're full | |
90 - remove unused keywords? | |
2774 | 91 |
92 - caching | |
3904 | 93 - force bits should be used only for nonregistered fields |
94 - change envelope parsing not to use get_headers() so imap.envelope can | |
95 actually be cached without all the headers.. | |
6105 | 96 - if there's no other pressure for compression, we should do it when |
97 enough temp fields are ready to be dropped | |
3904 | 98 - we could try compressing same field values into a single |
99 location in cache file. | |
7259 | 100 - place some maximum limit of fields to cache file? maybe some soft and |
101 hard limits, so when soft limit is reached drop fields that have | |
102 been used only once. when hard limit is reached drop any fields to get | |
103 more space. all this to avoid cache file growing infinitely. | |
2774 | 104 |
3087 | 105 - mbox |
4449 | 106 - dirty state should be stored to index (with mbox_very_dirty_syncs) |
107 - after some locking timeouts: mbox-lock.c: line 518 (mbox_lock): | |
108 assertion failed: (lock_type == F_RDLCK || mbox->mbox_lock_type != F_RDLCK) | |
109 - controldir for mboxes too and place subscriptions file there? | |
3904 | 110 - syncing existing indexes takes 4x longer than creating new one, why? |
111 - how well does dirty sync + status work? it reads the last mail every | |
112 time? not very good.. | |
113 - always add empty line. make the parser require it too? syncing should | |
114 make sure there always exists two LFs at end of file. raw-mbox-stream | |
115 should make sure the last message ends with LF even if it doesn't exist | |
116 in the file | |
117 - Quote "From ", unquote ">From " | |
4449 | 118 - COPY doesn't work to itself (lock assert crash, for now just disallowed) |
2143 | 119 - if we can't create dotlock file for mbox, make sure it still can be |
120 selected in read-only state | |
121 | |
122 - index | |
123 - if log file is lost, generate it from old and new index | |
3087 | 124 - read-only support for mailboxes where we don't have write-access |
6105 | 125 - index file format changes: |
126 - pack UIDs to beginning of file with UID ranges | |
127 - use squat-like compressed uid ranges everywhere | |
6782 | 128 - write first extension intros in dovecot.index.log always with names |
129 - or better yet, drop the intro concept completely as it is now | |
130 - add "transaction boundaries" so we know which records belong to a | |
131 single transaction. | |
132 - only after that we can remove the transaction log offset | |
133 overwriting (otherwise we can get partial transactions in views) | |
2143 | 134 |
4449 | 135 - namespaces |
136 - namespaces: add new "auto_disable" flag so if the mailbox can't be opened | |
137 (eg. file doesn't exist), just ignore the problem and disable the | |
138 namespace | |
139 | |
140 - lib-storage | |
5557 | 141 - rename: allow moving between storages, as long as they're of same type |
4449 | 142 - x search charset asdf all -> should fail |
3087 | 143 - should we allow following symlinks in mbox/maildirs? they are now. |
144 - if we implement shared mailboxes with shared indexes, never do that or | |
145 others could symlink your personal mailboxes and see the indexes | |
146 created for it which may contain envelope etc. data | |
147 - this allows circular mailbox hierarchies which should be prevented by | |
148 eg. allowing max. 20 hierarchies. | |
149 | |
150 - login | |
4449 | 151 - when pipelining login command + post-login commands, login process should |
152 pass the command to imap/pop3 process (at least one pop3 client does this) | |
153 - imap-login: Master sent reply with unknown tag 1. client closed | |
154 connection at the exact same time master was logging it in? | |
155 see master_request_abort() | |
3087 | 156 - Digest-MD5: support integrity protection, and maybe crypting. Do it |
157 through login process like SSL is done? | |
158 - x login foo bar | |
159 x NO Authentication failed. | |
160 x login cras pass | |
161 * BYE Disconnected for inactivity. | |
162 ^ but it's not disconnecting! (buggy dovecot-auth not replying) | |
3744 | 163 probably because userdb lookup didn't reply, and fd was already sent |
164 for master.. should imap-login be handling it anymore?.. | |
3087 | 165 - imap-login: Authenticate PLAIN failed: Authentication failed: |
166 Authentication server isn't connected, try again later.. [127.0.0.1] | |
167 ^ NO Authentication failed. (should be Temporary login failure!) | |
168 - if auth process dies, login process should retry authentication if | |
169 possible. or if not, disconnect the client so it doesn't think the auth | |
170 failed. | |
171 - send client IP immediately after accept() to master process. make sure | |
172 master shows the IP if login dies unexpectedly. master should probably | |
173 also kill the login process if it doesn't kill itself soon enough.. or | |
174 maybe just log the IP immediately. | |
474 | 175 |
3087 | 176 - auth |
4449 | 177 - with blocking passdb we're not caching lookups if the password was wrong |
5557 | 178 - non-plaintext authentication doesn't support all features: |
179 - multiple passdbs don't work, only the first one is used | |
180 - auth cache's last_success password change check doesn't exist | |
6987 | 181 - allow returning multiple gids |
3087 | 182 - SIGHUP restarts auth processes .. but does it wait until they've finished |
183 with all requests? no. | |
184 - does dovecot-auth really break when it runs out of fds? | |
4449 | 185 - dovecot-auth should limit how fast authentication requests are allowed |
186 from login processes. especially if there's one login/connection the speed | |
3087 | 187 should be something like once/sec. also limit how fast to accept new |
188 connections. | |
189 - support read-only logins. user could with alternative password get only | |
190 read-access to mails so mails could be read relatively safely with | |
191 untrusted computers. Maybe always send [ALERT] about the previous | |
192 read-only login time with IP? | |
4449 | 193 - dovecot-auth workers: create a separate dovecot-pam worker which shares |
194 pretty much all code with dovecot-auth worker but isn't linked against | |
195 any libraries. or..? this might be difficult to do, especially because the | |
196 workers currently can handle any kind of passdb/userb requests. perhaps | |
197 there should be a completely separate simple PAM authenticator binary. | |
526 | 198 |
3087 | 199 - master |
200 - configurable syslog prefix | |
3450 | 201 - if there are duplicate settings, complain about it |
3087 | 202 |
203 - quota | |
204 - if dovecot-uidlist can't be written, assume the new mails have UIDs | |
205 beginning from uidlist.next_uid. Whenever mails are expunged, overwrite | |
206 the next_uid field with the current highest next_uid. Whenever we have | |
207 assumed UIDs and uidlist gets updated, throw the client out with | |
208 "inconsist mailbox". | |
209 | |
210 - ssl | |
211 - add setting: ssl_options = bitmask. by default we enable all openssl | |
212 workarounds, this could be used to disable some of them | |
213 - gnutls support isn't working | |
965 | 214 |
3087 | 215 - search |
6232 | 216 - message header search: we should ignore LWSP between two MIME blocks(?) |
217 - message_search_init() could accept multiple search keywords so we | |
1250 | 218 wouldn't need to call it separately for each one (so we wouldn't need |
219 to parse the message multiple times). | |
220 - could optionally support scanning inside file attachments and use | |
221 plugins to extract text out of them (word, excel, pdf, etc. etc.) | |
222 - Create our own extension: When searching with TEXT/BODY, return | |
223 the message text surrounding the keywords just like web search engines | |
224 do. like: SEARCH X-PRINT-MATCHES TEXT "hello" -> * SEARCH 1 "He said: | |
225 Hello world!" 2 "Hello, I'm ...". This would be especially useful with | |
226 the above attachment scanning. | |
0 | 227 |
4449 | 228 - deliver |
5557 | 229 - recipient_delimiter setting so user+mailbox@domain works directly |
4449 | 230 |
3087 | 231 - general |
4449 | 232 - stop using atol(), atoi(), strtoul() etc. in places where we actually |
233 care about what they return, and rather create our own function which | |
234 checks if the input overflows the integer, and if so call i_fatal() | |
3087 | 235 - rfc2231 continuation support (useless?) |
1553 | 236 - rfc2557 support for BODYSTRUCTURE, as specified by RFC3501 |
4449 | 237 - LMTP server |
238 - ability to build plugins statically into the binaries | |
3087 | 239 - ~/.dovecotrc to override system wide settings. namespace settings should |
240 override all the previous namespace settings instead of adding new. | |
241 - things break if next_uid gets to 2^32 | |
376
fd1fc4cf11b7
updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents:
340
diff
changeset
|
242 |
fd1fc4cf11b7
updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents:
340
diff
changeset
|
243 capabilities: |
4449 | 244 - preferrably all should be implemented as plugins |
376
fd1fc4cf11b7
updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents:
340
diff
changeset
|
245 - possibility to disable them from config file |
3087 | 246 - THREAD=ORDEREDSUBJECT - although pretty useless I'd think. |
5621 | 247 - acl (rfc4314) |
376
fd1fc4cf11b7
updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents:
340
diff
changeset
|
248 - id (rfc2971) |
fd1fc4cf11b7
updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents:
340
diff
changeset
|
249 - must be configurable what gets sent, default to only name=Dovecot |
fd1fc4cf11b7
updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents:
340
diff
changeset
|
250 - separate pre/post-login settings |
fd1fc4cf11b7
updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents:
340
diff
changeset
|
251 - optionally log configured parts of the client information, but only |
fd1fc4cf11b7
updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents:
340
diff
changeset
|
252 once, probably at the same time as logging "Logged in", |
fd1fc4cf11b7
updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents:
340
diff
changeset
|
253 "Disconnected", etc. |
fd1fc4cf11b7
updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents:
340
diff
changeset
|
254 - remember to force truncating values longer than 30 chars, |
fd1fc4cf11b7
updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents:
340
diff
changeset
|
255 especially before logging |
fd1fc4cf11b7
updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents:
340
diff
changeset
|
256 - mailbox-referrals (rfc2193) |
fd1fc4cf11b7
updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents:
340
diff
changeset
|
257 - this is useful whenever we would otherwise need to make the |
fd1fc4cf11b7
updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents:
340
diff
changeset
|
258 connection ourself. for example load balancing and shared mailboxes |
fd1fc4cf11b7
updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents:
340
diff
changeset
|
259 requiring another UID to run. |
fd1fc4cf11b7
updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents:
340
diff
changeset
|
260 - this rfc defines no exact way for server to detect if client |
fd1fc4cf11b7
updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents:
340
diff
changeset
|
261 supports referrals or not. I don't think there's much point in |
fd1fc4cf11b7
updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents:
340
diff
changeset
|
262 supporting only referrals, as most clients don't support them. |
fd1fc4cf11b7
updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents:
340
diff
changeset
|
263 Instead we should return referrals when we know that client |
fd1fc4cf11b7
updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents:
340
diff
changeset
|
264 supports them, otherwise do the connecting ourself. If client |
fd1fc4cf11b7
updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents:
340
diff
changeset
|
265 issues RLIST or RLSUB command, it's safe to assume it supports |
fd1fc4cf11b7
updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents:
340
diff
changeset
|
266 referrals. |
fd1fc4cf11b7
updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents:
340
diff
changeset
|
267 - for load balancing this works just fine, but what about shared |
fd1fc4cf11b7
updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents:
340
diff
changeset
|
268 mailboxes which require different UID? If we login with our own |
fd1fc4cf11b7
updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents:
340
diff
changeset
|
269 username, we end up with our own UID instead of what we wanted. |
fd1fc4cf11b7
updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents:
340
diff
changeset
|
270 IMAP URLs don't support separated authorization id which would |
fd1fc4cf11b7
updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents:
340
diff
changeset
|
271 have made this very easy.. We could give the "userid@group" as |
fd1fc4cf11b7
updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents:
340
diff
changeset
|
272 userid, but clients probably treat it as different userid and |
fd1fc4cf11b7
updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents:
340
diff
changeset
|
273 ask the password again. |
fd1fc4cf11b7
updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents:
340
diff
changeset
|
274 - problems, problems, .. maybe not worth the trouble. |
fd1fc4cf11b7
updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents:
340
diff
changeset
|
275 - drafts: |
fd1fc4cf11b7
updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents:
340
diff
changeset
|
276 - http://www.imc.org/ids.html |
fd1fc4cf11b7
updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents:
340
diff
changeset
|
277 - annotate (draft-ietf-imapext-annotate) |
fd1fc4cf11b7
updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents:
340
diff
changeset
|
278 - per-message annotations. this will be major change. especially |
fd1fc4cf11b7
updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents:
340
diff
changeset
|
279 because currently there's no suitable storage for them, and |
fd1fc4cf11b7
updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents:
340
diff
changeset
|
280 they'll probably change all the time.. maybe if we moved into |
fd1fc4cf11b7
updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents:
340
diff
changeset
|
281 berkeley db to store the .data file and these annotations. |
1178 | 282 - this is separate problem from index files. indexes are treated as |
283 temporary files, annotations are permanent data. we'd have to | |
284 support non-db way to do this too, which would probably be just a | |
285 simple (slow) text file. | |
5621 | 286 - use lib-dict probably |
287 - metadata (draft-daboo-imap-annotatemore) | |
376
fd1fc4cf11b7
updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents:
340
diff
changeset
|
288 - server and per-mailbox annotations. much easier than |
fd1fc4cf11b7
updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents:
340
diff
changeset
|
289 per-message annotations, but they'd be easier to place into |
fd1fc4cf11b7
updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents:
340
diff
changeset
|
290 db as well. |
5621 | 291 - lib-dict, again |
376
fd1fc4cf11b7
updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents:
340
diff
changeset
|
292 - binary (draft-nerenberg-imap-binary) |