diff options
author | Frédéric Guillot <fred@kanboard.net> | 2019-01-31 20:06:49 -0800 |
---|---|---|
committer | Frédéric Guillot <fred@kanboard.net> | 2019-01-31 20:06:49 -0800 |
commit | 061ba4abe179829d7d0acd3422a16110dbc91da5 (patch) | |
tree | 895c45a3ecdd6b341c19cf7b5acde3bf0c196016 /app/Controller | |
parent | 928f80d569141601c4d8f6652b2ac0a2497c7be4 (diff) |
Avoid CSRF in users CSV import
Diffstat (limited to 'app/Controller')
-rw-r--r-- | app/Controller/UserImportController.php | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/app/Controller/UserImportController.php b/app/Controller/UserImportController.php index 6a9d5992..e878e605 100644 --- a/app/Controller/UserImportController.php +++ b/app/Controller/UserImportController.php @@ -3,6 +3,7 @@ namespace Kanboard\Controller; use Kanboard\Core\Csv; +use Kanboard\Core\Controller\AccessForbiddenException; /** * User Import controller @@ -35,6 +36,12 @@ class UserImportController extends BaseController public function save() { $values = $this->request->getValues(); + + // Note: $values is empty when the CSRF token is invalid. + if (empty($values)) { + throw new AccessForbiddenException(); + } + $filename = $this->request->getFilePath('file'); if (! file_exists($filename)) { |