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

Already on GitHub? Sign in to your account

New plugin: header_whitelist #3

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants

cventers commented Apr 7, 2011

This is meant to be used inside a qpsmtpd instance designed to clean up
outbound e-mail sent by noisy backend generators before it is relayed
through a real mail server to the Internet at large.

It is a useful tool but should be handled with care.

@cventers cventers New plugin: header_whitelist
This is meant to be used inside a qpsmtpd instance designed to clean up
outbound e-mail sent by noisy backend generators before it is relayed
through a real mail server to the Internet at large.

It is a useful tool but should be handled with care.
5de595e
Owner

msimerson commented Aug 5, 2013

This plugin has the potential to be quite useful. I'd like to see two changes though before I commit.

  1. Since you correctly say this plugin shouldn't be used on inbound messages, why not skip processing unless relayclient set? ($qp->connection->relay_client)
  2. Provide a name that makes is likely someone can understand what the plugin does based on the file name. In this case, the term whitelist is a very overloaded term for mail servers. The whitelist is merely the mechanism employed to strip/remove/sanitize unwanted headers from the message. I'd prefer a name like header_cleaner or header_sanitizer, or something to that effect.

@msimerson msimerson added a commit that referenced this pull request Sep 16, 2014

@msimerson msimerson Merge pull request #3 from flimzy/Base.pm
Kill some tab characters; PBP-suggested formatting for :? operator
d83268e
Owner

msimerson commented Jan 27, 2015

Abandoned.

@msimerson msimerson closed this Jan 27, 2015

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