Comment #2 on issue 1334 by th...@ginkel.com: Fragment expansion brings up
404 error
http://code.google.com/p/reviewboard/issues/detail?id=1334
The following test was performed with a web browser that did not previously
access
the review request, so client-side caching should not be in place.
When initially accessing the request and clicking "Expand" the error
happened again.
Just to be on the safe side I also shift-reloaded the page and hit "Expand"
again -
with unchanged results.
One of the "Expand" links reads:
-- 8< --
87 lines hidden [<a href="#"
onclick="javascript:expandChunk('file0', '2830', '0',
null, '0', this); return false;">Expand</a>]
-- 8< --
It apparently references the incorrect diff revision. In the same HTTP
response
(towards the top of the request) there is also a variable that correlates
to the diff
revision:
var gRevision = '1';
This one, however, seems to be correct and reference diff #1, so I'd rule
out any
HTTP-related caching issues.
Next, I restarted the Apache httpd that hosts Review Board to rule out any
in-memory
mod_python issues. The issue still persisted.
Restarting memcached, however, solved the issue, so cached content seems to
play an
important role.
--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"reviewboard-issues" group.
To post to this group, send email to reviewboard-issues@googlegroups.com
To unsubscribe from this group, send email to
reviewboard-issues+unsubscr...@googlegroups.com
For more options, visit this group at
http://groups.google.com/group/reviewboard-issues?hl=en
-~----------~----~----~----~------~----~------~--~---
