MantisBT - CDash | |||||
View Issue Details | |||||
ID | Project | Category | View Status | Date Submitted | Last Update |
0014995 | CDash | (No Category) | public | 2014-06-27 04:12 | 2014-06-27 14:21 |
Reporter | Julien Jomier | ||||
Assigned To | |||||
Priority | normal | Severity | minor | Reproducibility | have not tried |
Status | new | Resolution | open | ||
Platform | OS | OS Version | |||
Product Version | 2.2 | ||||
Target Version | 2.4 | Fixed in Version | |||
Summary | 0014995: User page HTTPS redirect only when HTTPS is available | ||||
Description | CDash user.php redirects to HTTPS but this doesn't work when https is not available. | ||||
Steps To Reproduce | |||||
Additional Information | See email Suggested patch to version 222 with regards to redirection to https:// from user.php by Kristofer Tingdahl > 1. Thanks for version 222!! > 2. My system administrator installed cdash 222, and everything went well > (in general). We did however have a problem as loading user.php > redirects to https, while our setup does not support that. As our system > is in a shielded off network, we don't see the need for it. > > The current functionality calls redirect_to_https(). My problem is that > the check that is done at: > > $https_check = @fsockopen($_SERVER['HTTP_HOST']); > > > does check the http connection (which works), and not a https > connection. The attached patch changes that, so the test becomes more > realistic to what we are about to do. I guess that this is only half > way, as you may wish the possibility to set the port of the https-call > than to the current http call. That is not covered by my patch. > > > I can see that in a public system, it is indeed very useful to do this > redirection. In the light of this, I am proposing a patch (as attached) > for your consideration. | ||||
Tags | No tags attached. | ||||
Relationships | |||||
Attached Files | ![]() https://public.kitware.com/Bug/file/5177/redirect_httpd.diff | ||||
Issue History | |||||
Date Modified | Username | Field | Change | ||
2014-06-27 04:12 | Julien Jomier | New Issue | |||
2014-06-27 14:21 | Kristofer Tingdahl | File Added: redirect_httpd.diff |
There are no notes attached to this issue. |