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

Allow Received-SPF header back. #32

Merged
merged 1 commit into from
Sep 11, 2017
Merged

Allow Received-SPF header back. #32

merged 1 commit into from
Sep 11, 2017

Conversation

jcbf
Copy link
Owner

@jcbf jcbf commented Apr 6, 2017

By request in #31 .
Also spamassassin check if this header is present and reuses the evaluation.

@jcbf jcbf self-assigned this Apr 6, 2017
@codecov-io
Copy link

codecov-io commented Apr 6, 2017

Codecov Report

Merging #32 into master will decrease coverage by 1.66%.
The diff coverage is 7.14%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #32      +/-   ##
==========================================
- Coverage   63.47%   61.81%   -1.67%     
==========================================
  Files           1        1              
  Lines         460      474      +14     
  Branches      198      204       +6     
==========================================
+ Hits          292      293       +1     
- Misses         84       95      +11     
- Partials       84       86       +2
Impacted Files Coverage Δ
smf-spf.c 61.81% <7.14%> (-1.67%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 451d113...7494cec. Read the comment docs.

@coveralls
Copy link

coveralls commented Apr 6, 2017

Coverage Status

Coverage decreased (-1.8%) to 80.335% when pulling 7494cec on jcbf/received_spf_header into 451d113 on master.

@jcbf jcbf merged commit 40dd600 into master Sep 11, 2017
@jcbf jcbf deleted the jcbf/received_spf_header branch April 26, 2018 18:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants