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

Build/releases #9

Open
wants to merge 3 commits into
base: build/releases
Choose a base branch
from

Conversation

montgomeryb
Copy link

@montgomeryb montgomeryb commented Mar 17, 2020

I wanted to use this project for limiting emails on postfix that are sent using postfix. Postfix has a plugin for mail sent by smtp but sendmail only has the milter protocol.

These files are the result of tweaking the project to run with postfix.

I had an issue running the program under the user postfix. I troubleshooted it to the fact that when I had the two programs looking at the same location, the socket file was still owned by root even though the process was running as postfix. This change changes the ownership of the socket file to that of the process in the milter-filter config file.

root added 3 commits March 17, 2020 17:58
Change the type to forking so that the service expects the called
process to exit and allow the program to continue running.
using postfix. milter-limit looks to this file to find details of the
domain socket.
Call auto_getconn to save the path, and call auto_setconn

Because this is still done as root, use the util function to change the
ownership of the domain socket file to that of the user that the process
is going to switch to.
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.

None yet

1 participant