Release version 996 generated by do-release.sh

Additional checks to keep zero-length messages from crashing the indexer.

uploads: return uploaded file info to caller

upload.c: more progress on returning values

do not require autoconf 2.71 wtf

Release version 995 generated by do-release.sh

serv_fulltext: don't try to index a null message

textclient: makefile no longer leaves behind test compiles

upload.c: write uploaded data to a temp file

view_mail.js: throbber and "Processing dropped files..." message appears

"Processing dropped files..." is not exactly the right message because
the upload message might have been clicked, but I don't want to throw
away the translations of that message.

Anyway, it's really cool, they appear in the list, and go away when the
uploads are complete.

view_mail.js: make the function names more consistent with webcit

(They were previously the same names used in the example, natch)
Also we're incrementing num_attachments

Simplify the upload dialog.

Reduce the number of divs and make it more generic.  Allow the upload dialog to
be attached to any parent div.  Make the dialog appear automatically when a file
is dragged into the parent div.

view_mail.js: move upload dialog from ctdl_big_modal to ctdl-upload

Uploads now have their own div in their own separate modal.
This is, of course, because I am trying to loosely couple the uploader so
that it can be attached to other parts of the system when needed.

Also I realized that my div names are ctdl-not ctdl_consistent with each other.
Trying to fix that.

upload.c: stub function for accepting uploads

view_mail.js: more progress on upload dialog

Improving my template-literal-fu

view_mail.js: convert ugly syntax to heredocs

Close attachments window on Cancel

view_mail.js: show/hide attachments screen, added title bar style and close button

loadtest: use a random password

Attachment download links target to _blank

Attachments now download with the appropriate filename.

This was accomplished by disregarding any REST components to the right of
the part number, allowing the filename to be placed there.  The filename is
sanitized to prevent script injection.

view_mail.js: better display of attachment links; attachments are now working

Displaying each one as a link definitely makes it stand out from the message text.
Display the size of each attachment --   _(bytes) is already in the webcit-classic translations
Downloads are working, but the wrong file name is derived

Where gettimeofday() is required, include both time.h and sys/time.h

removed some verbage, but really a git test

Additional memory allocated since startup: %d bytes message now identical to other components

Write server PID to citadel.lock in addition to locking it.

This will be useful for systems like FreeBSD that need to know a PID to monitor.

Fixed build failure induced by the new version of autoconf.

Die in a hail of capitalist machine gun fire, Richard Stallman.
Communism is straight from hell and so are you.

I have no words to describe the fuckery I am trying to reverse-enginner here.  Ever heard of COMMENTS?

Sanitize better.  "&" is handled before "<" and ">" so it doesn't sanitize the &gt; and &lt; ampersands.

Ampersands are awesome.  They look a lot like treble clefs, don't they?  Actually they hate each other.

Release version 994 generated by do-release.sh

webcit: sanitize instant messages against XSS type stuff

Obsolete sys/time.h and HAVE_TIME_WITH_SYS_TIME

cdb_chmod_data() : don't make so much noise unless there was an error

Suppress noise during configure

validate_recipients(): don't send mail to user 0

serv_expire: remove two TRACE directives I left in there by accident

TDAP: consolidated transcripts to a single summary

serv_expire: abort() if any malloc() calls fail.

This is drastic, but the alternative is to purge records that are still good.

Release version 993 generated by do-release.sh

berkeley_db: cdb_next_item() use DB_REALLOC, not DB_MALLOC

Fixes a 'Brown Paper Bug' that will leak memory ... catastrophically if ctdldump is run on a big db.

ctdldump/ctdlload: document the -y flag

serv_fulltext: better handling of exit by yield vs complete

Richard Stallman is a fucking left-wing communist douchebag.

textclient: don't prompt to change internet email addresses when deleting a user

The 'edit user' and 'delete user' workflows run through the same function since
the process of deleting a user simply edits them and changes their access level
to 0.  Add a check for cmd==25 (edit user) before asking the user if they want
to change the email addresses.

Omit the warning about session scheduled for termination but not idle.

Something about this warning makes it reference a bad pointer and crash
during periods of extremely high load.  I don't know why, but just leaving
it alone seems to fix the problem.

Release version 992 generated by do-release.sh

