annotate TODO @ 5317:4bc257a1874e HEAD

Crashfix
author Timo Sirainen <tss@iki.fi>
date Thu, 15 Mar 2007 03:34:44 +0200
parents a2d8624f8f2e
children 657406346288
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
3087
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
1 - keywords:
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
2 - add some limits to how many there can be
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
3 - don't return \* in PERMANENTFLAGS when we're full
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
4 - remove unused keywords?
2774
689e4235681f updated
Timo Sirainen <tss@iki.fi>
parents: 2143
diff changeset
5
689e4235681f updated
Timo Sirainen <tss@iki.fi>
parents: 2143
diff changeset
6 - caching
3904
9f8d69605d4b 1.0 beta1 released
Timo Sirainen <tss@iki.fi>
parents: 3744
diff changeset
7 - force bits should be used only for nonregistered fields
9f8d69605d4b 1.0 beta1 released
Timo Sirainen <tss@iki.fi>
parents: 3744
diff changeset
8 - change envelope parsing not to use get_headers() so imap.envelope can
9f8d69605d4b 1.0 beta1 released
Timo Sirainen <tss@iki.fi>
parents: 3744
diff changeset
9 actually be cached without all the headers..
9f8d69605d4b 1.0 beta1 released
Timo Sirainen <tss@iki.fi>
parents: 3744
diff changeset
10 - compression should drop fields with last_used <
9f8d69605d4b 1.0 beta1 released
Timo Sirainen <tss@iki.fi>
parents: 3744
diff changeset
11 (latest_mail_index_date - month)
9f8d69605d4b 1.0 beta1 released
Timo Sirainen <tss@iki.fi>
parents: 3744
diff changeset
12 - we could try compressing same field values into a single
9f8d69605d4b 1.0 beta1 released
Timo Sirainen <tss@iki.fi>
parents: 3744
diff changeset
13 location in cache file.
9f8d69605d4b 1.0 beta1 released
Timo Sirainen <tss@iki.fi>
parents: 3744
diff changeset
14 - support caching all message headers. this could be useful when
9f8d69605d4b 1.0 beta1 released
Timo Sirainen <tss@iki.fi>
parents: 3744
diff changeset
15 indexes are in local disk but actual mails are accessed through NFS.
2774
689e4235681f updated
Timo Sirainen <tss@iki.fi>
parents: 2143
diff changeset
16
3087
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
17 - mbox
4449
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
18 - dirty state should be stored to index (with mbox_very_dirty_syncs)
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
19 - after some locking timeouts: mbox-lock.c: line 518 (mbox_lock):
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
20 assertion failed: (lock_type == F_RDLCK || mbox->mbox_lock_type != F_RDLCK)
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
21 - controldir for mboxes too and place subscriptions file there?
3904
9f8d69605d4b 1.0 beta1 released
Timo Sirainen <tss@iki.fi>
parents: 3744
diff changeset
22 - size.physical isn't cached, but should it even be? ..
9f8d69605d4b 1.0 beta1 released
Timo Sirainen <tss@iki.fi>
parents: 3744
diff changeset
23 - syncing existing indexes takes 4x longer than creating new one, why?
9f8d69605d4b 1.0 beta1 released
Timo Sirainen <tss@iki.fi>
parents: 3744
diff changeset
24 - how well does dirty sync + status work? it reads the last mail every
9f8d69605d4b 1.0 beta1 released
Timo Sirainen <tss@iki.fi>
parents: 3744
diff changeset
25 time? not very good..
9f8d69605d4b 1.0 beta1 released
Timo Sirainen <tss@iki.fi>
parents: 3744
diff changeset
26 - always add empty line. make the parser require it too? syncing should
9f8d69605d4b 1.0 beta1 released
Timo Sirainen <tss@iki.fi>
parents: 3744
diff changeset
27 make sure there always exists two LFs at end of file. raw-mbox-stream
9f8d69605d4b 1.0 beta1 released
Timo Sirainen <tss@iki.fi>
parents: 3744
diff changeset
28 should make sure the last message ends with LF even if it doesn't exist
9f8d69605d4b 1.0 beta1 released
Timo Sirainen <tss@iki.fi>
parents: 3744
diff changeset
29 in the file
9f8d69605d4b 1.0 beta1 released
Timo Sirainen <tss@iki.fi>
parents: 3744
diff changeset
30 - Quote "From ", unquote ">From "
4449
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
31 - COPY doesn't work to itself (lock assert crash, for now just disallowed)
3904
9f8d69605d4b 1.0 beta1 released
Timo Sirainen <tss@iki.fi>
parents: 3744
diff changeset
32 - keep mbox lock for two extra seconds after sync (do we really need to?)
2143
11c1f1203b65 updated
Timo Sirainen <tss@iki.fi>
parents: 1585
diff changeset
33 - move /var/mail/user to ~/mbox if ~/mbox exists.. supposedly this
11c1f1203b65 updated
Timo Sirainen <tss@iki.fi>
parents: 1585
diff changeset
34 could be useful if /var/mail doesn't have quota, but ~/mail does.
11c1f1203b65 updated
Timo Sirainen <tss@iki.fi>
parents: 1585
diff changeset
35 now, what do we then do if we can move only some of the mails?..
11c1f1203b65 updated
Timo Sirainen <tss@iki.fi>
parents: 1585
diff changeset
36 - if we can't create dotlock file for mbox, make sure it still can be
11c1f1203b65 updated
Timo Sirainen <tss@iki.fi>
parents: 1585
diff changeset
37 selected in read-only state
11c1f1203b65 updated
Timo Sirainen <tss@iki.fi>
parents: 1585
diff changeset
38
3087
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
39 - maildir
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
40 - if indexes exist but dovecot-uidlist doesn't, it's not tried to be
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
41 recreated
3450
5c92b51f2b38 updated
Timo Sirainen <tss@iki.fi>
parents: 3087
diff changeset
42 - with pop3 don't move messages from new/ to cur/ before RETR
4449
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
43 - when sorting maildir files, sort based on Mxxxx first so the files are
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
44 sorted always in ascending order. required for proper out-of-quota
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
45 uidlist handling
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
46
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
47 - dbox
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
48 - keyword list header locking isn't correct now. saving uses file's
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
49 dotlock, sync uses sync lock. and what about reading?...
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
50 - dbox is leaking index fds
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
51 - append_offset in header shouldn't be trusted
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
52 - show in index if there are expunge-flagged mails in the mailbox
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
53 - pop3_lock_session doesn't work
3087
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
54
2143
11c1f1203b65 updated
Timo Sirainen <tss@iki.fi>
parents: 1585
diff changeset
55 - index
4449
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
56 - dd if=/dev/zero of=dovecot.index bs=1024 count=1 -> NOOP -> crash!
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
57 - delete >30min old dovecot.index.log.2 files when opening index?
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
58 - Panic: file mail-index.c: line 844 (mail_index_sync_from_transactions):
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
59 assertion failed: (hdr.messages_count == (*map)->hdr.messages_count)
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
60 - Corrupted transaction log file dovecot.index.log: record size wrong (type 0x4, 20 % 12 != 0)
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
61 -> mail-index.c: line 841 (mail_index_sync_from_transactions): assertion failed: (hdr.messages_count == (*map)->hdr.messages_count)
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
62 - optimize initial left_idx in _view_lookup_uid_range()
2143
11c1f1203b65 updated
Timo Sirainen <tss@iki.fi>
parents: 1585
diff changeset
63 - if log file is lost, generate it from old and new index
3087
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
64 - transaction log: when replacing log with a same sequence, we remove it
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
65 from log's file list, but we don't do anything to existing log views.
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
66 this can crash later in mail_transaction_log_view_set() because 'first'
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
67 is from log list, while we're comparing it into view->tail which it never
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
68 is. also overwriting it leaks memory..
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
69 - read-only support for mailboxes where we don't have write-access
2143
11c1f1203b65 updated
Timo Sirainen <tss@iki.fi>
parents: 1585
diff changeset
70
4449
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
71 - namespaces
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
72 - namespaces: add new "auto_disable" flag so if the mailbox can't be opened
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
73 (eg. file doesn't exist), just ignore the problem and disable the
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
74 namespace
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
75 - subscribe <namespace prefix> doesn't work.
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
76 - namespaces don't work in plugins: acl, trash, convert
3087
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
77 - subscriptions file should contain namespace prefixes. at least optionally.
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
78 there's the subscriptions = yes setting now for namespaces.. do it so that
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
79 if prefix = "" has subscriptions, it contains prefixes. otherwise not.
4128
9198a457f022 1.0.beta4 released
Timo Sirainen <tss@iki.fi>
parents: 3995
diff changeset
80 - for shared/public namespaces default to "no"
4449
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
81
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
82 - lib-storage
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
83 - calls fsync()s etc. less often (when copying). optionally disable them.
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
84 - x search charset asdf all -> should fail
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
85 - index_removal_timeout gets leaked in some conditions. how?
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
86 - subscribe: IMAP(anonymous): open(anonymous/mail/.temp...) failed: Permission denied
3087
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
87 - should we allow following symlinks in mbox/maildirs? they are now.
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
88 - if we implement shared mailboxes with shared indexes, never do that or
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
89 others could symlink your personal mailboxes and see the indexes
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
90 created for it which may contain envelope etc. data
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
91 - this allows circular mailbox hierarchies which should be prevented by
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
92 eg. allowing max. 20 hierarchies.
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
93
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
94 - login
4449
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
95 - when pipelining login command + post-login commands, login process should
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
96 pass the command to imap/pop3 process (at least one pop3 client does this)
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
97 - imap-login: Master sent reply with unknown tag 1. client closed
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
98 connection at the exact same time master was logging it in?
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
99 see master_request_abort()
3087
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
100 - Digest-MD5: support integrity protection, and maybe crypting. Do it
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
101 through login process like SSL is done?
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
102 - x login foo bar
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
103 x NO Authentication failed.
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
104 x login cras pass
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
105 * BYE Disconnected for inactivity.
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
106 ^ but it's not disconnecting! (buggy dovecot-auth not replying)
3744
d362e79fdb97 Alpha5 release
Timo Sirainen <tss@iki.fi>
parents: 3670
diff changeset
107 probably because userdb lookup didn't reply, and fd was already sent
d362e79fdb97 Alpha5 release
Timo Sirainen <tss@iki.fi>
parents: 3670
diff changeset
108 for master.. should imap-login be handling it anymore?..
3087
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
109 - imap-login: Authenticate PLAIN failed: Authentication failed:
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
110 Authentication server isn't connected, try again later.. [127.0.0.1]
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
111 ^ NO Authentication failed. (should be Temporary login failure!)
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
112 - if auth process dies, login process should retry authentication if
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
113 possible. or if not, disconnect the client so it doesn't think the auth
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
114 failed.
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
115 - send client IP immediately after accept() to master process. make sure
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
116 master shows the IP if login dies unexpectedly. master should probably
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
117 also kill the login process if it doesn't kill itself soon enough.. or
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
118 maybe just log the IP immediately.
474
47d0c169bdba updated
Timo Sirainen <tss@iki.fi>
parents: 376
diff changeset
119
3087
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
120 - auth
4449
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
121 - ability to specify default password scheme with passwd-file
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
122 - with blocking passdb we're not caching lookups if the password was wrong
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
123 - if PAM child process doesn't finish within a minute, kill it
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
124 - PAM / checkpassword should use passdb-blocking
3087
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
125 - support specifying hex/base64 encoding in password scheme. for example
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
126 {plain-md5.base64}
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
127 - auth cache: cache userdb data too.
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
128 - remove system_user and allow returning multiple gids instead.
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
129 - SIGHUP restarts auth processes .. but does it wait until they've finished
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
130 with all requests? no.
4373
936e90b7d525 Updated
Timo Sirainen <tss@iki.fi>
parents: 4257
diff changeset
131 - post-login-sql-command (userdb command doesn't do because of deliver)
3087
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
132 - does dovecot-auth really break when it runs out of fds?
4449
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
133 - dovecot-auth should limit how fast authentication requests are allowed
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
134 from login processes. especially if there's one login/connection the speed
3087
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
135 should be something like once/sec. also limit how fast to accept new
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
136 connections.
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
137 - support read-only logins. user could with alternative password get only
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
138 read-access to mails so mails could be read relatively safely with
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
139 untrusted computers. Maybe always send [ALERT] about the previous
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
140 read-only login time with IP?
4449
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
141 - dovecot-auth workers: create a separate dovecot-pam worker which shares
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
142 pretty much all code with dovecot-auth worker but isn't linked against
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
143 any libraries. or..? this might be difficult to do, especially because the
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
144 workers currently can handle any kind of passdb/userb requests. perhaps
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
145 there should be a completely separate simple PAM authenticator binary.
526
ab394352fcb3 updated
Timo Sirainen <tss@iki.fi>
parents: 497
diff changeset
146
3087
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
147 - master
4449
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
148 - Support listening in multiple sockets
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
149 - per-user/ip limits..
3087
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
150 - configurable syslog prefix
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
151 - SIGHUP rather shouldn't restart listening sockets if they didn't change..
3450
5c92b51f2b38 updated
Timo Sirainen <tss@iki.fi>
parents: 3087
diff changeset
152 - if there are duplicate settings, complain about it
3087
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
153
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
154 - quota
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
155 - if dovecot-uidlist can't be written, assume the new mails have UIDs
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
156 beginning from uidlist.next_uid. Whenever mails are expunged, overwrite
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
157 the next_uid field with the current highest next_uid. Whenever we have
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
158 assumed UIDs and uidlist gets updated, throw the client out with
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
159 "inconsist mailbox".
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
160 - make sure all syscalls check for ENOSPACE (and ENOACCESS while at it)
4449
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
161 - quota code should probably be always doing some quota_set_critical()
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
162 instead of using mail_storage_set_critical(), so that quota_last_error()
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
163 would work properly
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
164 - if storage=0 is given, use unlimited quota but track it anyway
3087
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
165
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
166 - ssl
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
167 - add setting: ssl_options = bitmask. by default we enable all openssl
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
168 workarounds, this could be used to disable some of them
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
169 - gnutls support isn't working
965
6f005d5d9931 0.99.6 out
Timo Sirainen <tss@iki.fi>
parents: 892
diff changeset
170
3087
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
171 - search
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
172 - message header search: we should ignore LWSP between two MIME blocks
1250
31510fc5e02f updates
Timo Sirainen <tss@iki.fi>
parents: 1211
diff changeset
173 - message_body_search() could accept multiple search keywords so we
31510fc5e02f updates
Timo Sirainen <tss@iki.fi>
parents: 1211
diff changeset
174 wouldn't need to call it separately for each one (so we wouldn't need
31510fc5e02f updates
Timo Sirainen <tss@iki.fi>
parents: 1211
diff changeset
175 to parse the message multiple times).
31510fc5e02f updates
Timo Sirainen <tss@iki.fi>
parents: 1211
diff changeset
176 - message_body_search() could support NULL MessagePart and the searching
31510fc5e02f updates
Timo Sirainen <tss@iki.fi>
parents: 1211
diff changeset
177 could be done while parsing the message. this would need changes to
31510fc5e02f updates
Timo Sirainen <tss@iki.fi>
parents: 1211
diff changeset
178 message_parse() as well.
31510fc5e02f updates
Timo Sirainen <tss@iki.fi>
parents: 1211
diff changeset
179 - could optionally support scanning inside file attachments and use
31510fc5e02f updates
Timo Sirainen <tss@iki.fi>
parents: 1211
diff changeset
180 plugins to extract text out of them (word, excel, pdf, etc. etc.)
31510fc5e02f updates
Timo Sirainen <tss@iki.fi>
parents: 1211
diff changeset
181 - use a trie index for fast text searching, like cyrus squat?
31510fc5e02f updates
Timo Sirainen <tss@iki.fi>
parents: 1211
diff changeset
182 - Create our own extension: When searching with TEXT/BODY, return
31510fc5e02f updates
Timo Sirainen <tss@iki.fi>
parents: 1211
diff changeset
183 the message text surrounding the keywords just like web search engines
31510fc5e02f updates
Timo Sirainen <tss@iki.fi>
parents: 1211
diff changeset
184 do. like: SEARCH X-PRINT-MATCHES TEXT "hello" -> * SEARCH 1 "He said:
31510fc5e02f updates
Timo Sirainen <tss@iki.fi>
parents: 1211
diff changeset
185 Hello world!" 2 "Hello, I'm ...". This would be especially useful with
31510fc5e02f updates
Timo Sirainen <tss@iki.fi>
parents: 1211
diff changeset
186 the above attachment scanning.
0
3b1985cbc908 Initial revision
Timo Sirainen <tss@iki.fi>
parents:
diff changeset
187
3087
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
188 - lib-charset
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
189 - utf8_toupper() is a must. and a bit difficult if we want to do it right.
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
190 - add support for other things than iconv() as well? we could reuse
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
191 the code from cyrus or courier
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
192 - cache iconvs? they'd probably be faster if we just reset the
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
193 conversion instead of opening new one every time. and there will likely
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
194 be only one or two charsets which are used for nearly all conversions.
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
195
4449
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
196 - deliver
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
197 - empty mail gives an error.
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
198 - We should always return EX_* failures and never our own 89 etc.
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
199
3087
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
200 - general
4449
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
201 - add imap_logout_format
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
202 - stop using atol(), atoi(), strtoul() etc. in places where we actually
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
203 care about what they return, and rather create our own function which
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
204 checks if the input overflows the integer, and if so call i_fatal()
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
205 - solaris sendfile is broken?
3087
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
206 - rfc2231 continuation support (useless?)
1553
25cef50e3b0e updated
Timo Sirainen <tss@iki.fi>
parents: 1418
diff changeset
207 - rfc2557 support for BODYSTRUCTURE, as specified by RFC3501
4449
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
208 - LMTP server
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
209 - ability to build plugins statically into the binaries
3087
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
210 - ~/.dovecotrc to override system wide settings. namespace settings should
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
211 override all the previous namespace settings instead of adding new.
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
212 - option to disable SORT, SEARCH and other memory/cpu-intensive features.
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
213 defaults and per-user by dovecot-auth.
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
214 - dotlock overriding is racy, but it's pretty difficult to fix it. Also
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
215 overriding someone else's dotlock in shared folder isn't possible. These
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
216 could be fixed by having separate lock process running as root, which
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
217 would chown() the file for another uid and then unlink() it as that user.
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
218 One problem with that is that if malicious user sets setuid+execute bits
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
219 on for the file, he could run the file and get changed to the new uid.
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
220 That hopefully shouldn't matter much since the new uid should be user
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
221 with minimum possible privileges. Anyway, optional..
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
222 - 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
223
fd1fc4cf11b7 updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents: 340
diff changeset
224 capabilities:
4449
a2d8624f8f2e Updated
Timo Sirainen <tss@iki.fi>
parents: 4373
diff changeset
225 - 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
226 - possibility to disable them from config file
3087
769e2e2bf1d5 sorted things
Timo Sirainen <tss@iki.fi>
parents: 3048
diff changeset
227 - THREAD=ORDEREDSUBJECT - although pretty useless I'd think.
376
fd1fc4cf11b7 updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents: 340
diff changeset
228 - acl (rfc2086, draft-ietf-imapext-acl), namespace (rfc2342)
fd1fc4cf11b7 updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents: 340
diff changeset
229 - probably do it like cyrus. "user.<username>" to access other
fd1fc4cf11b7 updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents: 340
diff changeset
230 users, with "" defaulting to "user.<myself>". these should be
fd1fc4cf11b7 updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents: 340
diff changeset
231 configurable however.
fd1fc4cf11b7 updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents: 340
diff changeset
232 - shared namespaces? maybe configurable in config file
fd1fc4cf11b7 updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents: 340
diff changeset
233 - easiest way to do ACL would be to use unix modes, but is that
fd1fc4cf11b7 updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents: 340
diff changeset
234 useful at all? Well, ACL2 has a bit better support for that, so
fd1fc4cf11b7 updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents: 340
diff changeset
235 maybe we could support it.
fd1fc4cf11b7 updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents: 340
diff changeset
236 - otherwise gets a bit trickly, we could keep all mail in "imapmail"
fd1fc4cf11b7 updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents: 340
diff changeset
237 group and 0600/0700 mode by default, but when mail is shared to others,
fd1fc4cf11b7 updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents: 340
diff changeset
238 the group read/write access bits would be set. or alternatively we
fd1fc4cf11b7 updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents: 340
diff changeset
239 could launch another imap process to handle it, which we should support
fd1fc4cf11b7 updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents: 340
diff changeset
240 anyway. ACLs could be stored into ".acl" ascii file in each folder.
480
24a649e9c6d1 updated
Timo Sirainen <tss@iki.fi>
parents: 474
diff changeset
241 - support for private and shared flags, configurable by mailbox admin.
24a649e9c6d1 updated
Timo Sirainen <tss@iki.fi>
parents: 474
diff changeset
242 this isn't in any draft yet, but ACL2 author was going to create one.
24a649e9c6d1 updated
Timo Sirainen <tss@iki.fi>
parents: 474
diff changeset
243 [SHAREDFLAGS (...)] would specify which ones are shared, don't know yet
24a649e9c6d1 updated
Timo Sirainen <tss@iki.fi>
parents: 474
diff changeset
244 how they would be configured.
376
fd1fc4cf11b7 updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents: 340
diff changeset
245 - id (rfc2971)
fd1fc4cf11b7 updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents: 340
diff changeset
246 - 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
247 - separate pre/post-login settings
fd1fc4cf11b7 updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents: 340
diff changeset
248 - 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
249 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
250 "Disconnected", etc.
fd1fc4cf11b7 updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents: 340
diff changeset
251 - 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
252 especially before logging
fd1fc4cf11b7 updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents: 340
diff changeset
253 - mailbox-referrals (rfc2193)
fd1fc4cf11b7 updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents: 340
diff changeset
254 - 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
255 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
256 requiring another UID to run.
fd1fc4cf11b7 updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents: 340
diff changeset
257 - 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
258 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
259 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
260 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
261 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
262 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
263 referrals.
fd1fc4cf11b7 updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents: 340
diff changeset
264 - 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
265 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
266 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
267 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
268 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
269 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
270 ask the password again.
fd1fc4cf11b7 updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents: 340
diff changeset
271 - problems, problems, .. maybe not worth the trouble.
fd1fc4cf11b7 updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents: 340
diff changeset
272 - drafts:
fd1fc4cf11b7 updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents: 340
diff changeset
273 - http://www.imc.org/ids.html
fd1fc4cf11b7 updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents: 340
diff changeset
274 - annotate (draft-ietf-imapext-annotate)
fd1fc4cf11b7 updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents: 340
diff changeset
275 - 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
276 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
277 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
278 berkeley db to store the .data file and these annotations.
1178
933c8f9c241f updated
Timo Sirainen <tss@iki.fi>
parents: 975
diff changeset
279 - this is separate problem from index files. indexes are treated as
933c8f9c241f updated
Timo Sirainen <tss@iki.fi>
parents: 975
diff changeset
280 temporary files, annotations are permanent data. we'd have to
933c8f9c241f updated
Timo Sirainen <tss@iki.fi>
parents: 975
diff changeset
281 support non-db way to do this too, which would probably be just a
933c8f9c241f updated
Timo Sirainen <tss@iki.fi>
parents: 975
diff changeset
282 simple (slow) text file.
376
fd1fc4cf11b7 updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents: 340
diff changeset
283 - annotatemore (draft-daboo-imap-annotatemore)
fd1fc4cf11b7 updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents: 340
diff changeset
284 - 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
285 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
286 db as well.
fd1fc4cf11b7 updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents: 340
diff changeset
287 - binary (draft-nerenberg-imap-binary)
fd1fc4cf11b7 updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents: 340
diff changeset
288 - perhaps not too useful. I'd like to make Dovecot fully
fd1fc4cf11b7 updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents: 340
diff changeset
289 binary-safe though.
fd1fc4cf11b7 updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents: 340
diff changeset
290 - view (draft-ietf-imapext-view)
fd1fc4cf11b7 updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents: 340
diff changeset
291 - slow, complex, luckily draft expired almost two years ago.
fd1fc4cf11b7 updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents: 340
diff changeset
292 i hope i don't have to implement this :)
fd1fc4cf11b7 updated with lots of new capability comments
Timo Sirainen <tss@iki.fi>
parents: 340
diff changeset
293 - can be done client-side just fine (evolution's virtual folders)