#316 Verify that IP addresses are logged when artifacts are changed

v1.0.0
closed
sf-2 (994)
General
nobody
2015-08-20
2010-07-07
Anonymous
No

Originally created by: sf-overlords

Created by: mramm
Created date: 2010-04-28 17:48:06.569000
Assigned to:nobody

It would be good to keep IP address for anti-spam issues.

Discussion

  • Rick Copeland - 2010-11-15
    • summary: Track IP addresses on versioned artifacts --> Verify that IP addresses are logged when artifacts are changed
    • custom_field__size: 0 --> 1
     
  • Mark Ramm - 2010-11-19
    • custom_field__milestone: nov-22 --> backlog
     
  • Mark Ramm - 2011-01-24
    • milestone: backlog --> jan-31
     
  • John Hoffman - 2011-01-26
    • assigned_to: John Hoffmann
     
  • John Hoffman - 2011-01-28

    John Hoffmann: quick question when you have time, for #316, you changed the description from 'track ip addresses' to 'verify that ip addresses are logged', can you elaborate a bit? i dug thru the code and didn't see us capturing them anywhere, though the history seems to indicate we might be doing it, perhaps outside of python? (httpd logs?)
    Richard Copeland: let's see -- I changed that in Nov, so I don't exactly recall
    John Hoffmann: it was created in April
    Richard Copeland: I think the point now is to make sure that the commit() method on versioned artifacts saves the IP in the snapshot object
    John Hoffmann: ok, i'll poke at it then
    Richard Copeland: thanks

     
  • John Hoffman - 2011-02-02
    • status: open --> code-review
     
  • John Hoffman - 2011-02-02
    • assigned_to: John Hoffmann --> Rick Copéland
     
  • John Hoffman - 2011-02-02

    Rick, can you fix the testing framework so it sets up a dummy request object?

     
  • Dave Brondsema

    Dave Brondsema - 2011-02-02

    There's also going to be code that runs without a request - all the background reactor processes.

     
  • Rick Copeland - 2011-02-02
    • assigned_to: Rick Copéland --> John Hoffmann
     
  • Rick Copeland - 2011-02-02

    New commit and sent back for review

     
  • John Hoffman - 2011-02-03
    • status: code-review --> closed
     

Log in to post a comment.