Skip to content

feat: Use filtermail for delivery to remote MTAs#916

Merged
j-g00da merged 3 commits intomainfrom
j-g00da/filtermail-transport-2
May 1, 2026
Merged

feat: Use filtermail for delivery to remote MTAs#916
j-g00da merged 3 commits intomainfrom
j-g00da/filtermail-transport-2

Conversation

@j-g00da
Copy link
Copy Markdown
Collaborator

@j-g00da j-g00da commented Apr 10, 2026

No description provided.

@j-g00da j-g00da temporarily deployed to staging-ipv4.testrun.org April 10, 2026 09:54 — with GitHub Actions Inactive
@j-g00da j-g00da temporarily deployed to staging2.testrun.org April 10, 2026 09:54 — with GitHub Actions Inactive
@j-g00da j-g00da changed the title wip wip: filtermail transport Apr 10, 2026
@j-g00da j-g00da force-pushed the j-g00da/filtermail-transport-2 branch from 17e3d30 to 0624533 Compare April 10, 2026 12:53
@j-g00da j-g00da temporarily deployed to staging-ipv4.testrun.org April 10, 2026 12:53 — with GitHub Actions Inactive
@j-g00da j-g00da temporarily deployed to staging2.testrun.org April 10, 2026 12:53 — with GitHub Actions Inactive
@j-g00da j-g00da force-pushed the j-g00da/filtermail-transport-2 branch from 0624533 to c1772af Compare April 13, 2026 17:40
@j-g00da j-g00da had a problem deploying to staging2.testrun.org April 13, 2026 17:40 — with GitHub Actions Error
@j-g00da j-g00da force-pushed the j-g00da/filtermail-transport-2 branch from c1772af to 516f616 Compare April 13, 2026 17:54
@j-g00da j-g00da temporarily deployed to staging2.testrun.org April 13, 2026 17:54 — with GitHub Actions Inactive
@j-g00da j-g00da temporarily deployed to staging-ipv4.testrun.org April 13, 2026 17:54 — with GitHub Actions Inactive
@j-g00da j-g00da force-pushed the j-g00da/filtermail-transport-2 branch from 516f616 to d6ce820 Compare April 15, 2026 18:53
@j-g00da j-g00da temporarily deployed to staging2.testrun.org April 15, 2026 18:53 — with GitHub Actions Inactive
@j-g00da j-g00da temporarily deployed to staging-ipv4.testrun.org April 15, 2026 18:53 — with GitHub Actions Inactive
@j-g00da j-g00da force-pushed the j-g00da/filtermail-transport-2 branch from d6ce820 to 4ee538e Compare April 15, 2026 19:58
@j-g00da j-g00da temporarily deployed to staging-ipv4.testrun.org April 15, 2026 19:58 — with GitHub Actions Inactive
@j-g00da j-g00da temporarily deployed to staging2.testrun.org April 15, 2026 19:58 — with GitHub Actions Inactive
@j-g00da j-g00da changed the title wip: filtermail transport feat: Use filtermail for delivery to remote MTAs Apr 15, 2026
@j-g00da j-g00da force-pushed the j-g00da/filtermail-transport-2 branch from 4ee538e to 87adc95 Compare April 16, 2026 08:02
@j-g00da j-g00da temporarily deployed to staging-ipv4.testrun.org April 16, 2026 08:02 — with GitHub Actions Inactive
@j-g00da j-g00da temporarily deployed to staging2.testrun.org April 16, 2026 08:02 — with GitHub Actions Inactive
@j-g00da j-g00da force-pushed the j-g00da/filtermail-transport-2 branch from 87adc95 to 54eea5a Compare April 16, 2026 08:11
@j-g00da j-g00da temporarily deployed to staging2.testrun.org April 16, 2026 08:11 — with GitHub Actions Inactive
@j-g00da j-g00da temporarily deployed to staging-ipv4.testrun.org April 16, 2026 08:11 — with GitHub Actions Inactive
@j-g00da j-g00da marked this pull request as ready for review April 16, 2026 08:11
Copy link
Copy Markdown
Contributor

@hpk42 hpk42 left a comment

Choose a reason for hiding this comment

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

lgtm except one nit on ltmp tls.

Comment thread cmdeploy/src/cmdeploy/postfix/main.cf.j2 Outdated
@hpk42
Copy link
Copy Markdown
Contributor

hpk42 commented Apr 16, 2026

