Feature #114

check that comments work

Added by yossarian over 3 years ago. Updated over 3 years ago.

Status:Closed Start:
Priority:High Due date:
Assigned to:yossarian % Done:

0%

Category:Articles
Target version:0.4 - Rebel Yell (2b || !2b)
Resolution:

fixed


Description

Make sure that comments work with the caching setup in combination with the captcha. I could see that it could easily be the case that the captcha image is cached when the first page view occurs and then comment publishing fails because the captcha is out of date. It might be working, but I'm not too sure what's going to happen when there are a bunch of people attempting to comment more or less simultaneously.

History

Updated by yossarian over 3 years ago

  • Status changed from New to Assigned

The captcha can indeed get out of date, so if Person A and Person B both try to comment, and Person A submits their comment first, Person B will get a validation error on the captcha - they can then go on to submit the comment no problem but it's kinda ugly.

Updated by yossarian over 3 years ago

  • Status changed from Assigned to Closed
  • Resolution set to fixed

This has now been fixed by making the comment form into an ajax thing. It's not an ideal solution but it's good enough for now.

Also available in: Atom PDF