Re: skiplist db corruption

2006-11-13 Thread Ken Murchison
Robert Mueller wrote: Just reviving an old issue to see if anyone has any more information. http://asg.web.cmu.edu/archive/message.php?mailbox=archive.cyrus-develmsg=1072 We still see this every now and then, most often with seen state databases. It's most often after an unclean shutdown,

Re: Sieve \\Seen flag actions not replicated

2007-01-02 Thread Ken Murchison
John Capo wrote: Quoting Ken Murchison ([EMAIL PROTECTED]): John Capo wrote: Tiring of running fixup scripts to keep seen state consistent, I decided to find the problem. In imap/lmtpd.c: sync_log_append(mailboxname); + +if (nflags user !strncmp(user

Re: Cyrus IMAP Readonly partitions

2007-03-29 Thread Ken Murchison
Michael D. Sofka wrote: I am working on a way to recover deleted (and purged) messages in Cyrus IMAP. This is not as silly as it may at first sound. On Exchange there is a setting whereby deleted items are not deleted from the server until they are backed up. Before this time, they can be

RE: CORRECT PATCH Re: sync_client bails when encountering a deleted message

2007-05-15 Thread Ken Murchison
-Original Message- From: John Capo [EMAIL PROTECTED] To: Ken Murchison [EMAIL PROTECTED] Cc: cyrus-devel@lists.andrew.cmu.edu Sent: 5/15/07 3:34 PM Subject: Re: CORRECT PATCH Re: sync_client bails when encountering a deleted message Quoting Ken Murchison ([EMAIL PROTECTED]): John Capo wrote

Re: ...and a question (SHA1 UUIDs)

2007-08-10 Thread Ken Murchison
David Carter wrote: On Mon, 6 Aug 2007, Ken Murchison wrote: My gut is to use the 'replace' patch. I'm not comfortable with the 21 byte scenario. I agree that 21 bytes is likely to be a pain to maintain long term. If we need a UUID schema version, we can always use a new MINOR_VERSION

Cyrus 2.3.9 RC1

2007-08-10 Thread Ken Murchison
I just put together a release candidate for Cyrus 2.3.9. I'd appreciate any independent testing before I release this to the masses. If there are any outstanding issues that you believe still need to be addressed in 2.3.9, please let me know.

Cyrus 2.3.9 RC2

2007-08-15 Thread Ken Murchison
I just put together a second release candidate for Cyrus 2.3.9. I'd appreciate any independent testing before I release this to the masses at the end of the week. http://www.contrib.andrew.cmu.edu/~murch/cyrus-imapd-2.3.9rc2.tar.gz -- Kenneth Murchison Systems Programmer Project Cyrus

Re: A couple of goodies

2007-08-22 Thread Ken Murchison
Bron Gondwana wrote: Sorry about not getting back to reply to the original of this. I still want to spend some polishing time on these patches. I'm getting ready to start applying delayed delete and MD5/SHA1 UUID code to CVS. What needs polishing? A couple of thoughts before I commit

Re: deleted DELETE

2007-08-29 Thread Ken Murchison
David Carter wrote: On Tue, 28 Aug 2007, Ken Murchison wrote: I just committed Bron's modified version of David's delayed delete patch, with a few small mods of my own. The only externally visible change is I renamed deleteprefix to deletedprefix. I also removed/modified some of the option

Re: deleted DELETE

2007-08-29 Thread Ken Murchison
David Carter wrote: On Tue, 28 Aug 2007, Ken Murchison wrote: I just committed Bron's modified version of David's delayed delete patch, with a few small mods of my own. The only externally visible change is I renamed deleteprefix to deletedprefix. I also removed/modified some of the option

Re: Need advice: mailbox-based \Seen flag

2007-08-30 Thread Ken Murchison
Ken Murchison wrote: Boris Lytochkin wrote: Hello! I need an advice in implementing per-mailbox \Seen flag (or 'shared' \Seen flag in per-user basis). For now I want to implement it this way: 1) add a new mailbox attribute, say 'sharedseen' 2) switch path for .seen file in imap/seen_

Re: Need advice: mailbox-based \Seen flag

2007-08-30 Thread Ken Murchison
Rob Mueller wrote: I actually went digging into seen state and cyrus.index a little today for another reason, and I realized that we wouldn't have to expand/upgrade the index file to accomodate a shared seen state. The existing field for system flags is just a bitmask, so we could easily

Re: Need advice: mailbox-based \Seen flag

2007-08-30 Thread Ken Murchison
Boris Lytochkin wrote: My quick thoughts would be the following: - Add a 'sharedseen' mailbox attribute as you suggest (as a mailbox annotation) - When a mailbox is opened that has this annotation enabled, we open the seen state database for the special 'anyone' user instead of the

