Skip to content

misc: report the out-of-office subject prefix - #92

Open
r4nc0r wants to merge 1 commit into
grommunio:masterfrom
r4nc0r:feat/oof-subject-prefix-endpoint
Open

misc: report the out-of-office subject prefix#92
r4nc0r wants to merge 1 commit into
grommunio:masterfrom
r4nc0r:feat/oof-subject-prefix-endpoint

Conversation

@r4nc0r

@r4nc0r r4nc0r commented Aug 31, 2026

Copy link
Copy Markdown

Out-of-office replies carry the subject stored alongside the OOF message. Clients configuring OOF over EWS cannot store one, because SetUserOofSettings has no field for it, so gromox falls back to gromox.cfg:autoreply_subject_prefix plus the subject of the incoming message.

A client that offers an out-of-office form has no way to tell the user what an empty subject will actually send. Expose the directive so it can.

The directive is read on each request rather than cached at startup: it is wanted rarely, and an admin editing gromox.cfg should not have to restart the API. An unset directive is reported as configured=false rather than guessed at, because the default that then applies is compiled into the running gromox build and is not readable from here.

Out-of-office replies carry the subject stored alongside the OOF message.
Clients configuring OOF over EWS cannot store one, because
SetUserOofSettings has no field for it, so gromox falls back to
gromox.cfg:autoreply_subject_prefix plus the subject of the incoming
message.

A client that offers an out-of-office form has no way to tell the user
what an empty subject will actually send. Expose the directive so it can.

The directive is read on each request rather than cached at startup: it
is wanted rarely, and an admin editing gromox.cfg should not have to
restart the API. An unset directive is reported as configured=false
rather than guessed at, because the default that then applies is
compiled into the running gromox build and is not readable from here.
Comment thread endpoints/misc.py


@API.route(api.BaseRoute+"/oofSubjectPrefix", methods=["GET"])
@secure(requireAuth=False)

@StefanAkie StefanAkie Sep 7, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This would allow anyone to read any OOF message across the server, which might include sensitive information, violating Austria's DSGVO

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants