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

New feature: Reject at SPF None results #41

Closed
mikaku opened this issue Feb 8, 2018 · 7 comments
Closed

New feature: Reject at SPF None results #41

mikaku opened this issue Feb 8, 2018 · 7 comments

Comments

@mikaku
Copy link
Contributor

mikaku commented Feb 8, 2018

Hello,

Thanks for this beautiful software.

I'd like to be able to reject emails at SPF None results. I mean, I'd want to reject all mail from domains that do not have an SPF policy.

# Refuse e-Mail messages at SPF None results
#
# Default: off
#
RefuseNone      off      # (on|off)

Do you know if that could be an accepted new feature?
Thanks.

@jcbf
Copy link
Owner

jcbf commented Feb 8, 2018

Hi !

I'll look at it. This behaviour already crossed my mind.

@jcbf jcbf added this to To do in Release SPF 2.4.0 Feb 8, 2018
@jcbf
Copy link
Owner

jcbf commented Feb 8, 2018

btw: When receiving DSN's the mail from is empty and the HELO hostname is verified. Do you think this should be applied to HELO name that doesn't have SPF enable?

@jcbf jcbf added this to the v2.4.0 milestone Feb 8, 2018
@mikaku
Copy link
Contributor Author

mikaku commented Feb 8, 2018

Hmm, haven't thought on it before.

I'd say that yes, it should be applicable also in the HELO phase if the domain identified doesn't have an SPF record. Since, it will be rejected later for sure.

But in the other hand, I don't know if all mail servers identify in HELO with the same domain name that appears in the From field. In case that's not true we could reject legitimate email.

Difficult to say. :-/

@jcbf
Copy link
Owner

jcbf commented Feb 8, 2018

For now, I'll leave the HELO identity out. It' very rare servers have the HELO name with SPF configured.

I can create another config flag just for that use case. It can be a good feature on is own.

@mikaku
Copy link
Contributor Author

mikaku commented Feb 8, 2018

Agreed.

@jcbf
Copy link
Owner

jcbf commented Feb 8, 2018

Merging code and close issue

@jcbf jcbf closed this as completed Feb 8, 2018
Release SPF 2.4.0 automation moved this from To do to Completed Feb 8, 2018
@ghost ghost removed the in progress label Feb 8, 2018
@mikaku
Copy link
Contributor Author

mikaku commented Feb 8, 2018

Thank you very much for your fast response.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Release SPF 2.4.0
  
Completed
Development

No branches or pull requests

2 participants