upload.c: skeleton code for load_attachments_from_message()

uploads.c: made the syslog less verbose

Also documented the JSON return values in api.txt

view_mail.js: more improvements to mail forwarding

 * "Fwd:" instead of "Re:"
 * The "--- forwarded message ---" po string is carried over from webcit-classic
 * Make the editor aware that a message is being forwarded
 * TODO: load attachments from the message being forwarded

Keep the "Attachments:" button updated.

When an attachment is added or removed using the upload window,
also update the count in the button.

"Number of attachments" div name is now variable.

Actually it's an array of strings, and the upload module will
update every div stored in there with the attachment count.

view_mail.js: remove upload from local array when deleted.

This makes the Attachments: counter decrement correctly.

view_mail.js: removed the is_quoted parameter.

is_quoted is now determined by whether quoted_msgnum is nonzero.
In mail, we always pull quote for replies.

view_mail.js: fixed handling of references field

Converting recipient arrays to recipient strings:

Fixed the dedupe so it works before now, and
also made it DRY.

databaselayout.md: updated to remove networking

Fixing the parameters sent to the mail composer.

Reply/ReplyAll/Forward need some things that are not strings
and the source arrays seem to still be in scope, so we go with
that.  Need to tune this up some more.  Some things are still
missing, such as the references and reply msgnum.

view_mail.js: make the Forward button appear.

Needs a lot of work.  Currently it is just a reply with no addressees filled in.
It needs to not quote the message, and carry over the attachments.

"make cc/bcc visible" now makes BOTH visible.

Somewhere along the line it lost CC and only showed BCC.

serv_extensions.c: style cleanup from 30,000 feet

No, I'm not takin a broader view of the code.  I'm actually on an airplane.

Focus on "To:" for new messages, body for replies.

mail view: focus the cursor in the correct place when composing

Restored the gap before the "Hi From Stu" sound.

Now I see why he put it there in the first place.
Some players (including the one in my browser) cut off
the first bit of sound.

Only attempt to insert quote when replying to mail.

Also do that after rendering the attachments window.
Doing it before doesn't work.
I don't know why.

Removed temporary diagnostic output

upload.js: when removing an attachment, delete the div

This is handled the usual way, when we get a successful response from
the HTTP DELETE method, it triggers the removal of the list element
showing the attachment.

When a mail compose is canceled, flush all uploads.

Most of this was already in place.  We now just run through our upload
list and do an async DELETE operation for each one.

upload.js: moved the delete-an-attachment skeleton code to its own function

Fixed the MIME boundary after the message text.

It had the trailing dashes in it, so it was getting interpreted as the end of the message.
Now that's corrected and the parsers of the world are happy.
Also, we eliminated the double-nesting thing, and everything is hunky dory now.

Quoted-printable encode HTML messages on the WebCit Server side.

This allows us to do multipart with attachments without nested encoding.

webcit-ng: fixed the way HTTP uploads are fed into the MIME Parser.

mime_parser() expects HEADERS+CONTENT but our h.request_body only contains CONTENT.
The fix: our transaction struct now has h.request_body_with_synth_headers that
begins with a synthetic Content-Type: header followed by a blank line and then the
request body.  h.request_body is now just a pointer into that buffer, starting at
the position where the actual body begins.

This gives us the ability to supply the body with or without headers, without having
to make two copies of it.

considering nuclear weapons

Looks like I need to remove some optimization cancer from mime_parser.c, but first, a bit of style cleanup.

Progress on upload

room_functions.c: remove diagnostic hex dump of recipient field

json.c: encode JSON strings the correct way (RFC 7159 section 7).

The previous code used the JavaScript encoder from the stringbuf library, which
is *not* the same encoding.  Now it doesn't make the browser's parser barf when
it encounters a control character.

War on old style continues.

I really ought to be writing some real code tonight.  But I get into a file and I see
old style and I want to make it clean and shiny and it's easier than the kind of heavy
brain work I should be doing.  When faced with a heavy think, I dawdle for a long time
before I really get started.  I've done this since I was a child.

stringbuf.c: chipped away at style modernization

This is a big one with a lot of outdated style in it.
Folded lines which were >80 columns but <132 columns.
Add {} braces to single-line conditional blocks.
Replaced C89 comments with C99 comments.

vcard.c: style cleanup

Changed some C89 style comments to C99 style.

This is what I do when I'm slacking off and should be doing something useful.

removed an unused function

Saving my place.

Note to self: there is something in the mailbox of admin@dev that is
causing the renderer to fail and the javascript console emits unhelpful
messages.

loadtest: counters row offset from banner is definable

loadtest: counter position is now derived from thread id, not socket number.

This is what I was actually trying to accomplish a couple of commits ago.  I made
a race condition and that's why it was malfunctioning.  This way is better and it
works.  It also once shot a man in Reno just to watch him die.

loadtest: paint the screen in a separate pass

tried some random cosmetic stuff and removed it

Revert "loadtest: change display"

This reverts commit fdd4f3034903e50395e72acf74636e4cb12c2291.

loadtest: change display

server/housekeeping.c: fix a small compiler warning

More progress on attaching attachments

messages.c: inching towards saving the attachments

Don't try to destroy the upload window if it doesn't exist.

Got multipart/mixed being transmitted

deactivate_uploads() hides the window.

broken: attachments window cannot be a child of the editor body

It causes the attachments window to be saved as part of the message text.
Oof.

upload: reduce attachment identifiers to 9 characters.

Also switched from a unicode X to a fontawesome rounded-X for the
'remove attachment' button.  The unicode one was too big.

move struct uploaded_file to webcit.h

loadtest: small file cleanup

upload DELETE method

Various operations in the /ctdl/p/ hierarchy to handle uploading of attachments

❌ marks the spot

uploads

upload.c: hold uploads in temporary file handles

upload: change REST path from /ctdl/a/upload/ to /ctdl/p/

api.txt: updated

libcitadel/configure.in: renamed to configure.ac

upload.c: rename "tempfilename" to "ref"

This will hopefully discourage people from trying to play stupid tricks
involving guessing the filename and trying to escape the containment.

upload.js: moved these functions into own file.

Good clean separation of functions.
This should allow us to handle uploads in lots of different places
in the system.

array.c: minor changes to comments

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