leave out some debug logging we don't really need any more.

leave some, which seems like it might come in useful.

some trawler tried to login as "" on one of the open servers just now, and
didn't get the response he should.

for a moment I thought we might have a security problems, but no such
luck. all this change does is ensure that the trawler gets a "login
refused" next time, as opposed to getting a long good silence.

Update VERSION.

remove the debug logging we don't need any more

make sure that unsent meaningless messages cannot clutter up the queue

also more logging. this logging has to die before release.

add some temporary logging to help us find out exactly what we tell tb

appending an int to a string will work, but not work

I already wanted to sweep the source for these, why didn't I do it?

this MessageCache should suck a lot less

if the sieve script in the db is bad, try to say something about why.

log the SMTP transaction ID explicitly when we assign it

support SORT=DISPLAYFROM

teach the selector how to search on many/all mailboxes at a time.

partly tested, not used.

aoxexport will use this in one of the next submits.

untypofy

cleaner

patriccia tree nodes, too, contain data that just isn't pointers

the MessageSet blocks are big and contain no pointers, to make the
Allocator's job easier.

tie the SMTP/LMTP id in the Received field to the log ID used

the tie isn't perfect, but it's good enough to be useful, I think.

largest() didn't work except for the first block. this should do.

remove() was moderately fucked. unfuck.

! oops

select the right log message

remove() now works

unfoo MessageSet::csl()

make remove() clean up empty blocks, so isEmpty() works and is fast

Rewrite MessageSet to improve some Os, particularly index() and value().

Remove MessageSet::where() and update callers.

Avoid calling MessageSet::add( 1, UINT_MAX ) since that now consumes lots
of RAM.

The new MessageSet uses a number of blocks, each containing one bit for
each number in the set. The memory usage is proportional to the number of
UIDs in the set, but tends to be low if the UIDs cluster together.

index() and value should be faster than they were, but can still use some
optimisation. but this works, so I submit it.

rename PatriciaTree::Node::payload to data, to match List::Node

print *anyIterator.cur.data will print the thing pointed to

give PatriciaTree (and thus Dict and Map) first() and last().

I'm not sure these match the List equivalents, but they do what we want
them to do, so they'll do for now. maybe List should change to match this.

no comment.

try to tell the allocator that everything in the Node may be a pointer.

there's a commented-out line that we can try if this one works well.

simplify some code

give the Node constructor a line so we have somewhere to break

make Query::bind( const MessageSet & s ) O( s.count() ).

this fixes half of tb's performance problem. the other half requires some
really clever hacking in ImapResponse or MessageSet, so we can convert
lots of UIDs to MSNs cleverly.

do not free memory we might want to use.

I love GC and wished we could use it even in the Allocator.

try to optimise Query::bind( List<uint> ) a little.

I saw it take a whole minute earlier today. oh no.

kalyani's g++ doesn't like patriciatree... see if this makes it work.

yank the query timeout down to 10 again.

I changed it to 1000 while valgrinding. I think we need a better solution
for valgrinding.

Log backend pid at debug.

UseLdaPAuthentication is misspelled and unused.

update callers to use the new function names introduced in the last two
commits.

rename *::take() to remove(), so we only have one function name to remove
things from data structures.

rename String::encode() and decode() to encoded() and decoded()

teach eQP() to OPTIONALLY defend against ^From and ^--, and use a hack in
encoded() to use the right form of eQP()

enough friendly frienship to satisfy lochnagar's g++

base Dict on PatriciaTree

remove keys(), change all callers to use Dict::Iterator

misc other changes to callers

clarify doc

add a Patricia tree and base Map on it instead of on an n-level m-way tree.

Minor logic error in addressPermitted().

Retain a different log() object for the ByteForwarder.

Revert the ByteForwarder logging changes meant only for debugging.

Don't spam the logs unnecessarily at info.

*OUCH* flush even if we don't have anything to write NOW

we might have written something earlier, that we want cryptlib to flush.

also log the exact error message as info, not error, since it isn't
necessarily an error. most commonly it's just a timeout.

a faster, simpler Flag::allFlags

logging the cryptlib error only when everything's okay makes little sense.

I think that was what the code did. so let's try it differently.

double the speed of large fetches ;)

computing the MSN can be slooow, and we don't need it here.

let's see if modseq can still be zero after this

compile, dammit!

support rcpt to: <user@example.org.>

and give better errors anyway. I didn't understand the problem.

the byteforwarder logs should be children of the connection that uses them

log more usefully in the byte forwarded.

come to think of it, bf could do even better. much better.

5258 changed the capability name from the draft

needs to go into the bugfix releases

fetching the database ID is cheap if we're querying anyway, and we
sometimes/often want it, so just fetch it.

we do not reveal where a logged-in user sits.

we do reveal that to the administrator (in the connections table and the
logfile), but revealing it to correspondents (in the Received field) is
unwarranted.

I just realised that I sometimes don't know people to know where I am.

emit the right kind of boundary line for zero-part multipart messages

