Release version 943 generated by do-release.sh

List-Unsubscribe: header is now being generated and is correct!

more idle style-cleanup work

listsub.c: style cleanup

Extend the source_room field all the way into the SMTP delivery loop.
Tested adding SMTP headers directly into the outgoing message at the
moment of transmission -- it worked really well.  Temporarily adding
an "X-Citadel-Room:" header as a placeholder, but in the next couple
of commits we will make it a "List-Unsubscribe:" header.

When delivering mailing list messages, populate the sending_room
field of the recipient list.  This in turn populates the source_room
directive in the SMTP delivery list, which we will use soon.

Fucking hell.  We actually HAVE a "Allow non-subscribers to mail to
this room" flag.  I was confused by the poor wording of this option
in the text client, and screwed up the server's behavior when I
rewrote the mailing list server.  It's fixed now, and once again
has the correct behavior: anyone can email a room when the flag is
set, and subscribers can email the room regardless of the flag.

WebCit-NG:
* Force webcit to be on the same host as citserver
* Remove local keys directory, use the ones from citserver directory
* Auto re-bind key and cert if either one changes
* Support .well-known directory for static content, supporting HTTP-01

setup was still in the makefile , oooops

We have to push the changes *and* the tags

Release version 942 generated by do-release.sh

Citadel Server and WebCit (classic) now both reload the key and cert if the modification time of either one changes.  This should allow us to replace or renew the certificate during normal operation without restarting.

Not sure what the point of that second call to SSL_accept() was for, but it was always failing, and removing it seems to be fine

BIG CHANGES: PAY ATTENTION
1. WebCit must now run on the same host as Citadel Server, which is how everyone runs it anyway.
2. WebCit now uses the SSL key and Certificate from the Citadel Server directory.

Removed the unused 'selenium' directory

Removed the obsolete 'setup' utility.  It is not used by any currently supported installation method.
Removed the unfinished 'setup wizard'.
Removed the unused 'tests' directory.

Finally removed the cached/hashed static directory stuff.  This should put us into a position to get the http-01 challenge working.

Removed vestiges of the epic editor

Removed the epic editor from the tree.  Why was it even there?  It wasn't being used.

I need to make the static web server just a static web server.  I hate working in webcit classic because it's such a tangled mess.  The present commit just cleans up some old cruft; there is no actual change in functionality yet.  The next couple of commits will attempt to only cache the templates while using a regular file open for everything else.  There's no need to cache everything else because the operating system can do it better than we can.  KISS principle.  Coming soon to a webcit near you.

Cleanup of crypto.c in wc-classic to prepare for certificate updates during runtime

Removed more legacy cruft no longer needed when OpenSSL>=v1.1

OpenSSL no longer requires thread locking callbacks.  REMOVED FROM WEBCIT-NG

OpenSSL no longer requires thread locking callbacks.  REMOVED FROM WEBCIT CLASSIC

OpenSSL no longer requires thread locking callbacks.  REMOVED FROM CITSERVER

Removed SSL cleanup function.  Let the operating system do this for us.

serv_pop3.c: updated to latest style guidelines

New functions to activate and deactivate the Loading... modal

If div names are random strings, they don't need to start with "ctdl_"

Slowly becoming a good JavaScript developer.  :)    Don't pass around div names when we can actually pass around references to the divs themselves.  This should shave off a few cycles because we don't have to keep calling document.getElementById()

scroll_to works again

Append messages to the DOM tree as div elements instead of as text.  This preserves the document tree's integrity when a malformed message arrives.

Major change to the javascript forum view.  We now wait for all messages to be loaded before rendering them all in one shot.  Rendering works, opening the reply box works, saving a message works, but we still have to figure out how to render a message in the location where it was entered.  Or maybe that's not a good idea?  Also the scroll_to semantics are broken but we are in a better position to redo this in a simpler way.

open the reply box with .append() instead of tweaking outerHTML

new div name

beautiful

do-release.sh now generates more meaningful descriptions of tags

How about that ... we're now correctly handling the reply-references and carrying over the subject, and Wes is on the way here with a burger for me.

It's Friday night, Wesley is out, Sammi is not hungry, and Miss Melissa is asleep on the couch.  Instead of making dinner I added URL parameter parsing to the web server.  I didn't think we'd need it but I want to be able to transmit some parameters to ENT0 and this seems like a reasonable way to do it.

