View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0007364 | mantisbt | security | public | 2006-08-17 04:46 | 2006-12-08 02:39 |
Reporter | chschulz | Assigned To | thraxisp | ||
Priority | normal | Severity | block | Reproducibility | always |
Status | closed | Resolution | fixed | ||
Product Version | 1.0.2 | ||||
Fixed in Version | 1.1.0a2 | ||||
Summary | 0007364: Custom field visible in history independent from user role | ||||
Description | We found a - for us - big security problem. We defined some custom fields for a project. One of this custom fields is: Name: test This field is NOT visible for user with roles less than Manager in die normal detail view. But, the big problem is that the field (including the change) is visible in the history of an issue for everyone independent from the user's role. This is a big security problem. Because we configured it that way, that it is only visible for Manager. But when a "viewer" for example look to the history, the viewer can see that the value of field x was changed to ABC. | ||||
Tags | No tags attached. | ||||
As additional information: We did not check it but we're sure that the same problem also occurs for the e-mail notification as this contains the history. In the e-mail notification, also in the history only those field should be available which the user who recevices the notification is able to read. |
|
The first issue, visibility of custom fields in the history could be fixed by adding the following in history_inc.php: After the line: add:
and before the end of the loop, add:
I do not know if this is the best way to fix it, but it works :-) |
|
I think this solution isn't really good, because there's still the problem that the custom fields will be shown in the e-mail notification. And also from my perspective, it may be a performance issue. Because fields/information are read from the database that are not really needed... |
|
fixed in CVS |
|