SiT! Bugs - SiT!
View Issue Details
0000723SiT!otherpublic2009-06-09 20:182011-06-04 17:18
paulh 
ivan 
normalfeatureN/A
closedno change required 
 
 
0000723: Get away from hardcoding /etc/sit.conf
As this makes it difficult to run multiple instances of SiT! on the same server and also these includes make no sense on Windows.

Ideally this should be a config variable though we need to be loaded before including (obviously) and the place we have for this at the moment is config.inc.php which is included afterwards.

My suggestion would be that you can optionally include a sit_config_location.conf (or similar named file) in inc which defaults includes (if it exists) which can declare this variable if its not set then we could default to /etc/sit.conf.
No tags attached.
related to 0000724resolved paulh Remove webtrack.conf include 
Issue History
2009-06-09 20:18paulhNew Issue
2009-06-09 20:19paulhProjectionnone => minor fix
2009-06-09 20:19paulhETAnone => < 1 day
2009-06-10 09:23kieranNote Added: 0001172
2009-06-10 09:23kieranStatusnew => confirmed
2009-06-10 11:40ivanRelationship addedrelated to 0000724
2009-06-10 11:42ivanNote Added: 0001173
2011-06-04 17:18ivanNote Added: 0003839
2011-06-04 17:18ivanStatusconfirmed => closed
2011-06-04 17:18ivanAssigned To => ivan
2011-06-04 17:18ivanResolutionopen => no change required

Notes
(0001172)
kieran   
2009-06-10 09:23   
I wasn't sure which of the two bugs to add this comment to, this one seems okay though.

Confirming this based on the problems we have installing SiT on sitracker.org:

Warning: file_exists() [function.file-exists]: open_basedir restriction in effect. File(/etc/sit.conf) is not within the allowed path(s): (/tmp:/var/sites) in /var/sites/s/support.sitracker.org/public_html/setup.php on line 85
(0001173)
ivan   
2009-06-10 11:42   
maybe we should just use config.inc.php in the sit directory, and if somebody wants to use another file they can put include('/etc/sit-other-file.php'); or whatever into that config.inc.php file.
(0003839)
ivan   
2011-06-04 17:18   
Solution via config.inc.php already, see notes.