Commit Graph

354 Commits

Author SHA1 Message Date
Pascal Bleser
31b219f41c groupware: move prefixed writer into export_integration_test.go + upgrade integration tests to Stalwart 0.16.8 + update Stalwart configuration to include necessary default roles 2026-06-16 16:51:37 +02:00
Pascal Bleser
50f56de304 groupware: jmap: add a test for Invocation seriaization 2026-06-16 16:51:37 +02:00
Pascal Bleser
98144177f8 groupware: simplify JMAP integration tests with Stalwart 0.16
* don't run the Stalwart container in recovery mode first, just run it
   in "proper" mode, import the configuration and use it like that,
   which only requires skipping the "destroy" steps

 * add code documentation and logging

 * replace the text templating of the Stalwart snapshot by doing the
   same as the formatting script was doing, but directly in Go, removing
   an additional step to perform when modifying a Stalwart
   configuration to use in the integration tests
2026-06-16 16:51:37 +02:00
Pascal Bleser
8690f7c6cb groupware: port tests to Stalwart 0.16.7 2026-06-16 16:51:37 +02:00
Pascal Bleser
92fbdda1ec groupware: fix unit tests 2026-06-16 16:51:37 +02:00
Pascal Bleser
7d4bce2307 groupware: add tracking of backend call durations
* add new configuration setting GROUPWARE_SEND_DURATIONS_RESPONSE
   (defaults to false)

 * keep track of lists of durations of backend calls

 * when enabled, report them as response headers Durations (human
   readable) and Durations-Nanos (as raw nanosecond values for machine
   consumption)
2026-06-16 16:51:37 +02:00
Pascal Bleser
7e8caab979 groupware: add strongly typed aliases for AccountId, PrincipalId and SupplierId
Purpose is to make APIs and parameters easier to understand, since plain
strings are used all over the place for all sorts of identifiers.
2026-06-16 16:51:37 +02:00
Pascal Bleser
52d905f62d groupware: remove unneeded jmap.QueryCommand methods
* remove the following methods on the jmap.QueryCommand interface, as
   they ended up not being needed:
   - GetPosition()
   - GetAnchor()
   - GetAnchorOffset()
   - GetLimit()
2026-06-16 16:51:37 +02:00
Pascal Bleser
3e8c37a13b groupware: refactoring for pagination and support for multiple query suppliers
* refactor APIs in JMAP and Groupware in order to implement pagination
   across multiple accountIds and multiple suppliers (currently
   implemented using a mock supplier for contacts)

 * requires go 1.26 due to use of self-reflecting generics type
   constraints

 * still missing: query criteria and sorting parameters

 * still missing: multi-accountId support for emails

 * errors are now all just 'error' in the APIs, instead of the
   specialized implementations, and are interpreted dynamically where
   necessary in order to transform them into HTTP responses

 * remove position, anchor, anchorOffset as individual query parameters
   as we now only support a 'next=...' token for subsequent pages
   (except in emails for now), and use jmap.QueryParams instead; those
   tokens have a header character for the format, followed by a JSON
   encoded QueryParams map, all wrapped into base62 to make it clearer
   that it is meant to be an opaque token, and not a parameter clients
   should tinker with or construct themselves

 * introduce QueryParamsSupplier as an interface to provide QueryParams
   for various scenarios (single supplier, multiple supplier, ...) per
   accountId

 * implement multi-supplier template methods slist and squery
2026-06-16 16:51:37 +02:00
Pascal Bleser
0e6fb14c0a groupware: API documentation updates 2026-06-16 16:51:37 +02:00
Pascal Bleser
92aa5fca67 groupware: refactor responses to a jmap.Response object
* in the JMAP API as well as in several places in the Groupware
   framework, use a single jmap.Response[T] object to return the
   payload, the language, the session state and the etag/state instead
   of individual multi-valued return values
2026-06-16 16:51:37 +02:00
Pascal Bleser
0847232baf groupware: fix test logic error in the JMAP ws integration test 2026-06-16 16:51:37 +02:00
Pascal Bleser
e772f46c97 groupware: omit "position" when "anchor" is specified in queries
* the "position" attribute is always set to 0 when using an anchor for
   pagination, but it does not accurately reflect the position and thus
   we decided to remove the attribute from the resulting object for
   clarity

 * omit "canCalculateChanges" attribute when its value is "true", which
   is going to be the case in 100% of calls against Stalwart
2026-06-16 16:51:37 +02:00
Pascal Bleser
ffc0e1f42a groupware: add support for anchor and anchor offset pagination
* add query parameters 'anchor' and 'offset':
   - anchor is an object identifier
   - offset is a numeric offset relative to the anchor
2026-06-16 16:51:37 +02:00
Pascal Bleser
ce9b4d59bc groupware: fix use of ?limit=0
* JMAP query limit of 0 is synonymous with "no limit", but we actually
   want to be able to perform queries without any results, for cases
   where we only want to count the total number of objects, and also
   because it makes more sense semantically

 * introduce query parameter validation checks, in order to only allow
   query parameters that are actually supported, which is going to be
   useful during development of clients
