Don't try to log in infinite number of times.

Review Request #813 — Created April 13, 2009 and submitted — Latest diff uploaded

Information

Review Board SVN (deprecated)
808

Reviewers

In some cases (which I'm still not 100% clear on), the post-review cookies can get messed up.  This causes post-review to go into a recursive loop trying to log in.  This is a simple fix which appears to solve the problem.
Using a cookie file that I could reproduce the problem with, I verified that it asked for my login after failing to post using the existing cookie.  I also verified that with a good cookie it still works, and with no cookies it asks for your login as expected.

I imagine you could reproduce this by editing post-review-cookies.txt and mangling the rbsessionid.
    Loading...