* Modified the text client so that the <E>nter message command does not work in room types other than BBS and Mailbox.

* shutting down wildfire seems to work now.

* free the complete handler struct, the added stuff too.

* more entry checking.

* zapped some of the 'TODO: document me' in citserver.c

* fix the server cookie feature.

* Noted in rdf_parsedate() that the YYYY-MM-DDTHH:MM format is actually a quasi-standard known as 'W3C DateTime'.

* use toplevel thingie tag to detect whether we're RSS or ATOM.

* Adjusted the 'caltoday' CSS to make it a color that doesn't render the text unreadable.  Also fixed a stylesheet bug that put today's date number in the middle of the box instead of on top.

* Silence warn_unused_result in setup.c

* Reverted r8058 because it is crashing Uncensored quite a lot.

* Restored the parsing of <link> tags in RSS feeds.

* add author to atoms
* fix author for rss

* fix crash in downloads.c; memset should prune SubTP not TP.

* set correct buffer end pointer, so we don't run into unknown ground

* fix buffer overrun conditions in StrBufRemove_token
* terminate string when cutting it

* silenced more warnings

* Silenced the remaining warn_unused_result in setup.c

* Gratuitous style cleanup of some of the imap code

* start atom feed reader implementation, most of the freshmeat.org feed seems to work now ;-)

* arm the thread on shutdown killer. killing is better then waiting for a -9 from the admin.

* If patch fails, send rejects to /dev/null instead of cluttering /tmp with them.
* Report pclose() failures in serv_wiki.c because they probably indicate that a diff or patch command failed.

* Currently working through a bug that's preventing wiki history pages from being replicated over the network.

configure will now detect the Boehm-Demers-Weiser garbage collection library
We don't actually use it yet but we will when I add the libsee javascript
engine that recomends it.
We will probably need to add the Boehm-Demers-Weiser library to easyinstall
since it seems only debian based distros have it as a package.

* function signature was wrong. luckily we didn't use that additional parameter.

* when reallocing the buffer, flush the external pointer

* Added a new category 'fallback smart host'.  These are smart hosts that are used only when direct delivery fails.  It's just the thing for those sites that have a smart host available but prefer not to use it.

Add a newline at end of file to stop some compilers moaning.

Fixed the SMTP sending as a thread. Now the thread doesn't think its
running when it isn't.

* Switched the SMTP Client back to a simple timer-event hook with a static variable lock.  I couldn't figure out why davew's locking system didn't work.  The code looks solid, but for some reason, after the first time it hits that 'already running' thread, once that first thread exits it never starts again.

* Trying to figure out why the SMTP Send queue is stuck on Uncensored.  I corrected one edge case where it was possible for the thread to exit without unlocking, but I don't think we were hitting that case.

* Renamed the 'wikipage' variable to 'page' in the enter message loop to make it identical to its corresponding variable in the read message loop.  This allows the page name to be displayed in the room banner during editing.

* Fixed a bug that was causing the initial visit to a Wiki room to fetch the page list instead of the home page.
* Wiki page lists, including search results, are now lists of links to those pages.

* The 'Search' box now works properly in a Wiki room.

* wiki page list

* same bug at StrBuf_Utf8StrLen too.

* fix exit point in Ctdl_GetUtf8SequenceLength; condition was wrong
* fix StrHtmlEcmaEscAppend; its utf-8 sequence mechanism was deffective
* fix StrBuf_Utf8StrCut; it needs to count glyphs and bytes in different steps here

Removed an un required extern

Removed an unrequired extern declaration

Added a FIXME tag.
This module is REALY horrible to the system.
It carries out IO whilst it still has the context list locked.
That alone should merit disabling this module.

Grabbed a chunk of code out of citserver.c cmd_term() and used it to create
a new function
CtdlTerminateOtherSession()
Placed this function in context.c since it manipulates the contexts.
Exposed this function to the API

Noticed we don't seem to have a client that uses the TERM command.

Moved terminate_idles_sessions into context.c since it is purely manipulating
the context list.

Cleaned a few warnings about deprecated functions.

Code clean up.
Removed some old code that wasn't used.

* don't end the header, end_burst does this for us.

Moved BumpNewMailCounter() into context.c because it manipulates contexts.
Exposed it to the API as CtdlBumpNewMailCounter() and deprected the old name.

Grabbed another chunk of code out of user_ops.c, purge_user()
This chunk checked if a user was logged in by testing the user numbers
It is now a function and exposed to the API as
int CtdlIsUserLoggedInByNum(int num)
Also marked the code with a FIXME since it is unsafe, a user could log in
after the test but before the delete and realy mess things up.

Grabbed that previously noted bad code from rename_user that walked
the context list without locking. Used it to create a function
int CtdlIsUserLoggedIn (char *username)
and exposed it to the API
Oh yes and I fixed it to walk the list safely.

* storing the message list sort order in bbsview breaks the mailbox view. this wasn't found yet, because of a missing </script>. Disabling for now.

* skeleton code for a wiki pagelist screen

* headers_euid() needs to look at the 'E' field, not the 'U' field.

* removed a FIXME tag

* Added a new message field '1' which may be set by BeforeSave hooks.  The presence of this field excludes the message from being included in the full text index.
* Set this field on wiki history pages.

Exposed logout() from user_ops.c, deprecated it in accordance with
the published coding style.

Exposed CtdlLocateMessageByEuid in ctdl_module.h and deprecated
locate_message_by_euid in euidindex.h
Another step towards the documented coding style.

* fix message escaping after adding attachments to the currently edited message

* since we load vcards as message body now, finding a vcard attachment already loaded is no mistake.