2026-06-16 16:51:37 +02:00
Pascal Bleser
ae0ff8e524 groupware: add documentation for SearchResultsTemplate + update pnpm 2026-06-16 16:51:37 +02:00
Pascal Bleser
92b85b215a groupware: rename 'offset' to 'position' to be consistent with the JMAP specification 2026-06-16 16:51:37 +02:00
Pascal Bleser
e0068c550b groupware: fixes after rebasing and Docker library API changes 2026-06-16 16:51:37 +02:00
Pascal Bleser
c710336b22 groupware: documentation changes after API generator updates for the use of template functions 2026-06-16 16:51:37 +02:00
Pascal Bleser
a115679f3d groupware: refactoring using function templates
* adds creating addressbooks, calendars, mailboxes

 * adds deleting mailbox, event, identity

 * adds modifying an email

 * introduce template functions for the Groupware API in templates.go,
   and use those in route function implementations whenever possible

 * add capability checking for mail, quota, blobs

 * adds Changes interface

 * adds JmapResponse interface
2026-06-16 16:51:37 +02:00
Pascal Bleser
38f7e24947 structs: add tests and documentation 2026-06-16 16:51:37 +02:00
Pascal Bleser
56e11c3ee4 groupware: improve tests
* jmap: add UpdateContactCard and UpdateCalendarEvent funcs
 * use JSON marshalling and unmarshalling into a map for toPatch()
   implementations
 * add updating ContactCards and CalendarEvents to tests
 * add deleting ContactCards and CalendarEvents to tests
 * make query response totals work when the value is 0
 * tests: use CreateContactCard and CreateCalendarEvent funcs to create
   objects in tests instead of using a different JMAP stack that works
   with untyped maps
2026-06-16 16:51:37 +02:00
Pascal Bleser
2023cdf444 groupware: refactoring: pass object type instead of namespaces
* make the JMAP internal API a bit more future-proof by passing
   ObjectType objects instead of the JMAP namespaces

 * remove the new attempt to contain operations even further using the
   Factory objects

 * move CalendarEvent operations to its own file, like everything else

 * fix email tests

 * ignore WS error when closing an already closed connection
2026-06-16 16:51:37 +02:00
Pascal Bleser
83e9953df6 groupware: refactor for conciseness
* introduce jmap.Context to hold multiple parameters and shorten
   function calls

 * introduce SearchResultsTemplate
2026-06-16 16:51:37 +02:00
Pascal Bleser
6e2eea65b2 groupware: model refactoring, introducing typed interfaces and Foo
* move ContactCard from jscontact to jmap, as it is actually a JMAP
   specification item, but also was causing too many issues with
   circular references from jscontact -> jmap

 * introduce Foo, Idable, GetRequest, GetResponse, etc... types and
   generics

 * first attempt at a Foo factory type for Mailboxes, needs to be
   expanded to further minimize repetition

 * add more specialized template functions to avoid repetition

 * introduce ChangesTemplate[T] for *Changes structs
2026-06-16 16:51:37 +02:00
Pascal Bleser
b77cbe3dbc groupware: significant refactorings of the JMAP framework, adding methods and more intelligence to the various request and response types to improve the use of template functions, reducing the risks of typos and copy/paste mistakes 2026-06-16 16:51:37 +02:00
Pascal Bleser
89fee977c8 groupware: add addressbook and calendar creation APIs
* add Groupware APIs for creating and deleting addressbooks

 * add Groupware APIs for creating and deleting calendars

 * add JMAP APIs for creating and deleting addressbooks, calendars

 * add JMAP APIs to retrieve Principals

 * fix API tagging

 * move addressbook JMAP APIs into its own file

 * move addressbook Groupware APIs into its own file
2026-06-16 16:51:37 +02:00
Pascal Bleser
6ef5988f30 groupware: add changes support for quotas, identities, submissions 2026-06-16 16:51:37 +02:00
Pascal Bleser
a49b0e239e groupware: framework refactorings + add support for /changes + add /objects 2026-06-16 16:51:37 +02:00
Pascal Bleser
4a2f9463e4 groupware: pollute code with NOSONAR comments, and make a little more use of JMAP API templates 2026-06-16 16:51:37 +02:00
Pascal Bleser
c87cbf0291 groupware: refactor contactcard changes, and Request framework
* implement ContactCard retrieval endpoint for syncing

 * re-implement that endpoint for Email too

 * fix the Mailbox changes endpoint to actually return changes about
   Mailboxes, and not about Emails

 * when querying the diff of Mailboxes without any prior state, return
   an error since the result is not what one would expect

 * introduce the 'changes' API tag and group

 * refactor the successful response functions to consistently return an
   object type and object state whenever possible

 * move the syncing endpoints under /accounts/*/changes/ for better
   clarity, e.g. /changes/emails instead of /emails/mailbox/*/changes
