Bug 1229171
| Summary: | [GSS] (6.1.z) Any authenticated user can see tasks which doesn't own by remote REST api | ||||||
|---|---|---|---|---|---|---|---|
| Product: | [Retired] JBoss BPMS Platform 6 | Reporter: | Hiroko Miura <hmiura> | ||||
| Component: | Business Central | Assignee: | Shelly McGowan <smcgowan> | ||||
| Status: | CLOSED EOL | QA Contact: | Lukáš Petrovický <lpetrovi> | ||||
| Severity: | high | Docs Contact: | |||||
| Priority: | high | ||||||
| Version: | 6.1.0 | CC: | alazarot, ibek, kverlaen, rrajasek | ||||
| Target Milestone: | CR1 | ||||||
| Target Release: | 6.1.0 | ||||||
| Hardware: | Unspecified | ||||||
| OS: | All | ||||||
| Whiteboard: | |||||||
| Fixed In Version: | Doc Type: | Bug Fix | |||||
| Doc Text: | Story Points: | --- | |||||
| Clone Of: | |||||||
| : | 1230947 (view as bug list) | Environment: | |||||
| Last Closed: | 2020-03-27 20:10:33 UTC | Type: | Bug | ||||
| Regression: | --- | Mount Type: | --- | ||||
| Documentation: | --- | CRM: | |||||
| Verified Versions: | Category: | --- | |||||
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||
| Cloudforms Team: | --- | Target Upstream Version: | |||||
| Embargoed: | |||||||
| Bug Depends On: | |||||||
| Bug Blocks: | 1230947 | ||||||
| Attachments: |
|
||||||
|
Description
Hiroko Miura
2015-06-08 08:40:04 UTC
Fixed (see next comment for explanation). Commits: 6.2.x: https://github.com/droolsjbpm/droolsjbpm-integration/commit/08a3c05a master: https://github.com/droolsjbpm/droolsjbpm-integration/commit/728a4536 What I've done is to add checks on both the client side so that GetTask* commands that use a different user id than the authenticated user, are rejected. If a different user id than the authenticated user is used, an exception is now thrown. **NOTE**: This may cause problems for users who are currently (ab)using this functionality! Users who are used to this "feature" (using user A to authenticate the REST or JMS request and requesting tasks for user B) will now have to modify their systems. I found a small bug related to this issue/fix and have also pushed a commit for this: 6.2.x: https://github.com/droolsjbpm/droolsjbpm-integration/commit/315a0bcc master: https://github.com/droolsjbpm/droolsjbpm-integration/commit/37c28f6a This has been pushed to 6.2.x because the patch will be rebuilt due to another issue. Because including this change in a patch to product 6.1.x would be disruptive to existing users, I've reverted the community 6.2.x commits. 6.2.x revert commits: https://github.com/droolsjbpm/droolsjbpm-integration/commit/b5a14ab4 Since the customer is not asking for a fix right now and this has a possibly larger impact on customers (as it might prevent them from doing something they were doing in the past), I suggest we don't address this in a patch update but in the next GA release instead. |