Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[local] feat: add "null" SMTP transport mode #49294

Merged
4 commits merged into from
Nov 19, 2024

Conversation

artonge
Copy link
Contributor

@artonge artonge commented Nov 14, 2024

Local copy of #48977 for CI

@artonge artonge force-pushed the artonge/local/IONOS/tl/dev/disable-mailing branch 2 times, most recently from 7ce032f to 2e2edcf Compare November 18, 2024 17:39
Currently $this->instance is never set, so the code is no-op. This
brings back caching of the instance.

Caching broke with

   be7db15

   Swift to \Swift_Mailer as abstraction

Signed-off-by: Thomas Lehmann <t.lehmann@strato.de>
The IDE hinted the value is immediately overwritten.

Signed-off-by: Thomas Lehmann <t.lehmann@strato.de>
== Goal

Allow disabling mail delivery altogether.

== Usecase

If mails ought to be send by other means than rendering messages from
templates and sending them via SMTP-like protocols.

Example: listening to specific Nextcloud events and pass parameters to
a centralized (i.e. REST-based) API that sends e-mails.

Signed-off-by: Thomas Lehmann <t.lehmann@strato.de>
When the mail transport is configured as null transport, the
configuration UI would not work.

== Background

The null transport is meant for situations where operators
implement mail delivery via custom mechanisms like REST APIs.

Signed-off-by: Thomas Lehmann <t.lehmann@strato.de>
@artonge artonge force-pushed the artonge/local/IONOS/tl/dev/disable-mailing branch from 2e2edcf to 3f6c969 Compare November 19, 2024 11:24
@artonge artonge closed this pull request by merging all changes into master in 73f3b9b Nov 19, 2024
@artonge artonge deleted the artonge/local/IONOS/tl/dev/disable-mailing branch November 19, 2024 14:02
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.

2 participants