debian/TODO.Debian

[Sven Mueller]

Henrique de Moraes Holschuh wrote on 30/01/2006 02:37:
> On Sat, 28 Jan 2006, Benjamin Seidenberg wrote:
> 
>>EXTRA HIGH
>>o --name for start-stop-daemon
> 
> start-stop-daemon should never be called using -exec (for stop), it must use
> -name instead.

Fixed

>>o VERIFY SASL LEAKING CRAP TO prot_stream
>>   [info-cyrus  Re: Log messages going into imap/pop stream]
> 
> 2.1.  I believe fixed in 2.2.  No matter what you do in cyrus, you
> ABSOLUTELY MUST never hook to fds 0,1,2.  If you do anything stupid (like
> close()'ing one of them, you have a window of opportunity for syslog() or
> something else to use one of those fds.  Cyrus will *override* that fd by
> force later, and you end up with syslog sending stuff into the IMAP control
> channel for example(!).  Immediate client confusion issues, client and
> server usually lose sync, and abort the connection.
> 
> So basically, nothing can close fds 0,1 and 2 and leave them closed, they
> must be permanently open (maybe to /dev/null).

This must be fixed, along others also in:
debian/patches/08-clean_socket_closes.dpatch
The bug has been reported in
http://asg.web.cmu.edu/archive/message.php?mailbox=archive.cyrus-devel&msg=760
Along with a fix (which we would need to backport/adjust to cyrus-2.2
with our fixes, which at least in part add additional location where the
fix needs to be applied - see above dpatch file).

>>o Revise rehash, it is busted...
> 
> rehash in 2.2 is probably fixed. It is a matter of testing if it can convert
> 2.1 spools to 2.2 and 2.2-with-virtual-domain spool layouts, for all
> combinations incuding hashed spools and FULLHASHed spools.

2.1 spools will always be converted to 2.2-with-virtual-domain spool
layouts (just with no virtual domain used). 2.2 spool and 2.2 spool with
virtual domains look exactly the same (except that the former doesn't
have any domain/ subdir). And rehash doesn't support moving 2.1 ('user')
 accounts to 2.2 virtual domain ('user at do.main') accounts.

>>o Write nice shell wrapper for chk_cyrus and suggest a event to run it
> 
> Self-explanatory.

Not to me. At least not to 100%. Do you want a cronjob which runs
chk_cyrus and if that reports any errors, notify the admin? If so: how
should it send notifications? Mail might not work as expected in that case.

>>o Clean up script, that finds all crap over one day old in .stage
>> and delete it (after logging a warning). Add it as event.
> 
> See above about .stage files.

Any idea how to test that script once it is written? I never had any
cruft left in .stage directories. At least I never was able to see any
(not even a .stage directory).

>>o https://bugzilla.andrew.cmu.edu/show_bug.cgi?id=1735
> 
> Probably still broken on 2.2, there are patches floating on the info-cyrus
> ML, and cyrus-devel.

http://asg.web.cmu.edu/archive/message.php?mailbox=archive.cyrus-devel&msg=829
reports it to be fixed in 2.2.10

>>o Fulldir-hash detection using the same mech as db-backend
> 
> We should support fullhash mode and if cyrus cannot detect and adapt at
> runtime (must check 2.2), we need to use the old "backend mismatch" stuff to
> do it.

What is fullhash mode BTW? I saw it referenced a few times, but never
found any explanation what it does exactly (but I must admit I didn't
search much for it since I didn't see any swith in imapd.conf to select
hashmode apart from switching it on or off with hashimapspool).

>>o i18n (GNU gettext, with kerberos guard for logs, messages;
>> sanity-enforced txt files for return messages)
> 
> definately keep.

Uhm, I don't mind i18n, but what exactly do you want to get localized?
Only log messages and startup messages (i.e. server-local output)? Also
server->client messages? What to do if admin wants one of those two to
be english and the other localized?

regards,
Sven
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 186 bytes
Desc: OpenPGP digital signature
Url : http://lists.alioth.debian.org/pipermail/pkg-cyrus-imapd-debian-devel/attachments/20060130/f3616cfd/signature.pgp