Revert "Merge branch '582001/aslota-remove-per-participant-permission-check' into 'master'"

Background

As part of discussions in this issue, and with approval from the AppSec team, we removed per-source and per-participant permission checks from the participants API endpoints to improve performance. These changes ensured that all participants were listed regardless of their current access level to the issuable's parent (project or group) or the participatable source (such as a note). We also updated the documentation to reflect that this is now the expected behaviour.

Purpose of revert

After receiving an influx of support tickets and concerns from customers about security, we decided to revert the changes and re-evaluate how we want to move forward.

This MR reverts !213623 (merged) and !214345 (merged).

References

Checklist

Milestone info

  • I am reverting something in the current milestone. No changelog is needed, and I've added a ~"regression:*" label.
  • I am reverting something in a different milestone. A changelog is needed, and I've removed the ~"regression:*" label.
Edited by Agnes Slota

Merge request reports

Loading