Skip to content

Remove current_user from JobsFinder

Avielle Wolfe requested to merge remove-current-user-from-jobs-finder into master

What does this MR do?

Removes the current_user constructor argument and all authentication logic from ::Ci::JobsFinder's constructor.

In all use cases of Ci::JobsFinder, the internal authentication duplicated existing authentication in controllers and API endpoints.

This MR is preliminary work needed to merge Security::JobsFinder into ::Ci::JobsFinder, which will allow all stages to fetch jobs filtered by report_type.

This is the first MR for #251015 (closed)

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Merge request reports