Changes between Initial Version and Version 1 of Ticket #20869, comment 23


Ignore:
Timestamp:
Aug 11, 2013, 1:43:45 AM (11 years ago)
Author:
Adam Brenecki

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #20869, comment 23

    initial v1  
    11So, my patch now works on Python 3, and the tests now pass.
    22
    3 Should I write any new tests? Should I also make `_check_token_present` in the tests assert that the CSRF token in the body `!=` the plaintext token?
     3Should I write any new tests? Should I also make `_check_token_present` in the tests assert that the CSRF token in the body `!=` the plaintext token (to make sure that this vulnerability isn't reintroduced somehow by accident)?
Back to Top