Re: *IMPORTANT* - bugfix sync_append_commit index breakage

2007-08-31 Thread Ken Murchison
I think your patch makes sense, but I'm not sure when The cyrus index file format has this clever ignore junk at the end until the exists count changes trick means. I know we leave junk in the cache file as a result of delayed expunge which gets cleaned up later, but I'm pretty sure the the

Re: Need advice: mailbox-based \Seen flag

2007-09-05 Thread Ken Murchison
I just committed a patch with your Perl changes and (hopefully) support for replication. Boris Lytochkin wrote: Ken, Thanks for patch! Here it is full patch (with cyradm patch additions) that works OK on my imapd 2.3.9 setup. I can not check replication process for now. Here it is some

Re: pcreposix, detect, use by default

2007-09-11 Thread Ken Murchison
Bron Gondwana wrote: On Tue, Sep 11, 2007 at 10:02:50AM +1000, Bron Gondwana wrote: On Mon, Sep 10, 2007 at 02:38:07PM -0400, Ken Murchison wrote: A lot of the patches (past and present) have been implemented. Some of the others will be implemented in time. Others may not. Some of them

Re: replication is unable to fix uidvalidity difference

2007-09-12 Thread Ken Murchison
David Carter wrote: On Mon, 3 Sep 2007, Bron Gondwana wrote: Reading the sync_client protocol, I notice that do_mailbox_single doesn't actually send the uidvalidity as part of the local mailbox status from the server, so it can't even tell there's a difference. [...], something to keep in

Re: replication is unable to fix uidvalidity difference

