Skip to content

refactor(postfix): remove unused "filter" lmtp service#1022

Merged
link2xt merged 1 commit into
mainfrom
link2xt/remove-filter
Jul 22, 2026
Merged

refactor(postfix): remove unused "filter" lmtp service#1022
link2xt merged 1 commit into
mainfrom
link2xt/remove-filter

Conversation

@link2xt

@link2xt link2xt commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

It looks like the reason it exists in the default configuration is an example for https://www.postfix.org/FILTER_README.html Using this filter requires configuration such as
"-o content_filter=filter:dummy" and we don't have it.

It looks like the reason it exists in the default configuration
is an example for <https://www.postfix.org/FILTER_README.html>
Using this filter requires configuration such as
"-o content_filter=filter:dummy" and we don't have it.
@link2xt
link2xt requested review from j-g00da and missytake July 22, 2026 20:04
@link2xt

link2xt commented Jul 22, 2026

Copy link
Copy Markdown
Contributor Author

Looks like if CI passes then it should be ok, but maybe someone wants to double-check that it is really not used.

The reason i looked at this: i was checking where do LMTP filters apply for #1005

@missytake missytake left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The process doesn't seem to be running on nine, probably because nothing calls it:

nine ~ # ps aux | grep filter | grep -v smtpd_proxy_filter | grep -v lmtp-filtermail
vmail     368216  0.5  0.5 1028460 373564 ?      Ssl  Jul02 167:10 /usr/local/bin/filtermail /usr/local/lib/chatmaild/chatmail.ini outgoing
vmail     368321  0.1  0.5 950328 352160 ?       Ssl  Jul02  54:41 /usr/local/bin/filtermail /usr/local/lib/chatmaild/chatmail.ini transport
root     1197503  0.0  0.0   6340  2208 pts/0    S+   22:26   0:00 grep --color=auto filter
vmail    4028937  0.0  0.1 993264 98748 ?        Ssl  Jul06  20:59 /usr/local/bin/filtermail /usr/local/lib/chatmaild/chatmail.ini incoming

@link2xt
link2xt merged commit 504fb24 into main Jul 22, 2026
8 checks passed
@link2xt
link2xt deleted the link2xt/remove-filter branch July 22, 2026 20:28
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