Skip to content

Added reject_sender_login_mismatch#819

Merged
johansmitsnl merged 1 commit intodocker-mailserver:masterfrom
TechnicLab:patch-1
Feb 6, 2018
Merged

Added reject_sender_login_mismatch#819
johansmitsnl merged 1 commit intodocker-mailserver:masterfrom
TechnicLab:patch-1

Conversation

@TechnicLab
Copy link
Copy Markdown
Contributor

Added reject_sender_login_mismatch parameter to prevent users from sending email using arbitrary email address. #811

@johansmitsnl johansmitsnl merged commit f68befd into docker-mailserver:master Feb 6, 2018
@TechnicLab TechnicLab deleted the patch-1 branch February 6, 2018 18:53
@17Halbe
Copy link
Copy Markdown
Contributor

17Halbe commented Feb 12, 2018

Hmm.. I might got this wrong, but don't you need a mapping file to make that work? Is this really denying a logged in user to spoof it's FROM address? I always had to implement a mapping file.

Which would probably look like:
main.cf:

...
smtpd_sender_login_maps = pcre:mapping.file
...

and the
mapping.file:

/^(.*)$/   ${1}

All from memory, but it took me quite a while back than to figure that out.. So it sticks! ;)
And yes, this would map every user to only his mailaccount.

@TechnicLab
Copy link
Copy Markdown
Contributor Author

TechnicLab commented Feb 12, 2018

Great idea! It is not working in my implemention, I actually created another pull request but there was a problem with requipment of mapping users. Could you fix this please?

@17Halbe
Copy link
Copy Markdown
Contributor

17Halbe commented Feb 12, 2018

I will fix it, if you implement a test for it! ;)

@TechnicLab
Copy link
Copy Markdown
Contributor Author

I am not familiar with tests that are used in this image, but I could try...

@TechnicLab
Copy link
Copy Markdown
Contributor Author

TechnicLab commented Feb 12, 2018

#825 - here is another pull request. Unfortunately working only with ldap.

@17Halbe
Copy link
Copy Markdown
Contributor

17Halbe commented Feb 12, 2018

Oh, great find! :) That would probably be replaceable with my idea.. I'll have a look
So let them do the work! ;)

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