#8047 Akismet filter needs to send original metadata when reporting spam/ham

v1.4.0
closed
General
Heith Seewald
2016-01-25
2016-01-18
No

We currently send the current user etc, when submitting spam or ham reports. Per http://akismet.com/development/api/#submit-spam its important that we send the original post's metadata.

Discussion

  • Dave Brondsema

    Dave Brondsema - 2016-01-18
    • status: in-progress --> review
     
  • Dave Brondsema

    Dave Brondsema - 2016-01-18

    db/8047

    To test, enable akismet in development.ini (make sure to disable again when running tests, as it affects them). Add some logging in akismetfilter.py get_data so you can see the requests. Then post a comment that gets moderated, confirm data is good. Then approve it or mark as spam and confirm data has original user info and no request data (ip addr, ua, referrer).

     
  • Heith Seewald

    Heith Seewald - 2016-01-20
    • Reviewer: Heith Seewald
     
  • Heith Seewald

    Heith Seewald - 2016-01-20
    • status: review --> closed
     
  • Heith Seewald

    Heith Seewald - 2016-01-20

    Looks good.

     
  • Dave Brondsema

    Dave Brondsema - 2016-01-25
    • labels: spam, sf-current, sf-1 --> spam, sf-1
     
  • Dave Brondsema

    Dave Brondsema - 2016-04-11
    • Milestone: unreleased --> v1.4.0
     

Log in to post a comment.