Fix security issue: Unexpected access to any tasks from a shared public board
This commit is contained in:
parent
a7f3e3bec5
commit
f9f5d7188b
|
|
@ -5,6 +5,10 @@ Improvements:
|
|||
|
||||
* Added support for HTTP header "X-Forwarded-Proto: https"
|
||||
|
||||
Security issues:
|
||||
|
||||
* Access allowed to any tasks from the shared public board by changing the URL parameters
|
||||
|
||||
Version 1.0.26
|
||||
--------------
|
||||
|
||||
|
|
|
|||
|
|
@ -23,13 +23,17 @@ class Task extends Base
|
|||
|
||||
// Token verification
|
||||
if (empty($project)) {
|
||||
$this->forbidden(true);
|
||||
return $this->forbidden(true);
|
||||
}
|
||||
|
||||
$task = $this->taskFinder->getDetails($this->request->getIntegerParam('task_id'));
|
||||
|
||||
if (empty($task)) {
|
||||
$this->notfound(true);
|
||||
return $this->notfound(true);
|
||||
}
|
||||
|
||||
if ($task['project_id'] != $project['id']) {
|
||||
return $this->forbidden(true);
|
||||
}
|
||||
|
||||
$this->response->html($this->helper->layout->app('task/public', array(
|
||||
|
|
|
|||
Loading…
Reference in New Issue