shevek / libspf2

Implementation of the Sender Policy Framework for SMTP authorization
http://www.libspf2.net/
37 stars 32 forks source link

Fix "redirect" modifier handling #10

Open ipronin opened 9 years ago

ipronin commented 9 years ago

The RFC demands that "redirect" modifier is applied only if all other mechanisms failed to match. So I moved it to the end of the mechanisms list.

dreasvandonselaar commented 7 years ago

The RFC indicates:

Any "redirect" modifier (Section 6.1) MUST be
   ignored when there is an "all" mechanism in the record, regardless of
   the relative ordering of the terms.

Based on your comment it seems to suggest that a redirect is still processed by your patch (just last)? Shouldn't it be ignored entirely instead if there is an all mechanism?