dammit, learn to spell

Many years ago, someone inserted an enum called eMesageText into the code
and used it all over the place without correcting the spelling.  This
commit finally fixes it.

serv_calendar.c: basically reformatting for modern standards

When I wrote this stuff I had a 109 column screen.  Now our standard
is 132 columns.  Also we are seeking and destroying any place where
spaces were used as indentation.  Our code is indented with tabs, the
way God intended.

serv_calendar.c: basically reformatting for modern standards

When I wrote this stuff I had a 109 column screen.  Now our standard
is 132 columns.  Also we are seeking and destroying any place where
spaces were used as indentation.  Our code is indented with tabs, the
way God intended.

End support for libical <2.0

Wondering how we ever got by with narrow terminals

clarify terms in license

dumploadtest.sh: added an exit 0 to the top of the script.

This is a testing script intended for developers.  The last thing
we need is for some dumbass to run it without knowing what it does,
and then come to us complaining that it deleted their database.

In case you haven't figured it out yet...

This is what I do when I'm avoiding some other task.

Grammar change in the license declaration.

Many of the files say "use, duplication, or disclosure ARE subject
to..." but the common phrasing of this statement is apparently "use,
duplication, or disclosure IS subject to..."

We are already taking great delight in the fact that we are using the
GPL without using Stallman's preferred declaration of it.  And of course
we call it open source software.

When Steve Jobs died, Richard Stallman said "I'm not glad he's dead,
but I'm glad he's gone."  I agreed with him.  When Richard Stallman dies,
I will say "I'm not glad he's dead, but I'm glad he's gone."  But
when Bill Gates dies, I will be glad he's dead.  I will celebrate.

style cleanup

More C99-style comments.

The sooner we can put webcit-classic out to pasture the better.

Between its origin as a shitty web application that I built when I had
no idea what the hell I was doing, to all of the ways other people fucked
it up later, this thing needs to die.  As soon as possible.

serv_func.c: simplify IDEN command

Remove parameters no longer used

Utilities displaying a color banner now use line drawing characters to form a box instead of clearing to the end of the line with the background color.  The latter does not have the desired effect on all terminals.

context.c: last_purge is local to dead_session_purge()

Changes to server session timeouts.

CLIENT_TIMEOUT was found in the header files but not used anywhere.  Removed.

dont_term was removed because it was not needed anywhere anymore.

It is the current year.  (c)-2024

Remove sock_close and sock_shutdown 'portability' defs.

Removed the dev_code, cli_code, and rev_level fields from the server session.

serv_session.c: style update

Starting to bring over calendar server code from webcit-classic.

view_calendar.js: finalized a fetch design pattern

This uses "fetch" followed by ".then" etc. etc. etc.
and is tuned to be a design pattern for future code

calendar_view.c (from webcit-classic) : clean up

Style and comments cleanup.  Some of this code is going to get
copied over to WebCit-NG so we need to bring it up to our current
coding standards (K&R, 1TBF, with C99-style comments)

Worked out the syntax for requesting a calendar range

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