View Issue Details

IDProjectCategoryView StatusLast Update
0001203CATS.cacert.orgOtherpublic2013-09-02 10:59
Reporterlaw Assigned ToTed  
PrioritynormalSeveritycrashReproducibilityalways
Status newResolutionopen 
Product Versionproduction 
Target Versionproduction 
Summary0001203: session_register was removed in PHP 5.4
Descriptionsession_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?
TagsNo tags attached.

Activities

law

2013-08-20 00:33

administrator   ~0004233

https://github.com/CAcertOrg/cats/pull/6

BenBE

2013-08-20 20:50

updater   ~0004243

Patch looks okay.

law

2013-09-02 10:59

administrator   ~0004266

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.

Issue History

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 View Revisions
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