search

emersion / go-msgauth

🔏 A Go library and tools for DKIM, DMARC and Authentication-Results
MIT License
168 stars 52 forks source link

dmarc, authres: Several improvments #11

Closed foxcpp closed 4 years ago

foxcpp commented 4 years ago

Most of these changes are independent but small. I decided to put them into a single PR so it will not look like spam.

codecov-io commented 4 years ago

Codecov Report

Merging #11 into master will decrease coverage by 0.79%. The diff coverage is 50%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master      #11     +/-   ##
=========================================
- Coverage   67.26%   66.46%   -0.8%     
=========================================
  Files           7        7             
  Lines         782      832     +50     
=========================================
+ Hits          526      553     +27     
- Misses        189      210     +21     
- Partials       67       69      +2
Impacted Files Coverage Δ
authres/parse.go 62.41% <29.16%> (-6.24%) :arrow_down:
authres/format.go 75.71% <67.85%> (-1.03%) :arrow_down:

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 26f5288...b1326ad. Read the comment docs.

foxcpp commented 4 years ago

Woops, it reason field should be always after methodspec, according to ABNF.