diff options
author | Florian Pritz <bluewind@xinu.at> | 2016-07-13 21:47:56 +0200 |
---|---|---|
committer | Florian Pritz <bluewind@xinu.at> | 2016-07-13 21:47:56 +0200 |
commit | 3bd289a3c14f377c3c689f97cf20366571f6add9 (patch) | |
tree | 9b523be2327def5e18c6a22e6084bbae570f5277 /application/controllers | |
parent | f67a3cf212a774945a0d10143e93071e19dc40f4 (diff) |
Throw public exception when accessing CLI only functions
We actually don't need to hide this from the user. The error should be
shown rather than a blank page being returned.
Signed-off-by: Florian Pritz <bluewind@xinu.at>
Diffstat (limited to 'application/controllers')
-rw-r--r-- | application/controllers/file.php | 8 | ||||
-rw-r--r-- | application/controllers/tools.php | 4 | ||||
-rw-r--r-- | application/controllers/user.php | 4 |
3 files changed, 7 insertions, 9 deletions
diff --git a/application/controllers/file.php b/application/controllers/file.php index 328fc551f..e1f37e0e8 100644 --- a/application/controllers/file.php +++ b/application/controllers/file.php @@ -1066,7 +1066,7 @@ class File extends MY_Controller { // Removes old files function cron() { - if (!$this->input->is_cli_request()) return; + $this->_require_cli_request(); $tarball_dir = $this->config->item("upload_path")."/special/multipaste-tarballs"; if (is_dir($tarball_dir)) { @@ -1124,7 +1124,7 @@ class File extends MY_Controller { /* remove files without database entries */ function clean_stale_files() { - if (!$this->input->is_cli_request()) return; + $this->_require_cli_request(); $upload_path = $this->config->item("upload_path"); $outer_dh = opendir($upload_path); @@ -1178,7 +1178,7 @@ class File extends MY_Controller { function nuke_id() { - if (!$this->input->is_cli_request()) return; + $this->_require_cli_request(); $id = $this->uri->segment(3); @@ -1196,7 +1196,7 @@ class File extends MY_Controller { function update_file_metadata() { - if (!$this->input->is_cli_request()) return; + $this->_require_cli_request(); $chunk = 500; diff --git a/application/controllers/tools.php b/application/controllers/tools.php index 664a9e324..845597b08 100644 --- a/application/controllers/tools.php +++ b/application/controllers/tools.php @@ -14,9 +14,7 @@ class Tools extends MY_Controller { parent::__construct(); $this->load->model('mfile'); - if (!$this->input->is_cli_request()) { - throw new \exceptions\PublicApiException("api/cli-only", "This can only be called via CLI"); - } + $this->_require_cli_request(); } function index() diff --git a/application/controllers/user.php b/application/controllers/user.php index e1c01051a..5d506f9d7 100644 --- a/application/controllers/user.php +++ b/application/controllers/user.php @@ -586,7 +586,7 @@ class User extends MY_Controller { function cron() { - if (!$this->input->is_cli_request()) return; + $this->_require_cli_request(); if ($this->config->item('actions_max_age') == 0) return; @@ -641,7 +641,7 @@ class User extends MY_Controller { function add_user() { - if (!$this->input->is_cli_request()) return; + $this->_require_cli_request(); $this->duser->require_implemented("can_register_new_users"); $error = array(); |