avoid decoding mime-parts unless we realy need to.

 - suff... doesn't need to decode all attachments; we just need the mime headers here.
 - when searching for the attachment we want to output, we also don't need the content.
 - when outputing an attachment, just decode the attachment were actualy interested in.

performance improvements: replace memcmp

 - looping forward while moving a pointer and doing memcmp isn't fast. replace it by strstr
 - \r\n can be done in a pointer dereferencing comparison.

Note: maybe we need to fallback to the memcmp way if there is binary content?
In that case we should do a little smarter and try to use contentlength-headers etc.

Add PID/GID/UID of communications parters with unix domain sockets to login-loglines.

This is an addition to Stuarts patch.

Log remote-IP on login attempts

By Stuart Cianos

Permalinks work now but they are ugly.

Began working on a permalink system WARNING NOT WORKING YET

avoid calling GETR for the number of total/read messages

so far webcit used a call of GETR to retrieve the number of messages available in the current room for displaying the room banner.
However, GOTO already gave us this information, and GETR is only available for aides, so we got Important messages with that error when logged in as joe average.
This went under the radar for quiet a while, since the error message wasn't printed but silently ignored.

Relax SMTP Client timeout settings as discussed.

 - default is 30 seconds
 - Server greeting 90 seconds (DNS lookups for RBLs might be slow
 - Server sending the response to the EOM sequence; here we might have background processing like virus or spam scanning that wants to finish before finaly accepting the message.

Began fleshing out the CSS for blog view

Only move from spooltmp to spoolin if there's actually something there.

Don't send identity header if we want to do gzip compression

Propfind replies send 'Content-encoding: identity' right now, which is superseeded by the frameworks 'Content-encoding: gzip' in case of the client indicates this.
sending that headers twice has uncertain results depending on the client; while the RFC demands to evaluate the later, some fail to read our reply then.

event_jid is a buffer, not a pointer, so don't free() it

Memleak: post_message(): don't forget about the recipient buffer.

Memleak: put request local strings into the http-header array (we used SVPUT to manage them in advance)

Memleak: CtdlMakeMessage(): don't forget about the original author.

smtp_do_bounce(): don't loose msg['A'] when overwriting its content.

Removed a few unneeded calls to scr_flush()

Began defining CSS for blog view

Memleak: free the list of search hooks on exit

Memleak: free the global inet-config on exit.

Memleak: free the XMPP-Eventqueue on shutdown

 TODO: the event_jid member contains an unclean pointer, no clue why.

Memleak: free the roomlist of the sieve-queued rooms on exit; other tiny leaks.

Memleak: fix the varous handler & config lists of the rss module on exit.

Memleak: free the imap-handlerlist on exit

Memleak: free the nttlist (network host list) on exit

Memleak: sock_connect() needs to call freeaddrinfo()

found one more place using IO directly, rerouting it to buffered IO else it will spoil the sequence.

New templates for rendering messages as blog posts and comments

Localized strings for blog view

fix buffer overrun while converting charsets

* DecodeSegment() didn't check whether one of the temporary buffers would be able to bear the transcoding results

msgbuttons are now back to being ordinary links.
They're smaller this way, and they also don't misbehave
under Internet Exploder.

Update translations to Launchpad R13

check whether the sorting algorithm specified via BSTRs is intended for us before using its groupchange detector.

add script to generate gcov/lcov output.

fix some incompletely transformed reply strings; thanks gcov & imaptest

more gcov files to ignore...

add a way to configure code coverage

Ignore data files left behind by gcov

Rewrite IMAP-Module to buffer its output

 - move defines from imap_tools.h into serv_imap.h
 - switch includes in sequence
 - we now have IMAP->Reply, which we write into, until IUnbuffer sends it to the client
 - IReply & IReplyPrintf do now send 'Params[0].Key BAD xxxxxx\r\n' to the client; this makes the code more compact.
 - IA* append stuff to the outbuffer that we want to send.
 - replace rumpelstilskin handler lookup by hashlist and central register calls.
 - replace if's in the sequence separating the handlers by their required pre-conditions by flags on the register call.

this should
 - (hopefully) fix some troubles with evolutions imap, which wouldn't like to read enough chunks until the line ends
 - speed up stuff like header fetching in huge lists, since we reduce the number of context switches here.

have flexible timouts while reading lines in our client mode connections

since some mailserver take a little longer to finish reading mails we need to wait after we send our '.\r\n'; its 90 seconds now, all other parts will wait 5s, quit will wait 1 second.

Nonblocking sockets need to select while writing

 we changed the SMTP outbound connections to be nonblocking so we could abort on not reacting SMTP-Servers.
 thus we need to select before writing, else the connection will fail.

handle errors during netedit:

 - put server errors into the ImportantMessage
 - use StrBuf
 - use GetServerStatus

New macro for putting a Citserver reply into important message: SRV_STATUS_MSG

Use it like that:
StrBuf_ServGetln(Line);
if  (GetServerStatus(Line, NULL) != 1) {
AppendImportantMessage(SRV_STATUS_MSG(Line));

if we tell the client we're in error state, return.

if SNET would fail to open the temporary file, it would tell the client 'error' and continue to read -> hang
we now abort after reporting the error.

don't fetch a RSS before the network frequency is elapsed

 currently the RSS-Thread is run over and over; we shouldn't stress our partrers to much, so we let the network poll frequency elaps before we try to poll the rss feed again.

fix crash in SSL BLOB-Reading

 - when diciding whether the read buffer contains more than the blob reader wants we had a wrong equation.

move old tokenizer over to the managesieve server; its only used there.

 - remove old_imap_tokenizer from imap_tools.h
 - move old_imap_tokenizer to modules/managesieve/serv_managesieve.c
 - move imap_tools.h to modules/imap; since its not needed globaly anymore.

Ups, its el_GR not el_EL

Do a more precise match to find out our UID

fix compilation

 - Makefile had a wrong include
 - msgfmt would abort on missing linebreaks

Cosmetic cleanup of previous two commits

Fixed the viewdefs code to handle the fact that enum 9 is now
drafts and 10 is blogs.  This fixes the select-o-matic thing.

viewdefs is now boolean

Blanks.

Blanks are beautyfull.

Add more url-shorteners that showed up in my way.

Update translation to Launchpad r7

Finalize fixing of netconfig loosing incident

 * need to set & check errno that tells us whether our tempfile was fresh created or already there
 * remove tmpfile on error
 * don't use fopen() because we can't specify the flags we need for that
 * ftruncate() the tempfile at the right place

Prevent writing empty network configs Part II
 * writenfree_spoolcontrol_file ()
   - serialise into a StrBuf
   - write that to a tempfile in the netconfig directory
   - if we fail, abort & log error
   - if we succed use rename () to replace the original config file.

add string representations of the RoomNetCfg as RoomNetCfgStr[maxRoomNetCfg]

* cmd_snet: make shure the target FS has enough space for the new config

 - Create our own filename insteaf of using mk*tmp*; CC->cs_pid should be uniq.; we use  O_CREAT|O_EXCL so we're shure its ours or fails.
 - Create that filename in the same directory as the filal config filename; by that we can be shure its in the same filesystem, and if thats full, we will fail
 - Stat the original file for sise (if not there assume 80) and write twice as much into that file so we're next to shure the content from the client will fit in
 - the citadel protocol is a little st00pit here, we don't know the size in advance, and we can't return an error _after_ accepting the file
 - instead of moving the content of the tempfile into the final file, we use rename() which is supposed to be atomic.

assoc_file_name(): return length of string

Translation update to Launchpad #6

 - add Greek
 - add Svenska
 - sort locale list

Moved the TinyMCE toolbar to the top of the editor window.

Added a little bit of clever JavaScript to make the TinyMCE window occupy
the full height of the browser window.

Upgrade TinyMCE

Don't output a space between imap namespaces
(patch suggested by samjam to fix javamail errors)

Merge branch 'master' of ssh://git.citadel.org/appl/gitroot/citadel

* blanks.

* blanx

* blank.

blanx?

blahangkx

* bllannnks.

* Blanks!

* blanks???

* blaahaaank

* o noes blanks?

oh, well, maybe a blank anyways here?

Merge branch 'master' of ssh://git.citadel.org/appl/gitroot/citadel

* oh noes, thas blank wasn't that nice after all.

blanks? blanks anyone? they're nice.

* make it const, not written anyways

* sock_getln (): CtdlSockGetLine() gives us lines without \r\n, no need to look for them again
* sock_getln_err (): add integer pointer returning the actual return state in case of error
* ml_sock_gets (): use sock_getln_err () so we can abort on impropper replies
* smtp_try (): make socket nonblocking, so we can abort on error
 -> we will wait 5 seconds for the remote site to send us the SMTP-State else we will abort the connection on failure.

Merge branch 'master' of ssh://git.citadel.org/appl/gitroot/citadel

Numerous style changes to the 'edit message' dialog.  The save,
cancel, and address book buttons now occupy the navbar instead of
a bunch of controls that shouldn't have been there anyway.  I
also fixed a javascript bug that was making the addressbook popup
silently fail in the background if we were in a room that didn't
require message recipients.
The attachment dialog now sits above the message body pane, but
this will soon get migrated into the navbar as well (maybe another
popup).
Finally, the background of the message entry div now more or less
occupies the entire space available to it; this still needs more
tuning in order to make the editor occupy the correct amount of
space in the pane.

* ParseMessageListHeaders_Detail(): don't overuse StrLength, we can know its result from other calls, etc.

* we already checked for '=' before, remove.

* use a lookup table to convert hex to char (like we already do vice versa) to be faster.
* don't use strcmp in CtdlDecodeQuotedPrintable

Merge branch 'master' of ssh://git.citadel.org/appl/gitroot/citadel

* ctype.h required for isdigit() and friends

* make the hash algorithm long, as hash now does

* don't free stacked pointers

Merge branch 'master' of ssh://git.citadel.org/appl/gitroot/citadel

The enter-message dialog is now contained by a div called '.entmsg'
instead of '.message' ... duplicated the existing styles for now.

* add a way to pass the workbuffers into the RFC-822 decoder, so we don't need to realloc it over and over
* add two temporary buffers to the WC Struct for that purpose
* add session logic to free them, and alloc them on need

"Subject:" and "Subject (optional):" were in the wrong places.
Swapped them to the correct places.

CSS3 gradient fill in the banner background ftw!

Blog rendering continued...

* Use new MSet-Matcher to retrieve seen state (10% of what we do in big mboxes)