2007-09-12 Thread Ken Murchison
David Carter wrote: On Wed, 12 Sep 2007, Ken Murchison wrote: Is there any reason why we want this to be a separate command, rather than just adding UIDVALIDITY to the MAILBOXES command? MAILBOXES dumps the current state of mailboxes on the sever (including the UIDvalidity with this patch

Re: [PATCH] cyrus-imapd: use getgrouplist instead of set/getgrent on systems supporting it

2007-09-17 Thread Ken Murchison
Tomas Janousek wrote: Hi Simon, On Tue, Jun 12, 2007 at 03:52:27PM +0200, Simon Matter wrote: I have slightly modified the patch to make older GCC happy with it. Does it still look okay? Yeah, it's fine, thanks. I applied Simon's version to CVS, but then I realized that it causes

Re: [PATCH] cyrus-imapd: use getgrouplist instead of set/getgrent on systems supporting it

2007-09-17 Thread Ken Murchison
Tomas Janousek wrote: Hello, On Mon, Sep 17, 2007 at 09:49:38AM -0400, Ken Murchison wrote: I applied Simon's version to CVS, but then I realized that it causes myfreestate() to crash on my Linux box, because the newstate-groups array doesn't get entirely populated. The following patch fixes

Re: intercept delete event

2007-09-20 Thread Ken Murchison
Paolo Spadafora wrote: Hello, is there a way to intercept folder events like: delete message, append, etc and then call a function or procedure, for instance an http request with some parameters (message-id, folder name, action) ? Preferably without changing cyrus source code, unless is the

Re: Large Mailbox Append Fix

2007-09-24 Thread Ken Murchison
Bron Gondwana wrote: On Mon, 24 Sep 2007 13:53:28 -0400, Ken Murchison [EMAIL PROTECTED] said: Bron Gondwana wrote: On Sat, 22 Sep 2007 10:44:55 +0100 (BST), David Carter [EMAIL PROTECTED] said: On Sat, 22 Sep 2007, Bron Gondwana wrote: We still need to sync the final last_uid, which may

Re: Large Mailbox Append Fix

2007-09-24 Thread Ken Murchison
Bron Gondwana wrote: On Mon, 24 Sep 2007 18:45:28 -0400, Ken Murchison [EMAIL PROTECTED] said: Bron Gondwana wrote: On Mon, 24 Sep 2007 13:53:28 -0400, Ken Murchison [EMAIL PROTECTED] said: Bron Gondwana wrote: On Sat, 22 Sep 2007 10:44:55 +0100 (BST), David Carter [EMAIL PROTECTED] said

Re: Large Mailbox Append Fix

2007-09-25 Thread Ken Murchison
Bron Gondwana wrote: On Mon, 24 Sep 2007 23:39:52 -0400, Ken Murchison [EMAIL PROTECTED] said: Yes, my for() loop test was incorrect, and SHOULD BE: (index_list-count max_count) (msgno = mailbox-exists) So, for mailbox-exists=1000, and max_count=1000, when we exit

Re: Sieve not applied to mailbox when using imapsync

2007-10-03 Thread Ken Murchison
[EMAIL PROTECTED] wrote: I have a sieve script that is doing what I want it to do when I email the account. I'd like to have this script run when moving mail from one mailbox into the account with the script, but it is not doing that when I use imapsync program. So.what do I need to do

[Fwd: problem with user canonicalization and pop3d]

2007-10-03 Thread Ken Murchison
Does anyone have any thoughts on this (the proxy piece isn't needed, since you can't proxy with USER/PASS)? Would it break any installations? It works in my testing. For sites using their own canonicalization plugin, this is probably necessary. -- Kenneth Murchison Systems Programmer

Re: Sieve not applied to mailbox when using imapsync

2007-10-03 Thread Ken Murchison
Marc Rassbach wrote: On Wed, 3 Oct 2007, Ken Murchison wrote: Sieve scripts only get executed when mail is delivered via LMTP, not when APPENDed via IMAP. There is a specification for an IMAP-Sieve extension Ok. Thank you for the clarifying response. Because I'm trying to put together

Re: 2.3.8 traditional murder POP3 proxy

2007-10-04 Thread Ken Murchison
Shawn Nock wrote: -BEGIN PGP SIGNED MESSAGE- Hash: SHA1 We rolled out 2.3.8 a few months ago (configuration specifics are at the end of the message)... and for the most part things have gone very well (After Bron solved our Reiserfs deadlock problem... he finds all the good bugs!) A

Re: Ready for 2.3.10?

2007-10-05 Thread Ken Murchison
Wesley Craig wrote: This is a show stopper, i.e., it results in data loss: https://bugzilla.andrew.cmu.edu/show_bug.cgi?id=2919 The two fixes are pretty trivial. I have other comments on running ctl_mboxlist -m from cyrus.conf START, but those are probably not as critical as these

Re: Ready for 2.3.10?

2007-10-05 Thread Ken Murchison
David Carter wrote: On Fri, 5 Oct 2007, Ken Murchison wrote: I spent a lot of time cleaning things up so that the number of warnings from 'gcc -W -Wall' are minimal. The one that has me baffled is this: warning: type-punning to incomplete type might break strict-aliasing rules I only

Re: Ready for 2.3.10?

2007-10-10 Thread Ken Murchison
Carson Gaspar wrote: You might get more testers if: http://cyrusimap.web.cmu.edu/anoncvs.html Linked to from: http://cyrusimap.web.cmu.edu/downloads.html Returned something more useful than a 404 error... Kind of hard to test when you can't get the source. Fixed. -- Kenneth Murchison

Cyrus 2.3.10 RC1

2007-10-10 Thread Ken Murchison
I just put together a release candidate for Cyrus 2.3.10. I'd appreciate any independent testing before I release this to the masses. If there are any outstanding issues that you believe still need to be addressed in 2.3.10, please let me know.

Re: Cyrus 2.3.10 RC2

2007-10-11 Thread Ken Murchison
Rudy Gevaert wrote: Ken Murchison wrote: I just put together a second release candidate for Cyrus 2.3.10 which hopefully fixes a couple of build issues with RC1. I'd appreciate any independent testing before I release this to the masses. If there are any outstanding issues that you believe

Re: Ready for 2.3.10?

2007-10-11 Thread Ken Murchison
Carson Gaspar wrote: Do I need to worry about these? autoconf-2.61, CVS HEAD as of 5 minutes ago. configure: creating ./config.status config.status: creating man/Makefile config.status: WARNING: man/Makefile.in seems to ignore the --datarootdir setting config.status: creating

Re: Cyrus 2.3.10 RC2

2007-10-11 Thread Ken Murchison
David Carter wrote: On Thu, 11 Oct 2007, Rudy Gevaert wrote: I'm wondering if uuidmode: md5 is gone, how can I get I md5 hash of a message, so I can compare a message on the master and the replica? Or am I missing something? make_md5 still works: it doesn't use the UUIDs/GUIDs. There is

Re: cyrus 2.3.9 + cyr_expire + Thunderbird 2 - Crash

2007-10-11 Thread Ken Murchison
Florian G. Pflug wrote: Hi When I setup annotiation on mailboxes (like Junk, Trash, ..) to have cyr_expire remove messages older than x days, Thunderbird stops responding when I try to open the folder after cyr_expire has run. I just wanted to check if anybody else has seen this, and might

Re: delayed delete bug?

2007-10-16 Thread Ken Murchison
Bron Gondwana wrote: On Mon, Oct 15, 2007 at 03:25:28PM +0200, Rudy Gevaert wrote: Hi, I'm busy looking at delayed delete. I'm using unix hierarchy seperator. I deleted a mailbox and see it like this: kavula.ugent.be lm DELETED/user/rudy.gevaert/Foo/[EMAIL PROTECTED] (\HasNoChildren)

Re: ptloader crash

2007-10-17 Thread Ken Murchison
Try this patch: https://bugzilla.andrew.cmu.edu/cgi-bin/cvsweb.cgi/src/cyrus/ptclient/ldap.c.diff?r1=1.10;r2=1.11;sortby=date;f=h;f=u Dmitriy Kirhlarov wrote: Hi, list I have succefully running cyrus-imapd-2.3.8_1 with ldap-based auth on FreeBSD 6.2-STABLE amd64. Currently I installing

Cyrus 2.3.10 RC3

2007-10-18 Thread Ken Murchison
I just put together a third release candidate for Cyrus 2.3.10 which fixes a couple of bugs, and hopefully resolves outstanding build issues with com_err and xversion. I'd appreciate any independent testing before I release this to the masses. If there are any outstanding issues that you

Re: Two more patches

2007-10-24 Thread Ken Murchison
Bron Gondwana wrote: On Wed, 24 Oct 2007 07:55:55 -0400, Ken Murchison [EMAIL PROTECTED] said: It look to me like we bail out if we are missing a message listed in cyrus.expunge. Is this correct? Do we want to abort a reconstruct in this case? No, just strip the record from the .expunge

Cyrus IMAPd 2.3.10 Released

2007-10-24 Thread Ken Murchison
I am pleased to announce the release of Cyrus IMAPd 2.3.10. This release should be considered production quality. This release is currently deployed on the production Murder at CMU, as well as several other large sites. This release adds support for delayed delete and shared \Seen state, as

Re: Cyrus IMAPd 2.3.10 Released

2007-10-25 Thread Ken Murchison
What does imapd.conf look like? Does the output of 'ctl_mboxlist -d' look reasonable? Does 'mbexamine user.igb' look reasonable? Ian G Batten wrote: I've just compiled 2.3.10 on batten.eu.org (my private x86 servers) and although it looks OK on the Solaris 10 system, it's in deep trouble

Re: Cyrus IMAPd 2.3.10 Released

2007-10-25 Thread Ken Murchison
Ian G Batten wrote: On 25 Oct 07, at 1248, Ken Murchison wrote: What does imapd.conf look like? Does the output of 'ctl_mboxlist -d' look reasonable? Does 'mbexamine user.igb' look reasonable? OK, there's a steady stream of imapd processes being forked and then dying on SIGSEGV. I've

statuscache

2008-01-17 Thread Ken Murchison
Looking (again) at integrating this. Two observations, the first somewhat minor, and the second somewhat major: - statuscache v.2 doesn't have support for HIGHESTMODSEQ. This is trivial to add to a v.3. - statuscache v.2 stores statuscache_data as a binary blob, which is platform

Re: statuscache

2008-01-17 Thread Ken Murchison
Bron Gondwana wrote: On Thu, Jan 17, 2008 at 02:47:27PM -0500, Ken Murchison wrote: Looking (again) at integrating this. Two observations, the first somewhat minor, and the second somewhat major: - statuscache v.2 doesn't have support for HIGHESTMODSEQ. This is trivial to add to a v.3

Re: statuscache

2008-01-17 Thread Ken Murchison
Rob Mueller wrote: Basically our startup scripts completely delete the statuscache.db file on startup, so migrating isn't an issue for us at all. Why do you delete the cache? I doesn't appear from the code that the cache entries would be invalid if the mailbox and seen state aren't touched.

Re: statuscache

2008-01-17 Thread Ken Murchison
Here's my modified patch against CVS. I suppose some code could be added to statuscache_lookup() to read both v.2 and v.3. There isn't any reason to invalidate all v.2 caches just because they don't have highestmodseq -- Kenneth Murchison Systems Programmer Project Cyrus

Re: Small memory leak in the status cache code

2008-01-30 Thread Ken Murchison
David Carter wrote: There is a small memory leak in the status cache code. Or at least I don't believe that DB-store() automatically free()s its input. Probably faster to just have a char data[250] on the stack. Good call. I did this in CVS. Thanks. -- Kenneth Murchison Systems

Re: Patch to avoid mailboxesdb corruption on concurrent renames

2008-04-04 Thread Ken Murchison
Bron Gondwana wrote: On Fri, Apr 04, 2008 at 07:10:27AM -0400, Ken Murchison wrote: Bron Gondwana wrote: Bron ( P.S. isn't it about time for a 2.3.12? I'm getting sick of posting skiplist patches to people running the lastest and having issues! ) Yes, it probably is. Perhaps

Re: Cyrus 2.3.12 RC1

2008-04-09 Thread Ken Murchison
Bron Gondwana wrote: On Tue, Apr 08, 2008 at 10:20:57AM -0400, Ken Murchison wrote: I just put together a release candidate for Cyrus 2.3.12. I'd appreciate any independent testing before I release this to the masses. http://www.contrib.andrew.cmu.edu/~murch/cyrus-imapd-2.3.12rc1.tar.gz

Re: compiling against specific libdb

2008-04-15 Thread Ken Murchison
Rudy Gevaert wrote: Hello, I'm compiling cyrus on Debian etch. libdb4.3 and libdb4.4, libdb4.3-dev packages are installed. If I understand the configure script correctly it tries to find the highest available libdb installed on the system. In this case that is libdb4.4. However I want to

Cyrus IMAPd 2.3.12 Released

2008-04-21 Thread Ken Murchison
I am pleased to announce the release of Cyrus IMAPd 2.3.12. This release should be considered production quality. Noteworthy changes: * Added statuscache.db to cache IMAP STATUS data which significantly reduces the amount of I/O necessary when neither the mailbox nor \Seen state has changed

Re: Cyrus IMAPd 2.3.12 Released

2008-04-24 Thread Ken Murchison
Simon Matter wrote: I am pleased to announce the release of Cyrus IMAPd 2.3.12. This release should be considered production quality. Noteworthy changes: * Added statuscache.db to cache IMAP STATUS data which significantly reduces the amount of I/O necessary when neither the mailbox nor

Re: Cyrus IMAPd 2.3.12 Released

2008-04-24 Thread Ken Murchison
Simon Matter wrote: Simon Matter wrote: I am pleased to announce the release of Cyrus IMAPd 2.3.12. This release should be considered production quality. Noteworthy changes: * Added statuscache.db to cache IMAP STATUS data which significantly reduces the amount of I/O necessary when neither

Re: Cyrus IMAPd 2.3.12p2 Released

2008-04-25 Thread Ken Murchison
Hajimu UMEMOTO wrote: Hi, On Fri, 25 Apr 2008 13:48:54 -0400 Ken Murchison [EMAIL PROTECTED] said: murch Simon Matter wrote: I am pleased to announce the release of Cyrus IMAPd 2.3.12p2. This release should be considered production quality. I was careless in applying the memory corruption

Re: Cyrus IMAPd 2.3.12p2 Released

2008-04-26 Thread Ken Murchison
Hajimu UMEMOTO wrote: Hi, On Fri, 25 Apr 2008 14:39:20 -0400 Ken Murchison [EMAIL PROTECTED] said: murch Hajimu UMEMOTO wrote: Hi, On Fri, 25 Apr 2008 13:48:54 -0400 Ken Murchison [EMAIL PROTECTED] said: murch Simon Matter wrote: I am pleased to announce the release of Cyrus IMAPd

Re: sieve addflag/setflag for \seen flag doesn't work with keep/implicit keep

2008-05-14 Thread Ken Murchison
Sorry for the delay in looking into this. I looked at the code, and finally realized why it was written in the way that it was: - For fileinto, we use the credentials of the script owner when delivering to the mailbox and optionally setting IMAP flags. We know that the script owner has

Re: [Cyrus-CVS] src/cyrus/imap by murch

2008-09-23 Thread Ken Murchison
Matt Selsky wrote: Quoting [EMAIL PROTECTED]: Update of /afs/andrew/system/cvs/src/cyrus/imap In directory unix36.andrew.cmu.edu:/var/tmp/cvs-serv5457 Modified Files: Makefile.in cyr_virusscan.c Log Message: aded code to append a notification message to user's INBOX when we remove

[Fwd: Re: Cyrus 2.3.13 RC1]

2008-09-25 Thread Ken Murchison
Anyone want to comment on these requests? I had planned on getting to these patches, but then the push to get out 2.3.13 began. -- Kenneth Murchison Systems Programmer Project Cyrus Developer/Maintainer Carnegie Mellon University ---BeginMessage--- Quoting Ken Murchison [EMAIL PROTECTED

Re: Things for 2.3.13

2008-09-25 Thread Ken Murchison
Wesley Craig wrote: On 23 Sep 2008, at 22:43, Bron Gondwana wrote: That's my entire wishlist for 2.3.13 (plus the buffer size patch you already included) I think there should be an RC2. I don't see: https://bugzilla.andrew.cmu.edu/show_bug.cgi?id=3075 included in RC1. I gathered that

Re: More special chars in mailbox names

2008-10-01 Thread Ken Murchison
Ken Murchison wrote: While looking over bug #3002, https://bugzilla.andrew.cmu.edu/show_bug.cgi?id=3002 I started wondering what other special characters we might want to allow. The original authors were more restrictive with GOODCHARS than the RFC. AFAICT the following characters

Re: Cyrus 2.3.13 RC1

2008-10-02 Thread Ken Murchison
This seems like something that should go into 2.3.13. Wes, Bron, and comments? John Capo wrote: Quoting Ken Murchison ([EMAIL PROTECTED]): I just put together a release candidate for Cyrus 2.3.13. I'd appreciate any independent testing before I release this to the masses. http

Cyrus 2.3.13 RC2

2008-10-06 Thread Ken Murchison
I just put together a second release candidate for Cyrus 2.3.13. I'd appreciate any independent testing before I release this to the masses. http://www.contrib.andrew.cmu.edu/~murch/cyrus-imapd-2.3.13rc2.tar.gz http://www.contrib.andrew.cmu.edu/~murch/cyrus-imapd-2.3.13rc2.tar.gz.sig

Re: Cyrus 2.3.13 RC2

2008-10-07 Thread Ken Murchison
Kenneth Marshall wrote: On Tue, Oct 07, 2008 at 12:18:01PM +0200, Simon Matter wrote: I just put together a second release candidate for Cyrus 2.3.13. I'd appreciate any independent testing before I release this to the masses.

Re: THREAD curiosity

2008-10-09 Thread Ken Murchison
David Carter wrote: On Thu, 9 Oct 2008, Ken Murchison wrote: How does this look? That is what I had in mind. In particular we are no longer running stristr() on a mmap()ed file and praying we find a '\0' before we run out of file. While we got away with this because of the following

Re: THREAD curiosity

2008-10-09 Thread Ken Murchison
David Carter wrote: On Wed, 8 Oct 2008, Ken Murchison wrote: Changing it to the following fixes it as long as References isn't the first header field: if ((refstr = stristr(headers, \r\nreferences:))) { If we're worried about it being the first header field, there is an easy test

Re: Replication broken in CVS

2009-02-04 Thread Ken Murchison
Should be fixed in CVS. Bron Gondwana wrote: Hi Ken, Replication is broken in CVS. Using the superpowers of git I have tracked it down to this commit: Author: murch murch Date: Wed Jan 14 15:50:47 2009 + Fixed incompatibility between older ManageSieve clients and newer

Cyrus 2.3.13 RC1

2009-03-13 Thread Ken Murchison
I just put together a release candidate for Cyrus 2.3.14. I'd appreciate any independent testing before I release this to the masses. http://www.contrib.andrew.cmu.edu/~murch/cyrus-imapd-2.3.14rc1.tar.gz Check doc/changes.html for a complete list of changes. If there are any outstanding

Re: Cyrus IMAP4 to Cyrus IMAP accidental change of greeting?

2009-04-01 Thread Ken Murchison
I don't recall if this change was intentional or accidental, but I wouldn't recommend trying to use any free-form banner information to determine a server's capability. In fact, some Cyrus servers might not even include Cyrus in the banner if folks turn the serverinfo option off. The proper

Re: Cyrus IMAP4 to Cyrus IMAP accidental change of greeting?

2009-04-02 Thread Ken Murchison
We could add a X-SHAREDSEEN capability, letting clients know that the server has the functionality, but its still a per-mailbox option. Bernhard Reiter wrote: Am Mittwoch, 1. April 2009 13:46:16 schrieb Ken Murchison: I don't recall if this change was intentional or accidental, Okay, I

Re: Small type bug in lmtpengine.c

2009-04-22 Thread Ken Murchison
Applied to CVS. David Carter wrote: The quota parameter to verify_user() is quota_t, not long. Solaris 10 with bundled GCC 3.4.3 wasn't doing type promotion correctly. (long)(-1) becomes (long long int)(4294967295), which is = 0. Consequently IGNOREQUOTA had no effect. -- Kenneth

Re: CONDSTORE by default?

2009-05-04 Thread Ken Murchison
There is a higher bookkeeping cost when only \Seen state changes because rather than just updating seen.db, we also have to tweak cyrus.index. The cost may not be enough to worry about, but since \Seen state is most likely the most frequently used flag (and most likely is changed separately

Re: Incorrect size calculations on bogus messages

2009-06-25 Thread Ken Murchison
I wonder if we should just reject these messages in lmtpd. Bron Gondwana wrote: Here's a funny one. I've recreated it as a simple testcase which I'll paste below. Basically, a message with invalid mime structure causes cyrus to put the wrong size information in its headers. Seems some

Re: FastMail Cyrus Patches for upstream

2009-08-25 Thread Ken Murchison
The CRC additions make sense to me. I will look at the rest of the patches when I get back from vacation next week. Bron Gondwana wrote: UP-FRONT notice. I'd particulary love feedback on the index format change. Here's the executive overview: index header: replace SPARE4 with HEADER_CRC -

Re: ANNOTATEMORE = METADATA and rfc 5464

2009-11-17 Thread Ken Murchison
What is your new format proposal? Bron Gondwana wrote: I'm in the process of implementing rfc 5464, which is what the ANNOTATEMORE drafts turned into. Unfortunately, Cyrus' support is an early draft, before the paths to everything were changed and the commands were renamed. It would be great

Re: ANNOTATEMORE = METADATA and rfc 5464

2009-11-17 Thread Ken Murchison
Bron Gondwana wrote: On Tue, Nov 17, 2009 at 09:03:11AM -0500, Ken Murchison wrote: What is your new format proposal? I'll see :) Not sure yet - but mainly not sizeof(unsigned long)! If we make a wholesale change to the database, perhaps this might be something we put in the 2.4 branch

Re: ANNOTATEMORE = METADATA and rfc 5464

2009-11-17 Thread Ken Murchison
Bron Gondwana wrote: On Tue, Nov 17, 2009 at 04:17:51PM -0500, Ken Murchison wrote: Bron Gondwana wrote: On Tue, Nov 17, 2009 at 09:03:11AM -0500, Ken Murchison wrote: What is your new format proposal? I'll see :) Not sure yet - but mainly not sizeof(unsigned long)! If we make a wholesale

Cyrus IMAPd 2.3.16 Released

2009-12-21 Thread Ken Murchison
I am pleased to announce the release of Cyrus IMAPd 2.3.16. This release should be considered production quality. Major changes in the release are the following: - Added 'user_deny.db' to be able to selectively deny users access to Cyrus services. - Added 'popuseimapflags' option which

Re: Cyrus IMAPd 2.3.16 Released

2009-12-22 Thread Ken Murchison
Done. Hajimu UMEMOTO wrote: Hi, On Mon, 21 Dec 2009 09:01:53 -0500 Ken Murchison mu...@andrew.cmu.edu said: murch I am pleased to announce the release of Cyrus IMAPd 2.3.16. This murch release should be considered production quality. Major changes in the murch release are the following

Cyrus development update 12/29/09

2009-12-29 Thread Ken Murchison
I spent a couple of days reviving the stale 2.4 development branch by merging all of the 2.3 changes into it. This branch will now be where the most active development will be taking place. Any activity on the 2.3 branch will be mostly bugfixes. New features that are already in 2.4 are

QoS on Cyrus traffic

2010-01-08 Thread Ken Murchison
Can we get some opinions on the feature request below: https://bugzilla.andrew.cmu.edu/show_bug.cgi?id=3184 I'm pretty much convinced that such a feature is worth adding, the only question is where should it be configured: imapd.conf or cyrus.conf ? -- Kenneth Murchison Systems Programmer

Re: QoS on Cyrus traffic

2010-01-08 Thread Ken Murchison
Philip A. Prindeville wrote: On 01/08/2010 12:48 PM, Ken Murchison wrote: Philip A. Prindeville wrote: On 01/08/2010 12:06 PM, Ken Murchison wrote: Can we get some opinions on the feature request below: https://bugzilla.andrew.cmu.edu/show_bug.cgi?id=3184 I'm pretty much convinced

Re: What's happening in future branch - March 25

2010-03-25 Thread Ken Murchison
Sounds good! Bron Gondwana wrote: Gosh, it's getting awfully close to my self imposed deadline of April isn't it - though I think I'll be pulling an Ubuntu and saying I meant the end of April, honest. Sysadmin stuff got in the way for rather longer than I intended. Due to making changes all

Re: QRESYNC

2010-04-06 Thread Ken Murchison
Sorry for the late response. I also agree that (1) seems like the best option. Robert Mueller wrote: 1) Pro: * simple to calculate - only updated by cyr_expire simple seems good :) 1) Con: * Higher bandwidth use in the stale client case. If you're not syncing that often, then

