Re: [PATCH] SPF plugin: refactored, tests, new config option

+

> +=head2 reject
> +
> +Set to a value between 1 and 6 to enable the following SPF behaviors:
> +
> + 1 annotate-only, add Received-SPF header, no rejections.
>

Should this be reject 0?  It's not really reject.


> + 2 defer on DNS failures. Assure there's always a meaningful SPF header.
> + 3 rejected if SPF record says 'fail'
> + 4 stricter reject. Also rejects 'softfail'
> + 5 reject 'neutral'
> + 6 reject if no SPF records, or a syntax error
> +
>

Is it worth moving to english names instead of numbers?