Changes between Initial Version and Version 1 of Ticket #20869, comment 23
- Timestamp:
- Aug 11, 2013, 1:43:45 AM (11 years ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Ticket #20869, comment 23
initial v1 1 1 So, my patch now works on Python 3, and the tests now pass. 2 2 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 ?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 (to make sure that this vulnerability isn't reintroduced somehow by accident)?