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

Is SMTP AUTH supported? #25

Closed
Gigadoc2 opened this issue Jan 27, 2018 · 3 comments
Closed

Is SMTP AUTH supported? #25

Gigadoc2 opened this issue Jan 27, 2018 · 3 comments

Comments

@Gigadoc2
Copy link

The readme makes it sound like the plugin supports displaying the results of smtp authentication in lieu of a DKIM signature. Is this actually the case?

My postfix generates an authres-header like this:
Authentication-Results: auth=pass smtp.auth=sendonly smtp.mailfrom=operations@five-ey.es, yet authres_status does not seem to recognize this as an authenticated mail.

@pimlie
Copy link
Owner

pimlie commented Jan 27, 2018

Hi, thanks for the report. This was a bug caused by a regression after adding the authservid check. As the smtp auth header does not list an authservid, it was always ignored. I have made the authservid check optional and added a test case for smtp auth headers. Please upgrade to v0.4.1 (please be aware that packagist on plugins.rc.net can take a while before picking up the new release)

@pimlie pimlie closed this as completed Jan 27, 2018
@Gigadoc2
Copy link
Author

Thanks for the clarification and quick response, it works like expected now! Although now I wonder if postfix should actually add the authserv-id to the header…

@pimlie
Copy link
Owner

pimlie commented Jan 27, 2018

Hmm, you might be on to something. Section 2.3 of RFC5451 specifies every Authentication-Results header has an authserv-id.

Could you check if this is a configuration issue or an issue with postfix itself? Eg if postfix is just not able to add the servauth-id maybe it would be better to add a configuration options to treat headers without authserv-id as 'trusted'. But maybe it would bebetter I will add that configuration option anyway, gives you some flexiblity at least.

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

No branches or pull requests

2 participants