Re: What's happening in future branch - April 19

2010-04-19 Thread Ken Murchison
When will all of this end up in CVS? I've been distracted with other things for a while, but I still want to complete what's left of RFC 5524 (BODYPARTSTRUCTURE). There are also a few relatively trivial additions/fixes in bugzilla that I was going to hit today or tomorrow. Bron Gondwana

Re: Seen databases

2010-05-04 Thread Ken Murchison
I've been thinking about this for a while and I keep coming back to the same answer. seen_local is legacy and I wouldn't expect to find this in the wild anymore. I don't think we should waste cycles doing anything with it. I don't recall why seen_bigdb was created by one of my predecesors,

Re: Cyrus Future status update

2010-07-06 Thread Ken Murchison
Bron, What is broken in nntpd? I don't know who is using it, but I will certainly fix it. Bron Gondwana wrote: So - I haven't pushed anything more to CVS for a bit! That's because I wanted to refactor some more list logic out of sync_support.c and into its own function where it can be

Re: MAILBOX_FORMAT_NETNEWS - is it actually needed?

2010-07-07 Thread Ken Murchison
I think its some legacy cruft. I have never used it in any code that I have written. It probably can be removed. Bron Gondwana wrote: I've just been searching through the sourcecode trying to find any evidence that MAILBOX_FORMAT_NETNEWS actually changes how anything works. It looks like