I don't entirely like this diff... it smells too much of a clever
programmer.

handle zero-line COPY statements

(without this, zero-bodypart multipart messages cannot be injected)

don't segfault, even if a client closes the connection at the worst
possible time.

wait for the banner before sending EHLO

should satisfy exim when smtp_enforce_sync is on.

Remove tried_at.

Update version.

subtransaction errors aren't.

subtransactions fail all the time and all that happens is that we ROLLBACK
TO SAVEPOINT and go on from there. no sense in logging an error.

Revert "send NIL for message data we don't have" and friends.

don't eat quite so much CPU updating unchanged mailboxes.

two more changes for the future:
1. use the mailboxes.change number suggested in the source
2. use a patricia tree based on full name instead of the child/sibling
   structure (which melts down when a single mailbox has tens of thousands
   of direct children).

don't run more than one mailboxreader per 2-3 second period, and don't run
more than one at a time.

if the mailboxreader is clogging up pgsql, this should help greatly.

http://archives.oryx.com/archives/mailstore-users/2342/thread

compile

don't bother updating the mailbox tree if we're shutting down already.

untested.

fix selecting newly created views.

not one, but several typos prevented selecting a newly created view from
working. also some generic cleanup, done while I was digging around in the
code. maybe some of it is unnecessary or irrelevant, I didn't check.

aox show search address asdf@asdf did not produce the right result. fix.

three changes:

1. the parser produced a suboptimal tree.

2. the display logic didn't work for Or.

3. added command line option -s to show the SQL as well, since I needed
   that to verify that the search was correct. undocumented for now, since
   it's a little shady.

make tests for absent header fields work

ie. 'messages which do not have header field x'

allow upper case in aox add user etc.

the dm_mm index may exist already when upgrade schema is run.

I chose to do it this way (create unless exists) because the three
alternatives seemed worse:

- requiring aox tune database after upgrade schema would suck
- automatically tuning would require that we store the desired tuning,
  which makes it hard for people to tune by hand
- we could support hand-tuning if we tried to classify the running
  database at upgrade schema time and adjusted the tuning in about the
  right way, but that task is MUCH too vague for me

only unblock imap commands if what we did (might) make a difference

this unfucks select, which is really rather fucked all by itself, but not
so much that we can permit it to be broken.

detect whether the message is there, take 5,

test for header fields if we know those. test for bodies if we know those.
one or the other has to hit in every important case.

take 4, or so. if the header contains NO fields, then the message isn't
there.

just looking for From wasn't enough, since we may have retrieved
header_fields rows but not address_fields ones. This test should be... no,
it shouldn't be better. We might have retrieved bodyparys, but nothing
else.

OK, take 5 coming.

oops. just oops.

best to make one delivery attempt after expired_at, just in case expired_at
passed while the server was down and the admin was fixing something.

keep sessions half-alive until the end of the commands that use them.

this will not lead to bad IMAP, because ImapSession checks the active
Session before sending the response. the only effect is that commands are
permitted to run to completion more often, e.g. when someone sends FETCH
and immediately LOGOUT.

move OK [CLOSED] to where it'll be sent whenever needed

we can send NIL for a message's various bits if the message has no From.
all unexpunged messages have From.

a better try at sending NIL at just the right times; still not quite good
enough.

if we expunge a message, then we won't be able to emit any news about its
possible recent flag changes.

oops, a syntax bug

doc update

a sprinking of paranoia

send the (modseq) appendage on the SEARCH response if so directed.

accidentally broke this week.

exists/recent/uidnext responses are like all others: once sent they've
been sent.

this fixes an actual bug in my 1200-line diff from last friday. oh no.

teach pop select to work without Session::refresh().

that's rather unusual code, needs to be explained

Unbreak select, which was moderately broken.

Select used to depend on various moderate breakage in session. session now
uses a new saner approach to sending responses, which apparently took
select by surprise.

In detail:

1. Remove Session::refresh(). Session::emitUpdates() now calls
   IMAP::runCommands(), so refresh() makes no sense for IMAP. (This might
   cause some breakage for POP though, when a nonempty mailbox is opened,
   we'll have to look at that.)

2. Remove callers to refresh(), including Select, which happily no longer
   needed to call it at all, thanks to ImapResponse.

3. Remove some dead code in select.cpp.

send NIL for message data we don't have

this is kind of dubious, but I think it's OK. dovecot and cyrus do it, so
who cares about the RFC? ;)

a one-day timeout for idle, instead of half an hour or three hours.

outlook remains broken. any timeout shorter than the interval between two
incoming messages will cause breakage with outlook, and a three-hour lull
is easily conceivable.

a 24-hour timeout still lets us flush brokenly broken connections.

another try: make sure DA doesn't lose its space in the SC queue as long
as it's in the SM list.

the previous try recursed, this one should not.

log more.

logging is good.

try to send responses when we have some, just in case we're in idle mode

keep mail spooled for delivery for two weeks rather than two days.

record the software version in Received, since other software does.