View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0001203 | CATS.cacert.org | Other | public | 2013-08-20 00:21 | 2013-09-02 10:59 |
Reporter | law | Assigned To | Ted | ||
Priority | normal | Severity | crash | Reproducibility | always |
Status | new | Resolution | open | ||
Product Version | production | ||||
Target Version | production | ||||
Summary | 0001203: session_register was removed in PHP 5.4 | ||||
Description | session_register is used by index.php:3 Upgrading to php 5.4 broke cats. As an emergency fix, this line was commented in the production system. See http://de3.php.net/session_register The usage of session_register does not make any sense to me in this place anyway. Why was this not noticed with 0001191? | ||||
Tags | No tags attached. | ||||
|
https://github.com/CAcertOrg/cats/pull/6 |
|
Patch looks okay. |
|
Lowering priority since a patch is active on the production system and was reviewed by one SA (note that this is not software under SP). While dicussing why this was not noticed in 0001191 it was noted that a similar change to the source code is in effect on the CATS test system. As a followup I suggest to the CATS software owner to bring the code in the (github) repository, the test system and the production system in sync and/or to use git branching feature to manage different versions/stages of the software. The file in question is not versioned, but a template is. The SCM on the production system also shows changes in it's working copy, which imho should not be the case on a production system. |
Date Modified | Username | Field | Change |
---|---|---|---|
2013-08-20 00:21 | law | New Issue | |
2013-08-20 00:21 | law | Assigned To | => Ted |
2013-08-20 00:23 | law | Description Updated | |
2013-08-20 00:33 | law | Note Added: 0004233 | |
2013-08-20 20:50 | BenBE | Note Added: 0004243 | |
2013-09-02 10:59 | law | Note Added: 0004266 | |
2013-09-02 10:59 | law | Priority | immediate => normal |