Re: RSS support in Cyrus

2011-09-19 Thread Ken Murchison
Jeroen van Meeuwen (Kolab Systems) wrote: On 19.09.2011 17:26, Dave McMurtrie wrote: Good day, Hi Dave, We've been tasked by our management with providing RSS support in Cyrus. Fortunately, the Cyrus httpd server that was recently developed makes this much less onerous of a task.

Re: RSS support in Cyrus

2011-09-21 Thread Ken Murchison
Alexey Melnikov wrote: Bron Gondwana wrote: On Mon, Sep 19, 2011 at 12:37:52PM -0400, Ken Murchison wrote: +1 for RSS (obviously) I think if it's going into production it should go onto the roadmap for 2.5 as well, unless we get 2.5 out before then. Otherwise the development effort

Cyrus NNTP/newspostuser

2011-10-05 Thread Ken Murchison
Is anybody using Cyrus NNTP and the newspostuser option in imapd.conf? I'm considering changing the behavior of newspostuser so that it constructs a To: header rather than a Reply-To: header but I don't want to break any existing installs. -- Kenneth Murchison Principal Systems Software

Re: Cyrus reviews

2012-01-24 Thread Ken Murchison
Greg Banks wrote: G'day, I've been told I should do reviews more openly. Ok, here goes. commit rename: ensure user owns both source and dest for Bug #3586 workaround Ok, but why? commit nntpd: use defaultdomain in conjunction with newspostuser to create Reply-To header addresses Looks

