some work on calendar

Calendar view: initial call of REPORT method.

Right now it's just a placeholder that outputs the raw XML to the screen.

Use the allprop tag

more noodling on backend detection

Test for existing BDB files (not used yet)

Release version 999 generated by do-release.sh

Fixed parsing of message-id angle brackets

Began adding support for <prop> stanzas.

Everyone who designed CalDAV deserves torture and ultra-violent death.

moved whitespace around

serv_notes.c: cleanup

internet_addressing.c: minor cleanups

validate_recipients() - completed removal of unused param

REPORT namespace output uses declaration defined on top

validate_recipients() remove extra copy of recipients

We no longer need to copy recipients_in to recipients, because we no longer
destructively tokenize it.  Now we non-destructively tokenize it into an array.

Also I discovered that there is an unused parameter in that call.  Removed it
and the one place where it was being non-functionally called.

Handle start and end times together

make it pretty

Completed time range search of recurring events.

xmpp_sasl_service.c: style cleanup

Made this code more brief

Took ical_dezonify.c back out of WebCit-NG

Defined 19010101T010101Z-99991231T235959Z consts.

These are now known as "the_beginning_of_time" and
"the_end_of_time" and closely track the time boundaries within
which libical can safely operate.

Supplied these as default values when filling variables to call the
time range compare function, allowing us to remove the check-for-null
code in that function.

ical_ctdl_is_overlap.c: cleaned it up

Do not convert compared times to UTC.

ical_ctdl_is_overlap() works because icaltime_compare() is really smart.
It looks at the time zone of the dtstart/dtend and can apparently go back up the icalcomponent
hierarchy to find its time zone data.  I tested this by creating an event with a fictional
time zone and it did the right thing.  It even showed the fictional name to me.  This saves us
from having to convert everything to UTC before comparing.  Nice!

Compare after convert works.  Saving.

WARNING NOT Y10K COMPLIANT

Set min (00010101T010101Z) and max (99991231T235959Z) dates to use when
min or max timestamp is not supplied for comparison

Changed "supplied_xxx" to "xxx_in" when we modify.

ical_ctdl_is_overlap() is now symlinked 3 places.

It's the same function used in Citadel Server, WebCit-Classic, and
WebCit-NG.  It isn't really suitable for use in libcitadel so we
made symlinks instead.

Bring ical_dezonify() into caldav_reports.c

Make a local copy of events being compared, dezonify them.
Now gotta write the rest.

In webcit-classic, include webserver.h from webcit.h.

This makes it easier to port ical_dezonify.c between webcit generations.

misc style cleanup

Only calculate ops/sec if time>0

Made a note to fix a divide-by-zero problem.

