Cached result contains header instead of message

Bug #1422325 reported by Bastian Blank
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
pypolicyd-spf
Fix Released
Medium
Scott Kitterman

Bug Description

policyd-spf includes a cache for old results. The cache is filed with the header, instead of the reject reason. This leads to weird responses:
<** 550 5.7.1 <email address hidden>: Recipient address rejected: Received-SPF: Softfail (domain owner discourages use of this host) identity=mailfrom; client-ip=192.0.2.0; helo=example.com; <email address hidden>; <email address hidden>

The attached patch caches the correct result, but it depends on #1422324.

Related branches

Revision history for this message
Bastian Blank (waldi) wrote :
Revision history for this message
Scott Kitterman (kitterman) wrote :

Thanks.

Changed in pypolicyd-spf:
status: New → Confirmed
importance: Undecided → Medium
Revision history for this message
Scott Kitterman (kitterman) wrote :

I'm trying to get a bug fix only release out now, so I'm holding #1422324 for the next feature release, so I redid this based on the current code. Thanks again.

Changed in pypolicyd-spf:
assignee: nobody → Scott Kitterman (kitterman)
status: Confirmed → Fix Committed
Revision history for this message
Scott Kitterman (kitterman) wrote :
Changed in pypolicyd-spf:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.