BigW Consortium Gitlab
The issue was with the `User#groups` and `User#projects` associations
which goes through the `User#group_members` and `User#project_members`.
Initially I chose to use a secure approach by storing the requester's
user ID in `Member#created_by_id` instead of `Member#user_id` because I
was aware that there was a security risk since I didn't know the
codebase well enough.
Then during the review, we decided to change that and directly store the
requester's user ID into `Member#user_id` (for the sake of simplifying
the code I believe), meaning that every `group_members` / `project_members`
association would include the requesters by default...
My bad for not checking that all the `group_members` / `project_members`
associations and the ones that go through them (e.g. `Group#users` and
`Project#users`) were made safe with the `where(requested_at: nil)` /
`where(members: { requested_at: nil })` scopes.
Now they are all secure.
Signed-off-by: Rémy Coutable <remy@rymai.me>
Name |
Last commit
|
Last update |
---|---|---|
.. | ||
admin | Loading commit data... | |
atom | Loading commit data... | |
dashboard | Loading commit data... | |
groups/members | Loading commit data... | |
issues | Loading commit data... | |
merge_requests | Loading commit data... | |
profiles | Loading commit data... | |
projects | Loading commit data... | |
security | Loading commit data... | |
tags | Loading commit data... | |
todos | Loading commit data... | |
builds_spec.rb | Loading commit data... | |
ci_lint_spec.rb | Loading commit data... | |
commits_spec.rb | Loading commit data... | |
container_registry_spec.rb | Loading commit data... | |
dashboard_issues_spec.rb | Loading commit data... | |
dashboard_milestones_spec.rb | Loading commit data... | |
environments_spec.rb | Loading commit data... | |
gitlab_flavored_markdown_spec.rb | Loading commit data... | |
groups_spec.rb | Loading commit data... | |
help_pages_spec.rb | Loading commit data... | |
issues_spec.rb | Loading commit data... | |
login_spec.rb | Loading commit data... | |
markdown_spec.rb | Loading commit data... | |
milestone_spec.rb | Loading commit data... | |
notes_on_merge_requests_spec.rb | Loading commit data... | |
participants_autocomplete_spec.rb | Loading commit data... | |
password_reset_spec.rb | Loading commit data... | |
pipelines_spec.rb | Loading commit data... | |
profile_spec.rb | Loading commit data... | |
projects_spec.rb | Loading commit data... | |
runners_spec.rb | Loading commit data... | |
search_spec.rb | Loading commit data... | |
signup_spec.rb | Loading commit data... | |
task_lists_spec.rb | Loading commit data... | |
triggers_spec.rb | Loading commit data... | |
u2f_spec.rb | Loading commit data... | |
users_spec.rb | Loading commit data... | |
variables_spec.rb | Loading commit data... |