* forgot to change EM->em here

Clean up a comment.

Moved the prototypes of CtdlLoginExistingUser and CtdlTryPassword into
ctdl_module.h.
They constitute part of the API for modules and we shouldn't need to
include user_ops.h in modules.

Added a log message to test for a problem in worker_thread

* this way its fastest: range check whats allowed and mask the rest.

* microshits RFC Violator will just show the tinnef part of a message you reply to, so you'll get phonecalls why you send 'em back their mail. -> ignore tinnefs while replying to mails

* Suppress the spurious 'message_content' div that gets stuck into wiki page edits.

* When saving a wiki page, set the message subject to the page name.  This will make it more trackable in other software.

Changed smtp sending to use a thread.

This change is the result of circumstances seen on Uncensored.
A bad host for a mailing list address was taking 4 minutes to time out.
This caused a general problem for Citadel because the EVT_TIMER event that
drive SMTP queue running was then tied up and unable to process SIEVE and
other stuff. Indeed the whole housekeeping stuff got blocked.

Basically it hilighted that we should NOT be doing IO during anything
trigered by housekeeping.

This change does nothing to improve the situation with the bad host but
it will allow housekeeping to work properly even if SMTP gets stuck.

Description of change.
======================
The outbound SMTP queue was originally processed durin an EVT_TIMER event.
This has been changed so that the event creates a thread to process the
queue and then returns.

To prevent the problem where more than one thread might be created to process
the queue a mutex is tested before the thread is created.
If the mutex is locked then no thread is created because there already is one.
If the mutex is unlocked then we lock it and create the thread.
The last operation of the thread is to unlock the mutex.

* predefined table is a little faster...

* don't use sprintf while doing urlescappend; do it in place.

* just calculate the language for non-static requests; else we will run this useless for every picture we deliver
struct change -> make clean

* first check for done before running into failed buffer reads

* use right function to load the room list from the server
* cheaply check for 000

* 64 bit int / long blablabla

* When performing a network poll of another Citadel node, connecting to a node other than the one we were expecting is now an error condition that causes processing to immediately stop.

Fix the build problems revolving around CitContext and context.h
Must have missed a make clean before I comitted.

Header file ordering and dependancies are a nightmare in modules. Sooner
I get that cleaned up to only need ctdl_module.h the better.

* apply the_mgts patch to fix the menu corner
* write em lowercase its in some i18n ed strings...

Add some comments to rename_user indicating why the code is bad.
It traverses an unlocked context list and it leaves a big hole for
the actual operation to do something unexpected.

Increased the load average before strangling the server. Its now 10.00
we will make this configureable later.

Also some cleanups in the CitContext stuff to make it a bit more readable
though I thought I had already committed those :-(

* don't call make distclean, this creates hickups

* close tag, thanks to the_mgt to find out

* add trailing template after the bbs view

* Completed the code to revert a wiki page to a previous version with just one click.  Combined with the previous commit, the code paths for 'view' and 'revert' operations are now unified in both citserver and webcit.
* Re-enabled the ability to create Wiki rooms.  Our wiki module is now functionally complete in a barebones sort of way, and the on-disk format is finalized.  I will probably add some more bells and whistles (such as 'compare versions') before the final release.

* Reordered the wiki rev code

Allow simulation mode to be terminated before completion

Added a new command line option to citserver "-s" takes a numerical parameter.
When this is specified with a number greater than 0 the server will start
almost as normal except it will not start any workers or module related
threads.
Instead it will create the specified number of simulation threads and time
how long it takes for them to be created, create a context, delete the
context and get cleaned up. A message is written to the log stating how long
it took and then the server will exit normally.

This is to help tune the client connection code and thread creation stuff.

* did some work on the wiki history reverter

* Made some annotations regarding bug 368 in the code.

* removed a spurious trace msg

Fixed the bug that prevented DOWN from working properly.

Also broke the context handling routines out of sysdep.c and other places
and put them in context.c / .h
This has been done to make it easier to improve the speed with which we
create / destroy contexts and threads.

Fixed a bug in the threading code that would prevent new threads starting.

It was possible for threads that are running (not idle) to still be
marked as Blocked (idle).

Also Identified another problem that could prevent threads from starting.
If all existing worker threads were given a task during the same GC tick
then the system load average would go to 100% and prevent new threads.
This was due to a bad method of calculating the load average. We always
wanted the machine load average to limit the creation of new threads
so that the machine didn't become overloaded. Now we get the machine load
and use that to limit new threads. We also log a WARNING in this event.

* by the_mgt: wrap a box around the knrooms treeview so it looks nicer

* Noticed something that was technically correct but confusing: when viewing a wiki history, each line displayed the date and author of the diff, which is NOT the same thing as the date and author of the page you would get if you reverted to that version.  Configured some off-by-one logic in the display code to compensate for this.  As an added bonus, the last line, which simply displayed an empty message, has been replaced by a new first line, which displays the date and author of the current version.

* Completed the WebCit UI functions to view a historical version of a wiki page.

* Code and comments cleanup

* Performing a 'fetch' operation on an old revision of a wiki page now stuffs the reverted version into a room called '9999999999.WikiWaybackMachine'.  Due to the invalid namespace, the DAP will clean up after us later.

* Wrote the code to fetch a particular revision of a wiki page and stuff it back into the msg base

* Completed the code to patch a wiki page back to the desired revision.  todo: do something with the results, such as output or reply the reversion

* version blabla not overwrite me apt update

* used wrong encoding for room links

Don't register commands twice, only register them before threading begins.
Because its filling up my log unecesarily.

* Disable creation of wiki rooms temporarily