Re: Cyrus reviews

2012-01-24 Thread Ken Murchison
Ken Murchison wrote: Greg Banks wrote: This code +newdest = buf_release(buf); will leak the string, as newdest is never free()d (and indeed in some other branches of the logic, cannot be). A better solution would be const char *newdest = NULL; ... newdest = buf_cstring(buf

Re: Let's map tickets to milestones

2012-04-15 Thread Ken Murchison
The CalDAV code is almost entirely orthogonal to the base Cyrus code so I don't see it effecting the stability of the other services. Those changes that were made to the base code are running at CMU so they should be fine. That being said, if httpd is complete crap, then it could effect the

Re: cyrus-imapd-2.4.17-caldav-beta5 released

2013-06-12 Thread Ken Murchison
On 11. 6. 2013, at 16:01, Ken Murchison mu...@andrew.cmu.edu mailto:mu...@andrew.cmu.edu wrote: Hi Ondrej, I have not done any testing with unixhierarchysep or virtdomains. From the error messages below, it looks like the mailbox name translation is not being done properly, or not being done

Re: cyrus-imapd-2.4.17-caldav-beta5 released

2013-07-01 Thread Ken Murchison
On 07/01/2013 06:25 AM, Ondřej Surý wrote: Hi Ken, On Wed, Jun 12, 2013 at 7:12 PM, Ken Murchison mu...@andrew.cmu.edu mailto:mu...@andrew.cmu.edu wrote: Both of these, along with some other fixes that I worked on at the CalConnect interop testing last week, will be part of beta6

Re: nntptest little issue !?

2013-09-23 Thread Ken Murchison
On 09/14/2013 10:19 PM, Diego Woitasen wrote: Hi, I was using nntptest to test a Cyrus NNTP server with Kerberos enabled. It didn't work at first because nntptest didn't send the initial challenge. I had look at the code and saw that sasl_client_start() doesn't return anything because the

Re: Issues with 2.4.17-beta9 beta10 on OSX

2014-10-08 Thread Ken Murchison
Hi Marty, The sanity check of the vCard properties is done in store_resource() in http_carddav.c I also recently committed a logic error that was preventing vCards to be updated with the BusyContacts client which may have been effecting other clients as well. On 10/08/2014 03:33 AM,

Re: Review Request: backports to support imap4flags in cyrus-imapd-2.4

2014-10-31 Thread Ken Murchison
I think backporting strarray is just fine. In fact, I may have already done so for caldav-2.4. I know I backported a bunch of struct buf API additions. On 10/31/2014 02:51 PM, James Cassell wrote: Hello, I've backported six strarray_t patches to cyrus-imapd-2.4.17 to support the sieve

Re: Misbehavior cyrus caldav and evolution. Calendar is readonly.

2015-01-28 Thread Ken Murchison
Hi Stephan, Thanks for your work on this. I am actually at a calendaring conference at the moment, but I will look at this later in the week and do some Evolution testing myself. On 01/28/2015 12:21 PM, Stephan wrote: Hi, I am aiming to run cyrus with caldav and carddav support. On the

  1   2   >