Age | Commit message (Collapse) | Author |
|
Nextcloud can inject the userId but only for web processes. Now that
messages are sent also in the background and not just from a web request,
the user ID can be null and make the address collection log an
exception.
Following the pattern that user ID should only be injected into a
controller, the user ID is now passed as argument of the collector,
making it suitable for background jobs too.
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
|
|
Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
|
|
This does not show any measurable performance improvement but precents
the issue that the IMAP command can be too long for some IMAP servers
otherwise.
Ref https://github.com/nextcloud/mail/commit/e22742731adc1c1c03c394029948c7fa67e31b74
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
|
|
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
|
|
The message and the recipients are inserted in one transaction but the
recipients are read another time outside a transaction. Read-write split
database clusters might not be in full sync mode and then reading the
recipients gives partial or no results.
The insert will assign the primary key value to the recipient entities.
Therefore we can skip reading the data.
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
|
|
This code was used before the outbox. Now it's dead.
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
|
|
Groups were expanded in the accounts controller. Since moving over to
the outbox logic this feature was missing and internal group identifiers
were passed to SMTP.
With this patch groups are expanded again just before a message is sent.
This means the group memberships are read as late as possible and
editing an outbox message looks like the original message because
members have not been expanded there yet.
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
|
|
nextcloud/fix/take-sso-into-consideration-when-provisioning
Consider passwordless signins when provisioning accounts
|
|
Signed-off-by: Anna Larch <anna@nextcloud.com>
|
|
change columnst to bigint, create named foreign keys
Signed-off-by: Anna Larch <anna@nextcloud.com>
|
|
Ref https://symfony.com/doc/current/console/verbosity.html
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
|
|
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
|
|
Signed-off-by: Anna Larch <anna@nextcloud.com>
|
|
Signed-off-by: Richard Steinmetz <richard@steinmetz.cloud>
|
|
This reverts commit 7000208236c2c0421709115e9c2a7fc0509cef0d.
|
|
Signed-off-by: Anna Larch <anna@nextcloud.com>
|
|
Add local outbox
|
|
Signed-off-by: Anna Larch <anna@nextcloud.com>
|
|
Fix message account sync error if uidCandidates list is empty
|
|
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
|
|
Signed-off-by: pravussum <pravussum@users.noreply.github.com>
|
|
Signed-off-by: Joas Schilling <coding@schilljs.com>
|
|
Co-Authored-By: Christoph Wurst <christoph@winzerhof-wurst.at>
Signed-off-by: Anna Larch <anna@nextcloud.com>
|
|
Signed-off-by: Anna Larch <anna@nextcloud.com>
|
|
|
|
This changes the IMAP client usages to be used as a *resource* that is
freed after finished use. Previously we just memoized connections to the
same account to lower the number of connections, but that has still
shown to cause too many open connections during tests but possibly also
in production.
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
|
|
Co-authored-by: julia.kirschenheuter <julia.kirschenheuter@nextcloud.com>
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
|
|
|
|
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
|
|
* Trigger alert for number of recipients of a single message
* Trigger alert for number of messages per time period (15m, 1h, 1d)
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
|
|
Signed-off-by: Anna Larch <anna@nextcloud.com>
|
|
Signed-off-by: Richard Steinmetz <richard@steinmetz.cloud>
|
|
Signed-off-by: Richard Steinmetz <richard@steinmetz.cloud>
|
|
Signed-off-by: greta <gretadoci@gmail.com>
|
|
Signed-off-by: Anna Larch <anna@nextcloud.com>
|
|
Signed-off-by: Anna Larch <anna@nextcloud.com>
|
|
Add debug logs to the imap message mapper findByIds
|
|
This can show if unexpected results are fetched.
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
|
|
The initial sync is broken up into chunks. These chunks are estimated
because we only know the lowest and highest UID and there are likely
gaps in this range. One of the last chunks might query a range that has
more messages than the allowed chunk size. We slice it down. This means
the actually feched max UID is lower than the theorical max UID for this
chunk.
If the stop condition is to stop at the theoretical max UID then we
occasionally stop too early. Where we actually have to stop is the
actually fetched highest UID.
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
|
|
|
|
The provider is not injectible on Nextcloud 20. Therefore it needs some
error handling to factor in the possible absence.
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
|
|
Signed-off-by: Anna Larch <anna@nextcloud.com>
|
|
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
|
|
Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
|
|
Signed-off-by: Anna Larch <anna@nextcloud.com>
|
|
Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
|
|
Signed-off-by: Anna Larch <anna@nextcloud.com>
|
|
Signed-off-by: Anna Larch <anna@nextcloud.com>
|
|
Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
|
|
Signed-off-by: Anna Larch <anna@nextcloud.com>
|