only build chkpw and chkpwd if we can

Remove the warning-this-is-unfinished warnings from ctdldump.c and ctdlload.c

Most of these tools are now building successfully on FreeBSD.

Include -lintl where we need it on systems that have it

Include -lresolv only if we have it

loadtest: run with a warning instead of exiting when not all threads connect.

loadtest: pretty display when running with hundreds of threads

fulltext: be smarter about exiting silently if there's nothing to do.

fulltext: after the indexer runs for 60 seconds (tunable), yield the thread

loadtest: minor changes

Release version 991 generated by do-release.sh

Moved the creation of the data directory.

It was happening too late in the new code and failing on some new installations.

Release version 990 generated by do-release.sh

Release version 989 generated by do-release.sh

On second thought, let's do it this way.

Revert "setup: reflect the fact that the operator should restart the server"

This reverts commit c7589e995080a55e55c587d65c85a5889beb4194.

setup: reflect the fact that the operator should restart the server

Removed background and restart from citserver.

These functions should be managed by an external supervisor program.

fix compiler warning

Renamed Makefile to GNUmakefile to enforce the use of gmake

Rewrite of the indexer is complete!

No more search hooks.  Modules can call each other now and that's a lot simpler.
We also eliminated the index cache, because it made things a lot more complex, and
a modern database is going to mmap most of the index as it gets used anyway.

Removed all the disused CtdlUnregister functions.

We don't call Unregister functions anymore because our objective is now
to close the databases and GTFO when the server needs to shut down.

search function is complete

Wrapping indexer transactions inside a critical section seems to make it not deadlock

Rewrote the indexer

Removed double-fetch-room in the indexer.

I don't want to celebrate prematurely, but we've gotten through 10,000 ops
in every of 10 threads without deadlocking.

free(msglist)

Totally ripping apart the indexer.

And yet, it doesn't seem to be anything 'deep' that's making it deadlock.
If I get desperate I may have to just give it its own thread or something.

serv_fulltext: wordbreaker now returns a libcitadel Array

CtdlForEachMessage() don't process message 0

threads.c: commented out try_critical_section() , I don't want this

loadtest.c: after creating accounts and rooms, log back in using a non privileged socket

Makefile: proper incantation for compiling loadtest

context.c: fixed invalid pointer

Temporarily disable the indexer

no diags

cdb_tick() called periodically

When opening a cursor, always read through to the end.

This should theoretically eliminate the need to ever call cdb_close_cursor()

cosmetic stuff

cleaned that up a bit

loadtest.c: multiple threadds

Generate listsub token with a hash instead of crypt

.gitignore: add dbnothing and loadtest

Makefile: use ${CC} selected by configure

Removed the 'ucred' stuff.  It's insecure and it doesn't work on FreeBSD anyway.

FreeBSD compatibility (first of several commits)

Removed extra diagnostics

diagnostics

Found a place where CM_SetAsField() was owning db memory.

Changed that, of course.  It was causing a double free.

CM_SetField() no longer accepts a length, just uses strdup()

loadtest.c: when deleting, always make sure we deleted 1 message.

If there are multiple loadtest processes running -- and that's really
the whole idea -- we will often try to delete a message that was already
deleted by another process.  That's not an error condition, but it would
cause posts to outnumber deletes and the rooms would grow endlessly.  So
if we didn't successfully delete something, we go back and try deleting
another one.

Fun fact: the algorithm used to decide which message to delete is the same
algorithm used in the 'fortune' program.

loadtest.c: added

berkeley_db.c: improvements to transactional data store

https://docs.oracle.com/cd/E17276_01/html/programmer_reference/transapp_env_open.html

The use of the DB_RECOVER flag to dbenv->open() appears to do more than just try to
recover the environment upon startup.  It also creates __db.00X files, which are
perhaps holding the environment???   In any case it seems to be more solid this way.
I am also removing the use of any "private" transactions, since they are no longer
needed.  Operations which write to the database check to see if the thread's dedicated
transaction pointer is in effect, maintaining that transaction if it is, and creating
a transaction in the same pointer otherwise.

There is a "double free" bug somewhere in here and I'm still trying to figure out what
conditions cause it to happen.

master_cleanup() is now the global shutdown/exit function