2026-06-16 16:51:37 +02:00
Pascal Bleser
f5660fd0a6 groupware: documentation: more calendar examples 2026-06-16 16:51:37 +02:00
Pascal Bleser
0bed356966 groupware: fix typos and minor issues
- fix a bunch of minor issues and typos that were found using GoLand
   and gosec

 - add a gosec Makefile target for Groupware related files, in
   services/groupware/Makefile

 - enable checking JMAP session capabilities for events and contacts,
   and only enable skipping that check for tasks until those are
   implemented in Stalwart as well

 - fix a CWE-190 (integer overflow or wraparound) found by gosec

 - consistently use struct references for methods of Groupware and
   Request, instead of mixing up references and copies

 - always log errors when unable to register a Prometheus metric
2026-06-16 16:51:37 +02:00
Pascal Bleser
a1ac3ebcf3 groupware: upgrade Stalwart image in integration tests from 0.15.0 to 0.15.5 2026-06-16 16:51:36 +02:00
Pascal Bleser
d1aa3025d8 groupware: add UI example of a ContactCard to the documented examples 2026-06-16 16:51:36 +02:00
Pascal Bleser
6b8f7207d6 groupware: add OIDC authentication support between Groupware backend and Stalwart
* re-implement the auth-api service to authenticate Reva tokens
   following the OIDC Userinfo endpoint specification

 * pass the context where necessary and add an authenticator interface
   to the JMAP HTTP driver, in order to select between master
   authentication (which is used when GROUPWARE_JMAP_MASTER_USERNAME and
   GROUPWARE_JMAP_MASTER_PASSWORD are both set) and OIDC token
   forwarding through bearer auth

 * add Stalwart directory configuration "idmoidc" which uses the
   OpenCloud auth-api service API (/auth/) to validate the token it
   received as bearer auth from the Groupware backend's JMAP client,
   using it as an OIDC Userinfo endpoint

 * implement optional additional shared secret to secure the Userinfo
   service, as an additional path parameter
2026-06-16 16:51:35 +02:00
Pascal Bleser
8244e4b22f groupware: remove remnants of go-swagger documentation, since we are switching to using our own tool instead 2026-06-16 16:50:52 +02:00
Pascal Bleser
984a15f379 groupware: add jscontact examples, and move them to jmap/model_examples.go to avoid dependency recursion 2026-06-16 16:50:52 +02:00
Pascal Bleser
124ed20e13 groupware: as suggested in the review PR 1994, remove all filename prefixes that match the package (groupware_xyz.go -> xyz.go), for groupware, jmap, jscontact, jscalendar 2026-06-16 16:50:52 +02:00
Pascal Bleser
6fdc4a830d groupware: convert documentation to proper markdown 2026-06-16 16:50:52 +02:00
Pascal Bleser
48b00db7a7 groupware: more API examples 2026-06-16 16:50:52 +02:00
Pascal Bleser
ef7b1d4a72 API documentation changes for groupware-apidocs
* add example generator infrastructure, with some examples for pkg/jmap
   and pkg/groupware, with more needing to be done

 * alter the apidoc Makefile to stop using go-swagger but, instead, use
   the openapi.yml file that must be dropped into that directory using
   groupware-apidocs (will improve the integration there later)

 * add Makefile target to generate examples

 * bump redocly from 2.4.0 to 2.14.5

 * introduce Request.PathParam() and .PathParamDoc() to improve API
   documentation, as well as future-proofing

 * improve X-Request-ID and Trace-Id header handling in the middleware
   by logging it safely when an error occurs in the middleware
2026-06-16 16:50:11 +02:00
Pascal Bleser
7ee2cc2e91 groupware: update to Stalwart 0.15.0 2026-06-16 16:50:11 +02:00
Pascal Bleser
b67e3b4cc4 groupware: fix failing pkg/jscontact unit tests 2026-06-16 16:50:11 +02:00
Pascal Bleser
28c0c3d368 groupware: fix failing pkg/jscalendar unit tests 2026-06-16 16:50:11 +02:00
Pascal Bleser
c5e67d7d6f groupware: improve email submission and testing
* jmap/EmailCreate: add more attributes that were omitted: Headers,
   InReplyTo, References, Sender

 * add jmap GetEmailSubmissionStatus

 * improve email integration tests by adding a thorough test for email
   submission

 * jmap integration tests: provision principals and domains using the
   Stalwart Management API, switching from an in-memory to an internal
   directory
2026-06-16 16:50:11 +02:00
Pascal Bleser
531e913ca4 groupware: add retrieving and adding mailboxIds for drafts and sent if they are missing 2026-06-16 16:50:11 +02:00
Pascal Bleser
f45d2d5a70 groupware: WS push improvements, add getting email changes to WS integration test 2026-06-16 16:50:11 +02:00
Pascal Bleser
135bc066ae groupware: JMAP WS push notifications support 2026-06-16 16:50:11 +02:00
Pascal Bleser
66b86af3c7 groupware: fix email summaries and allow negative offsets
* fix a bug in how email summaries are flattened across multiple
   accounts, which was previous resulting in empty email objects

 * allow negative offset in email pagination

 * make all /emails endpoints return emails without bodies
2026-06-16 16:50:11 +02:00