| Issue 1888: | Reviewed check mark missing | |
| 6 people starred this issue and may be notified of changes. | Back to list |
Affected Version: 2.6 rc-1 What steps will reproduce the problem? 1. Review some files, 2. When reviewing the file the box reviewed is ticked 3. In the overview the check mark is no longer there What is the expected output? What do you see instead? I was expecting to see the check mark behind the files that have been reviewed as I saw in 2.5.2 Please provide any additional information below.
Apr 25, 2013
#1
sop@google.com
Status:
AwaitingInformation
Apr 25, 2013
Yes I am...
Apr 25, 2013
The recommendation has changed to work around a "feature" of Apache: https://gerrit-review.googlesource.com/Documentation/config-reverseproxy.html#_apache_2_configuration
Status:
Invalid
Apr 26, 2013
I have currently two server running. One is now OK. The configuration for apache2 is identical: 1. Ubuntu 12.04, apache 2.2.22 Gerrit 2.6-rc1 2. Windows XP, apache 2.2.22, Gerrit 2.6-rc1--315-gdfbaa62 (build today from master) The funny thing is that for server 2 I get the tick for the file changes but not for the commit message while for server 1 I get the tick for both. Any idea?
Apr 29, 2013
Issue 1892 has been merged into this issue.
Apr 30, 2013
Could this not be made a bit more robust? The ticks do work for the "Code-Review" and "Verified" labels so this is not something that can't be done.
Apr 30, 2013
I've the problem that the latest version I can get apache to is 0.0.2 from supporting the NoDecode option (and On or Off does not work in its place), and a full-distribution upgrade represents a major task (which of cause have to be done sooner or later, but this is not only a gerrit server and other systems may break during this).
May 8, 2013
Issue 1906 has been merged into this issue.
Aug 14, 2013
It would be nice if this problem could be detected and admin users notified about it. Optionally, a "sanity check" url would be useful -- It could check for various problems and report if things are good or not. |
|
| ► Sign in to add a comment |