response headers now use an array instead of a linked list

http request headers are now an array instead of a linked list

HTTP headers are shown in purple in the syslog

Change all instances of "URI" to "URL" because that's more sensible

Begin gathering references and message IDs for replies

Don't use \r\n with ctdl_printf() because it adds a \n on its own.  Only add \r when CRLF is needed.  Also some UI tweaks.

Display an alert if we cannot connect to Citadel Server, instead of leaving the user sitting there bewildered.

When delivering mail from a mailing list room, it is not enough to set the Reply-To: header to the room's address; we must set the From: address too.  Doing otherwise annoys the recipient's DKIM validators.

Changed the reject message to be identical to the one used by Mailman so we can steal their translations later

Release version 941 generated by do-release.sh

Mail to a mailing list room must be from a subscriber (or a logged in user) otherwise incoming message is rejected.

Cleaned up another set of conditionals in the SMTP server.

Updated the comments in do-release.sh

Another win for short circuit evaluation

Made the RBL evaluation conditional cleaner by assuming short-circuit evaluation.

make a note that we have to retain the reply message number

Capture the new message number from the etag of the post transaction

Link entry in the editor now works.  The example code at https://thatsoftwaredude.com/content/8912/create-a-basic-text-editor-in-javascript was a bit naive because it assumes the URL is already known at the moment the user presses the Link button.  We open a box for URL entry, but when the user clicks into that box the original selection disappears.  So we have to save the selection range in hidden fields so we know where to replace the text.

bold, italic, and list buttons now work.

split utility function out into a separate util.js

Placed the bold/italic/list/link buttons in the editor.  (They don't work yet.)

Prettied up the qp encoder function a bit.

Post messages in quoted-printable instead of base64, just like WebCit Classic does.  This avoids double-base64-encoding of inline images.

Save messages as multipart/mixed with the HTML component in base64 encoding.  We probably should rethink this because now we're double-base64-encoding inline images.

Revert previous commit because it didn't work.  Furthermore, remove the content-length header from the PUT operation because the browser wants to do that on its own.

Permit the Content-Transfer-Encoding: header to be forwarded from an HTTP transaction to a post on a Citadel server.  This isn't typical for HTTP but we need it.

Moved the remaining else blocks

Handle posting errors a little more elegantly.

Detect whether the save operation succeeded by checking the response status of the XHR transaction.  Also made the default Citadel server dev.citadel.org instead of uncensored.citadel.org because I am posting a lot of test messages.

Wow!  Posting a message to the server worked on the first try, even using the existing DAV operation and no new C code.  That was a pleasant surprise.

Got it working, now it's pretty

content editable window with focus and cursor

Fixed the vertical float issue by specifying the height of the text div and letting the rest wrap around as it wants to.

how about that scroll?

Hot damn!  I just discovered scrollIntoView()

save/cancel

replybox adornments

 more work on the reply box

Intermediate commit: open a new div beneath the existing one when Reply or ReplyQuoted is selected

fujobi

Release version 940 generated by do-release.sh

Removed the remaining code where LDAP was optional.

Removed the 'old threaded view save for reference' code from the tree.  It's still in the git history if we need it, and JWZ is still a self-righteous asshole.

increased the margin around the message reading boxes

notes

Split the forum view into a separate .js file.  We will do this for all views.

nip and tuck

Reply/ReplyAll/Delete buttons are pretty now

The buttons now look like buttons, the reply button has a nice reply icon in it, and they change color when you hover over them.

When sending MSGS commands, unescape the command being sent to the server

Tinkering with the headers some more

Added CSS style for the Loading Messages, Older Posts, Newer Posts divs

Display the subject line -- only if present

Add subject to json output

Tuned up the CSS for message board view.  It looks pretty good now.

Use overflow:hidden style to keep the user avatar photo from breaking through the rounded corner of the container.

Changing signal handlers to the correct type eliminates the last of the warning messages

Suppress format-truncation warnings because they are WRONG

Poking around in the text client to determine whether we can remove dependency on libcitadel.

Finished converting all the comments in webcit-ng to C99 style

backslashes are fun