View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
12011 | Feature requests | Security | public | 2016-12-19 18:12 | 2017-09-22 10:59 |
Reporter | DenisChenu | Assigned To | DenisChenu | ||
Priority | none | Severity | feature | ||
Status | closed | Resolution | fixed | ||
Fixed in Version | 2.x.x | ||||
Summary | 12011: Need an updatable runtime path | ||||
Description | Currently all files in tmp are accessible by default. For example : log files is saved in temp/runtime, template preview file in tmp tmp/assets : must be public , web accessible | ||||
Additional Information | Then i try to set tmp directory out of access : there only onse settings ... then assets must be in this directory too .... In Yii runtime are in protected directory. It's difficult to set it elsewhere BY DEFAULT, but allow admin user to really secure LS seems a good idea. PS : if we can move whole PHP file anywhere else it can be better (protected directory of Yii), but it's really more work. Maybe use 2 tmps directory ? Default is the same ./tmp/ but runtime use $config['privatetmp'] and assets $config['tmp'] | ||||
Tags | No tags attached. | ||||
Bug heat | 252 | ||||
Story point estimate | |||||
Users affected % | |||||
related to | 12018 | closed | DenisChenu | Bug reports | Cache files are world-writable |
@ollehar : to really protect a directory files from web : the only solution ar to put out of directory of host (see php session file for example). Then , the idea config-default: After just use tempdir when set runtime path or upload path (or some other). By default : leave like curretly, but with manual: we can easily explain how to secure more LS. |
|
Maybe we just need See https://github.com/LimeSurvey/LimeSurvey/blob/master/application/core/LSYii_Application.php#L61 And we can do same for asset (but very less necessary) |
|
Fix committed to develop branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=21888 |
|
https://manual.limesurvey.org/index.php?title=Optional_settings&type=revision&diff=69431&oldid=69362 |
|
In fact : Yii already offer to set runtimePath : then best solution are : I do it for master and merge in develop after (because i know what problem happen : i update code at same place ). |
|
Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=22153 |
|
Fix committed to develop branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=22225 |
|
Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=22339 |
|
LimeSurvey: develop 3a83f96f 2017-01-03 09:57 Details Diff |
New feature 12011: updatable runtime path Dev: todo : manual update |
Affected Issues 12011 |
|
mod - application/commands/console.php | Diff File | ||
mod - application/config/config-defaults.php | Diff File | ||
mod - application/core/LSYii_Application.php | Diff File | ||
LimeSurvey: master 64d7240f 2017-01-20 12:39 Details Diff |
New feature 12011 : allow user to set runtimePath (Yii way) |
Affected Issues 12011 |
|
mod - application/controllers/InstallerController.php | Diff File | ||
mod - application/core/LSYii_Application.php | Diff File | ||
LimeSurvey: develop 1878a118 2017-02-01 00:39 Details Diff |
New feature 12095 assetUrl and assetPath can be set in config.php New feature 12011 : allow user to set runtimePath (Yii way) |
Affected Issues 12011, 12095 |
|
mod - application/config/config-defaults.php | Diff File | ||
mod - application/core/LSYii_Application.php | Diff File | ||
mod - index.php | Diff File | ||
LimeSurvey: master 9b45ea98 2017-02-20 18:23 Details Diff |
Dev: New feature 12011 for console too Dev: better system in develop, then quick fix |
Affected Issues 12011 |
|
mod - application/commands/console.php | Diff File |
Date Modified | Username | Field | Change |
---|---|---|---|
2016-12-19 18:12 | DenisChenu | New Issue | |
2016-12-19 18:16 | DenisChenu | Note Added: 42566 | |
2016-12-22 02:06 | DenisChenu | Relationship added | related to 12018 |
2016-12-22 11:34 | DenisChenu | Note Added: 42593 | |
2016-12-25 15:49 | DenisChenu | Summary | Separate tmp/public and tmp/private|protected => Need an updatatle runtime path |
2016-12-25 15:49 | DenisChenu | Assigned To | => DenisChenu |
2016-12-25 15:49 | DenisChenu | Status | new => assigned |
2016-12-25 16:39 | DenisChenu | Summary | Need an updatatle runtime path => Need an updatable runtime path |
2017-01-03 08:59 | DenisChenu | Changeset attached | => LimeSurvey develop 3a83f96f |
2017-01-03 08:59 | DenisChenu | Note Added: 42612 | |
2017-01-03 08:59 | DenisChenu | Resolution | open => fixed |
2017-01-03 08:59 | DenisChenu | Status | assigned => resolved |
2017-01-03 08:59 | DenisChenu | Fixed in Version | => 3.0 |
2017-01-03 09:09 | DenisChenu | Status | resolved => feedback |
2017-01-03 09:09 | DenisChenu | Resolution | fixed => reopened |
2017-01-03 09:09 | DenisChenu | Note Added: 42613 | |
2017-01-03 09:09 | DenisChenu | Status | feedback => resolved |
2017-01-03 09:09 | DenisChenu | Resolution | reopened => fixed |
2017-01-20 08:17 | DenisChenu | Status | resolved => feedback |
2017-01-20 08:17 | DenisChenu | Resolution | fixed => reopened |
2017-01-20 08:17 | DenisChenu | Note Added: 42782 | |
2017-01-20 08:17 | DenisChenu | Status | feedback => new |
2017-01-20 08:17 | DenisChenu | Status | new => assigned |
2017-01-20 11:43 | DenisChenu | Changeset attached | => LimeSurvey master 64d7240f |
2017-01-20 11:43 | DenisChenu | Note Added: 42786 | |
2017-01-21 14:02 | DenisChenu | Status | assigned => resolved |
2017-01-21 14:02 | DenisChenu | Resolution | reopened => fixed |
2017-01-21 14:02 | DenisChenu | Fixed in Version | 3.0 => |
2017-01-31 23:54 | DenisChenu | Changeset attached | => LimeSurvey develop 1878a118 |
2017-01-31 23:54 | DenisChenu | Note Added: 42925 | |
2017-02-20 17:28 | DenisChenu | Changeset attached | => LimeSurvey master 9b45ea98 |
2017-02-20 17:28 | DenisChenu | Note Added: 43038 | |
2017-09-22 10:59 | c_schmitz | Status | resolved => closed |
2017-09-22 10:59 | c_schmitz | Fixed in Version | => 2.x.x |