(ops_completed / (time(NULL) - time_started)

I thought my new machine had a problem but it turned out to just be
running so fast that "time(NULL) - time_started" really was zero.

Moved default_zone_name declaration to webcit.h

cal_zone means "calendar timezone" also "calzone"

ical_dezonify() is going to be needed in both WebCit Classic and
WebCit-NG , so this needs to be cleaned up and generalized.

Smuggled classified secrets into this repository.

Skeleton code for time-range test (section 9.9)

After finally getting to the point where we are recursing up
through both the nested components and the nested queries in a
way that matches them properly, we are now able to start writing
the individual property and parameter queries defined in the RFC.

time-range (section 9.9) is the obvious first choice to implement
because it is the most useful.  Wrote and bound the skeleton
function to do this.

I hope letitia james and arthur engoron get run over by a truck

Count nesting level only for comp-filters

added more trace messages

Refining XML rules

XML is like violence: if it doesn't work, you're not using enough.

Updated the text client command set help

Completed the first two filter tests.

comp-filter on the root element are the first two tests.  Yes, these
are ridiculously easy, but hopefully it gets the design pattern started
and we can move on from there.

Made the log easier to read

Try new strategy for CalDAV report filter parsing.

The recursive strategy I initially built looked clever but it's
not going to work because we need to iterate through both the
rules and the components/properties/parameters at the same time.

Reworked the code to go through filters iteratively instead of
recursively.  This protocol is way too complex and the people who
designed it need to be tortured.

I wish much death and suffering to Cyrus Daboo, Lisa M. Dusseault, and Bernard Desruisseaux.  They must be punished severely for creating CalDAV.

Shorthand for the XML namespace

Skeleton code for filters.

All of the filters specified in RFC4971 9.7.1 through 9.7.5 will
be handled in this loop: comp-filter, prop-filter, param-filter,
is-not-defined, text-match, and probably others as we complete the
spec.

Also, in all of the .c files I unwrapped the license declaration onto a single
line, since the whole thing fits inside 132 characters, and 132 characters is
our official line length now.

Completed framework for caldav_apply_one_filter()

Now we have an array of tokens containing the filter name and all attributes.

Filters defined in RFC4791 9.7.1-9.7.5: DRY ingest

The code to parse these XML stanzas and ingest them into our filter
list was getting quite redundant.  This commit abstracts them into
a single handler.

Yet another attempted framework for caldav filters.

CalDAV is stupid, the people who created it are stupid, and anyone who
still thinks it was a good idea are stupid.  But now we're stuck with it
and I have to implement it.

Final changes to XML parser trace logging.

This is extremely inefficient code, but it will be ifdef'ed out
during normal use.  Like the rest of the protocol trace options,
they are only getting ifdef'ed into use while I am developing.

more logging and chardata stuff

better logging presentation of chardata

Improved logging of XML parse

Solved P=NP

Unified multistatus output for all REPORT outputs.

"multiget" and "query" REPORTs now use the same code path for
outputing results in DAV "multi-status" format, keeping this logic
DRY.

"query" reports are currently operating through a vile sleazy hack
that makes it output every item in the calendar without applying
any filters.  That's the next piece to work on.

Moved multiget output into its own function.

This will allow the output code to remain DRY when it is called
by several different REPORT types.

ctdldump/ctdlload: add version number to banner

Added context state to output of RWHO command.

Hopefully the next time some douchebag from out of town floods our
site with stuck-open MSA sessions, we can figure out why it isn't
unbinding or timing out.

Release version 998 generated by do-release.sh

add a comment to the previous thing

Awesome sauce, read description below for details.

My 'ctdlload' operation kept failing because there were a few user
records dumped that had insane data in them.  CtdlDecodeBase64() was
silently doing buffer overruns because it doesn't know the size of
the target buffer.

While noodling a workaround, I realized that you can pass the same
pointer to CtdlDecodeBase64() as both the source *and* target buffer.
This works because the decoded data pointer will ALWAYS be behind the
encoded data pointer.  This allows a "decode-in-place" operation!

By performing a decode-in-place and then using safestrncpy() to copy
the results to the finitely-sized string buffer in the target struct,
we guarantee no buffer overruns.  Valgrind no longer bitches about it
and the program does not crash.

Giving up on doing citserver-side REPORT parsing.

webcit and citserver are guaranteed to be on the same host now, and
performance over a unix domain socket is likely to approach that of
doing it the "hard" way anyway.  So I pulled out all the crap I was
adding.  It was making the data model very ugly.

enabling http debugging because here comes caldav ugh

"To hell with cloud. Run Citadel." --Taylor Swift

CtdlCheckExpress() now accepts a session context.

This is an experiment in reducing the use of the "CC" macro
that appears everywhere in the server.  Willi used to do a lot
of "struct CitContext *CCC = CC" but that still called the macro
once per function call and it added complexity to the code.  This
method is the right way, actually passing the session context up
the stack like we do in WebCit-NG.

Release version 997 generated by do-release.sh

Indent with tabs, as God intended.

databaselayout.md: added code blocks

binaries.txt: converted to markdown

Codified plans for world domination

Changed the BIGMSGS semantics.

We now identify the memory location of the start of message body
during serialization, so we don't have to measure it twice.  Then
we store the body in the BIGMSGS table only if the total message
size would exceed 4000 bytes.  This assumes that memory and disk
pages are 4096 bytes, so we want to optimize for a single I/O
operation in most cases.

loadtest: default threads now 10

loadtest: added ops/sec meter

Assert that eMessageText MUST be serialized last.

Also assert that the calculated length of the serialized message
is actually the final length.  Either of these conditions not
being true is worth crashing the server so the developer who broke
it knows immediately.

CtdlSerializeMessage() now returns struct smr_ret

Instead of giving it a pointer to a struct to fill, now it just
returns a struct.  Because it is the current year and the compiler
can handle that.

Rearranging code to prepare for multiple REPORT types

remove leading/trailing whitespace from hrefs

This was found in the CALDAV 'calendar-multiget' REPORT and has been fixed

Noodling on calendar query commands.

The server command 'ICAL report' is now 'ICAL query' with the intent that
it will handle CalDAV 'calendar-query' REPORTs, but not other REPORTs.

Likewise, functions with names including the string 'calendar_report' have
been changed to 'calendar_query'.

Made our cproto output even more beautiful

Began refactoring CALDAV REPORT flow of control.

Removed calendar_functions.c and brought some of its code over
to caldav_reports.c so that we can use the same code path for all
three types of REPORTs.

This is motivated by an idea that perhaps the user interface can
use CalDAV as its REST API instead of inventing something new.  If
that doesn't work, we needed to finish CalDAV eventually anyway.

Don't bother reading this commit.

Blockquotes are now rendered to console as dim instead of italic

New config option: set smtp_advertise_starttls to nonzero to advertise STARTTLS in ESMTP

updated description

move calendar prototypes around, add more with cproto

Experimenting with the code layout for calendar report handling.

The goal is to provide calendar searches (date range etc) for both WebCit-NG
and for CALDAV REPORT method in a DRY way.

Don't log each table open/close, just the whole operation

START_CHAT_MODE is renamed to SEND_THEN_RECV

This protocol mode began almost 30 years ago when we handled real time chat by
holding a session open and moving data back and forth until told to stop.  We
don't do that anymore, but there are several places where a command requires
sending long data in both directions.  SEND_THEN_RECV is a better name for this.

The damn thing finished.  Let's see what it did.

Still waiting  for the computer.  Or rather, for other computers.

Still waiting...

This time I'm waiting for the computer.

Can you tell I'm REALLY avoiding another task right now?

The little LEDs on the NanoPi blink when there is activity.

It's fun to watch.  It's fun to build on ARM first and try x86 later.

That about wraps it up for serv_calendar.c

Seeing this code has really brought me back to the upstairs room
in my old house where I wrote most of it back in 2002-2003.  I can
see the sunlight coming in the windows, the color of the carpet,
the little alcove where the computer was, the cat sleeping on top
of the big CRT monitor.

Hopefully the stuff I need to add to it now will be straightforward
since it looks like I really took my time and commented it heavily.
It looks pretty clean, especially now that I've reformatted it with
the new screen width an C99 comments.  It does seem a bit WET though.

Continuing the war against C89-style comments

serv_calendar.c: more formatting changes

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.