moar skeleton

webcit.h: regenerated function prototypes using cproto

calendar_functions.c: added skeleton

room_functions.c: get_msglist() now returns Array

Previously it returned a long* which contained a list of longs.
Assembling this required the same allocation logic used in our
array class, so now we're just using the array class.

Changed the arrangement of bytes in the universe

Updated copyright date in files where git blame shows 2023 mods

Added stub renderers for Calendar and Contacts rooms

views.js: convert strings to template literals

Found a place where 1TBS was not used, and fixed it.

1TBS is important.  People who do not use 1TBS are literally Hitler.

More template literals.

I really like template literals.

And now that I know how to use them, I will be using more of them.

Tuned up the CSS on more banner buttons

Make the font sizes less cartoonish at extreme viewport sizes

When forwarding an email, pre-load its attachments.

This completes the effort to server-side-preload the attachments from
the original message.  They appear as attachments in the forwarded copy
of the message in the composer, using the same API.  A lot of effort
went into making the 'attach' and 'forward attachments' functions DRY
with respect to each other.

serv_smtpclient.c: style cleanup

Mostly just made stuff look nice in 132 columns

upload.c: completed the code to reload attachments.

Now we have an API call to load attachments from existing messages
instead of from the browser.  This is the server side API.  Still need
to integrate it into the client.

view_mail.js: don't list attachments in a quote or forward

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