Different subtypes of SPF Include Loop are treated differently

[thesyst]

There is an inconsistency in how the two types of SPF Include Loop are handled right now.

• if you have domain.com with a record for that says "v=spf1 include:domain.com" this will raise a SPFIncludeLoop exception
• if you have domain1.com with a record "v=spf1 include:domain2.com" and domain2.com with a record "v=spf1 include:domain1.com" this will not raise any Exceptions but will create a warning: "Duplicate include"

In my opinion, these two types of loops are equally bad and should be handled in a single way. Both types should result in an exception.