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

Replace from header function #112

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

trevor87
Copy link

I had the same problem as described in #75. It has bugged me quite a few times and I saw the work already done by jjakob in #96 and so I thought I'll give it a shot.

I have not coded C for a very long time, so I am not sure if this is the best way to do things. It works fine on my system now and I would be very happy if it could also help others. I'm happy to get feedback and would change the code as needed.

Basically this is the simplest form of fixing the error with cron etc.: If the REPLACEFROM setting is set in the config file (e.g. REPLACEFROM [email protected]) then the header-from will always be replaced with the mail address passed in the config (even if there is no header-from existent at all). The old header-from (e.g. "From: root") is not saved anywhere.

PS: Not sure if I made the changes right in dma.8, never worked with that before.

@trevor87
Copy link
Author

Note: Just noticed that this approach somehow does not work when sending an email via "mail root" when having aliases set up. When sending a mail through "mail [email protected]" it works. I will try to figure out what the problem is there.

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.

1 participant