Syntax.

*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.

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.

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

(Edit: Integrated with some changes: 2.x has no Granter, and no DbSchema
support. -- AMS)

Remove tried_at.

Schema #76: minor cleanups.

This means both the new threading code and the new spoolmanager code
must go into 2.12, which I think is fine.

Make the DeliveryAgent work without tried_at.

be more careful about what looks like a boundary line and may need qping

this should prevent '---- BEGIN PGP BLAH ----' from matching, which is
good, because a certain (inexplicably) well-regarded mail client can't
handle qp'd PGP messages.

q-p lines that start with "--" or "From "

If a string includes a line starting with From, but there's no other
reason to q-p it, then we don't q-p it.

send \noselect when a client asks for the separator

pattern a*b may match children of a/c.

do lsub correctly. 2.12 fodder, (assuming that anyone ever cares about odd
lsub behaviour).

that _is_ an error, so log it like that

roll back a transaction if something throws Invariant while processing it

Be careful to not hold locks and die.

Use a faster query to see whether a mailbox is really empty.

This also should evade a type problem: count(*) returned a bigint
for Simon Heimlicher in the previous version of this query, so I
cast it to bigint in case there's a version discrepancy and then
use getBigint() to retrieve it.

Schema #75: Restore index on bodyparts.hash.

Signed-off-by: Abhijit Menon-Sen <ams@oryx.com>

permit two bodyparts to have the same MD5 hash.

the injector has accepted hash duplications for a few weeks, and we don't
use the hash for anything except speeding up injection, so this is now
okay.

Fix some URLs.

authenticate can also contains passwords (base64-encoded, but still)

the code to not close listeners only caught one of the two instances. this
catches the other.

also closes the connection instead of just forgetting it.

also forgets about connections even if they're already closed.

order RCPT TO commands more comprehensibly

the old order was stable, but that wasn't easy to see. now it orders them
by domain,localpart.

aox delete alias would fail if there were several rows in the addresses
table with that localpart and domain (e.g. blah@example.com and Boris Lah
<blah@example.com>).

Fix comment.

Don't close Listeners.

We have no good way to identify the Connection that originated a query
in Query::notify(), but we can be pretty sure that a Listener isn't it
while we're looking for parents of the query's owner's log.

Spelling fix.

support MAIL FROM:<...> AUTH=<...>

Avoid writing plaintext LOGIN passwords to the log file.

Done this way on the basis that it fixes the obvious case (a login x y),
and that login is an unlikely source of problems that we'd desperately
need the first line to debug anyway. Suppressing the line seems better
than trying to find and obscure the password (cf. "a login x {7}").

Doesn't do anything about AUTHENTICATE initial-response etc.

Fix another warning.

A stupid one, this time. g++ doesn't realise that i cannot be > 5, and
it complains about the array subscript being out of bounds.

Another warning.

Fix g++ 4.3 warning reported by Simon.

if an IMAP command needs a session and doesn't have one, then something
really bad is happening. this can happen if the client closes the
connection abruptly and the close isn't quite handled.

this change adds code to throw an exception if that happens, which will
close the IMAP connection without deliberation. bang.

probably should go into 2.11.

new function Address::lpdomain(), and caller updates.

String::replace(), untested and so on. both this and prepend() need
unit tests.

p4raw-id: //oryx/src/main@55139

improve the error message; I think the new text is more likely to be
correctly interpreted by J. Clueful User.

so how could this ever pass a single test? unsubscribe has to execute its
query, it can't just hope and pray that the query magically resolve
itself.

accept this:

  From: a@b, c@d, e@f
  Sender: a@b, c@d

accept Sender which is a straight copy of From, even if From contains more
than one address.

require needs capability names to be quoted

text/html; charset=iso-8859-1 has to have charset=

Yes, 8859-1 is the default for html, but we have to specify this. Not
doing it requires too much code to agree, and it WILL surprise someone who
uses ASCII as default, as the MIME RFC says to.

text/html has a default charset of iso-8859-1. if we see a charsetless
text/html bodypart in the db, we'd better assume that it's 8895-1.

calling List::count() where isEmpty() will do is sloooow

don't bother checking that a script actually uses the extensions it
'requires'. noone else checks and the RFC doesn't ask us to (I think I
added the checking because of wording in a draft).

move the From workaround that depends on this host's hostname so we try
everything else first.

move the Reply-To workaround that depends on From to after all the Froms.

content-type seems to suffer from more errors than other fields.

there's some general code which works around from::::: a@example.org, but
it only does anything when the field can't be parsed as it should be.
since c-t is suffers from clueless spammers, c-t can parse almost
anything, and often doesn't fail.

therefore, c-t now gets its own colon-skipping code. feh.

be properly case insensitive. thanks to simon heimlicher and the
squirrelmail developers, who like upper case.

make the reject and fileinto extensions magic, in that they can be used
without being declared.

