Create a new table to store auth_results

solarissmoke / php-dmarc

A simple DMARC report parser written in PHP

Other

70 stars 21 forks source link

Create a new table to store auth_results #4

Closed jkrehm closed 8 years ago

jkrehm commented 8 years ago

Per our conversation on #3, I've created a new table to house the auth_results data and moved it off of rptrecord. At the same time I updated rptrecord.dkim_result and rptrecord.spf_result to use the info from policy_evaluated.

Take a look and let me know what you think. It does change the schema by removing the domains from rptrecord. If you want to leave those alone as technical debt, we could keep them there but not populate them.

solarissmoke commented 8 years ago

Thanks for this! Merged in ffa42db42fa02567fa86fbb9c8eae959b21c6156 . I debated whether to leave the two columns in the rptrecord table but decided against it and put a warning in the release notes.