would be good to do an update of https://chatmail.at/doc/relay/overview.html#chatmail-relay-dependency-diagram as well, maybe as part of this PR here.

@j-g00da
Copy link
Copy Markdown
Collaborator Author

j-g00da commented Apr 27, 2026

Marking as draft for now as we need to implement implicit MX: chatmail/filtermail#126

@j-g00da j-g00da marked this pull request as draft April 27, 2026 12:19
Copy link
Copy Markdown
Contributor

@hpk42 hpk42 left a comment

Choose a reason for hiding this comment

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

ok, then my request-changes comment is misplaced here. lost track a bit of the various inter-dependent PRs.

@hpk42
Copy link
Copy Markdown
Contributor

hpk42 commented Apr 28, 2026

but why is this PR still marked as "draft"? also rebase on master would be good.

@j-g00da
Copy link
Copy Markdown
Collaborator Author

j-g00da commented Apr 28, 2026

but why is this PR still marked as "draft"?

#916 (comment)

@hpk42 hpk42 force-pushed the j-g00da/filtermail-transport-2 branch from e3cc067 to 6065346 Compare April 29, 2026 18:29
@hpk42 hpk42 temporarily deployed to staging.chatmail.at/doc/relay/ April 29, 2026 18:29 — with GitHub Actions Inactive
@j-g00da j-g00da temporarily deployed to staging.chatmail.at/doc/relay/ May 1, 2026 04:46 — with GitHub Actions Inactive
@j-g00da j-g00da force-pushed the j-g00da/filtermail-transport-2 branch from c29f076 to 8c99573 Compare May 1, 2026 04:47
@j-g00da j-g00da temporarily deployed to staging.chatmail.at/doc/relay/ May 1, 2026 04:47 — with GitHub Actions Inactive
@j-g00da j-g00da force-pushed the j-g00da/filtermail-transport-2 branch from 8c99573 to 41f3974 Compare May 1, 2026 08:46
@j-g00da j-g00da temporarily deployed to staging.chatmail.at/doc/relay/ May 1, 2026 08:46 — with GitHub Actions Inactive
@j-g00da
Copy link
Copy Markdown
Collaborator Author

j-g00da commented May 1, 2026

Note: We decided to go with mxdeliv/https channel already in this PR. Bumped filtermail to 0.6.4 that includes it.

@j-g00da j-g00da force-pushed the j-g00da/filtermail-transport-2 branch from 41f3974 to a2222d6 Compare May 1, 2026 09:18
@j-g00da j-g00da temporarily deployed to staging.chatmail.at/doc/relay/ May 1, 2026 09:18 — with GitHub Actions Inactive
@j-g00da j-g00da force-pushed the j-g00da/filtermail-transport-2 branch from a2222d6 to 0896555 Compare May 1, 2026 09:36
@j-g00da j-g00da temporarily deployed to staging.chatmail.at/doc/relay/ May 1, 2026 09:36 — with GitHub Actions Inactive
@j-g00da j-g00da marked this pull request as ready for review May 1, 2026 09:59
@j-g00da
Copy link
Copy Markdown
Collaborator Author

j-g00da commented May 1, 2026

This needs ref swapped in the CI after chatmail/cmlxc#11 is merged and new version of cmlxc is released, but other than that is ready now.

@j-g00da j-g00da force-pushed the j-g00da/filtermail-transport-2 branch from 0896555 to 60f939f Compare May 1, 2026 15:18
@j-g00da j-g00da temporarily deployed to staging.chatmail.at/doc/relay/ May 1, 2026 15:18 — with GitHub Actions Inactive
j-g00da added 3 commits May 1, 2026 17:37
Signed-off-by: Jagoda Ślązak <jslazak@jslazak.com>
Signed-off-by: Jagoda Ślązak <jslazak@jslazak.com>
For compatibility with madmail,
we want to use path with no leading
slash. This change saves us from
having to follow redirects.

Signed-off-by: Jagoda Ślązak <jslazak@jslazak.com>
@j-g00da j-g00da force-pushed the j-g00da/filtermail-transport-2 branch from 60f939f to 44fe2dc Compare May 1, 2026 15:38
@j-g00da j-g00da merged commit 44fe2dc into main May 1, 2026
7 checks passed
@j-g00da j-g00da deleted the j-g00da/filtermail-transport-2 branch May 1, 2026 15:54
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