Skip to content

Add user to be required for allowed_for?

Mohamed Hamda requested to merge fix-incident-18268 into master

What does this MR do and why?

Related to gitlab-com/gl-infra/production#18268

The causing MR

The Fix:

  • Adjust to check the user's existence

We verified that the fix for issues and epics, and specs fails if our new fix is commented out, which means it solves the issue.

Click to expand

Screenshot_2024-07-11_at_17.07.37

Screenshot_2024-07-11_at_17.13.06

With this fix, no error without a logged-in user on issues/epics

Click to expand

image__5_

image__6_

cc/ @sxuereb

MR acceptance checklist

Edited by Mohamed Hamda

Merge request reports