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

Update data_post_headers doc #259

Merged
merged 2 commits into from
May 2, 2016
Merged

Update data_post_headers doc #259

merged 2 commits into from
May 2, 2016

Conversation

dani
Copy link
Contributor

@dani dani commented May 2, 2016

Clearly explain that data_post_headers is only to alter headers, and no reject can be done. Also add a hook in data_post for the dmarc plugin, to do the real reject.
Fix #258

Daniel Berteaud added 2 commits May 2, 2016 15:31
We cannot reject at this stage, which is only there to alter headers.
Fix #258
Followup of #258: we cannot reject a connection during data_post_headers. So add a new hook in data_post to do the real rejection
@coveralls
Copy link

coveralls commented May 2, 2016

Coverage Status

Coverage decreased (-0.04%) to 49.215% when pulling 6425f8a on dani:master into d13eae3 on smtpd:master.

@msimerson msimerson merged commit 9f3f9db into smtpd:master May 2, 2016
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.

4 participants