$#@$!. I commited the wrong thing.

this allows empty scripts provided they have nonzero length.

accept empty scripts provided the length is nonzero

if we have to guess the content-type based on filename, then preserve the
original garbage for possible human inspection.

Accept garbage like this:

   Content-Type: multipart/mixed; boundary=\"asdf\"

Spammers must die.

be kind to avelsieve, which wants the capabilities in a given order

accept a whitespace-only script as valid

Accept Sieve scripts containing bare LFs.

The RFC demands CRLF, but smartsieve and avelsieve sometimes send LF.

accept a string argument to reject

typo2

typo

when we add a Received field, wrap it properly

and add a for clause if appropriate (or a comment)

uh, 'From ' should happen only for RFC822 headers, so handle only that

handle "From " lines correctly, even if they come in via SMTP.

this is suboptimal; we could look for the address, store it, and use it to
salvage some From: fields. but it's OK.

this change also moves the " Mar " code to work around Courier into the
Courier-specific code, where it ought to have been all the time.

accept and ignore "To: Valued Customer <>"

Make jam clean and git status play nicely.

HeaderField::ContentBase has to go. this has been a bug for ages.

should go into 2.11.

latest DNS TLD list. this goes into 2.11, woo hoo.

mutf7 MUST end in ascii mode. '&' is not legal, '&-' is (since - toggles
back to ascii mode)

add a plan b for malformed boundaries.

if we can't parse the parameters but the boundary seems to be there and we
need it, then try _really_ hard to dig it out of the nonsense.

fetch the recipients for each delivery in a predictable, easily testable
order.

try again quickly (then slowly) if the smarthost doesn't answer

lock the mailbox before deleting it. fixes all races.

call commit() instead of rollback() in expunge if we discover that we
don't need to do anything, and haven't done anything either.

the rollback would finish late sometimes and I'd check whether it was okay
or a timeout.

Fetch (use)names as text.

name another possible error in a sensible way.

maybe we could use the codes too. 23505 is always a UNIQUE violation.

ignore parse errors in mime field parameters silently and stop parsing the
parameters (with two exceptions - we keep looking for charset and
boundary).  the variety of errors in parameters is too great and the value
of the errant data is too low to care.

I'm being stubborn. "any header contains ''" is true.

change the table comments so they'll read acceptably (not perfectly) in
the table list on aox.org/db/

Fix typo.

Activate schema #73.

Code to upgrade to schema #73.

Revision #73: Store client address and port separately.

Don't listen on :: if use-ipv6 is false.

If no listen address was specified for a service, we would try to listen
first on :: and then on 0.0.0.0. Now we do the first only if use-ipv6 is
true. (We ignore use-ipv4 for this case, but it doesn't really matter.)

Thanks to Ron Peterson for reporting the problem.

Scope x( (Log*)0 ) should not overwrite parent's log.

if we want to close, and the client closes, that's ok. no point in logging
that.

Accept (and throw away) To: @host.na.me

parse content-type: charset=us-ascii

accept c-t-e: 64bit, 9bit and various other deformed names spammers use

trim trailing whitespace from all subject fields

accept c-t-e: "quoted-printable" as well as quoted-printable. wrong, but
feh.

some people place far too many '"' characters in their mime fields. skip
some harmless ones.

the previous commit could loop infinitely. oops.

parse 'Name: : : : value' as though it were 'Name: value' if that succeeds
and the original fails.

now that ContentType does this, Header doesn't need to

stop parsing a mime field if there's a parse error, and it seems to be
because a second field is glued to the end of this, e.g.

  Content-Type: text/plain; charset=x; Subject: y

remove the charset in "text/plain; charset=us-ascii" as part of parsing the
field instead of as fixing the header. this makes a difference when the
message has more than one c-t fields.

do the same for html/latin1.

fix the charset's name if it differs from the correct name other than in
case (so iSO_8859_1 becomes iso-8859-1).

if the message does not contain any sensible mime type, but does contains
a suggested file name, then we try to guess the mime type based on the
file name.

press the start button to stop the computer.

split Subscribe into two and have Unsubscribe do its own work.

Subscribe used to prevent insertion of duplicate subscriptions rows. the
code didn't work, not sure why. it did have a race condition, but surely
we didn't hit that on kalyani? anyway, this version doesn't try quite so
hard, and expects the database constraint to do the rest.

get rid of Retired Commands before letting anyone look at the list of
commands.

There was just too many spots that called command and immediately skipped
over the Retired ones.

we deleted that function

some more response codes based on draft-gulbrandsen-imap-response-codes

Make CRAM-MD5 correctly deal with spaces in the username (again).

Since we don't support login names with spaces in them, the error in the
previously submitted code is not important; nevertheless, since it broke
an example from some RFC, I chose to fix it rather than change the test.

Cast relname to text when fetching.