Skip to content

Adjust order of verifications for CommitsController

Vasilii Iakliushinrequested to merge
vi-change-execution-order into master

What does this MR do and why?

Contributes to https://gitlab.com/gitlab-com/gl-infra/production-engineering/-/issues/27625

Problem

The previous merge request !206365 (merged) introduced graceful degradation for unauthenticated routes. But it had a problem with the execution order commits being loaded before auth. This is suboptimal.

Solution

Ensure that authentication is triggered before commits.

MR acceptance checklist

Evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Merge request reports