Do not check if a user can read_build when returning job.detailed_status.details_path

What does this MR do and why?

In MR !186354 (merged), a permission check was added to the details_path method that verifies if a user can read_build before returning the path. However, this is redundant and causes issues because:

  1. The frontend already uses the has_details? method to determine whether to display build status information as a link
  2. This permission check prevents the workaround for the bug described in issue #427969

References

Screenshots or screen recordings

Screenshot_2025-04-23_at_12.10.20_PM

How to set up and validate locally

  1. Login as root and add another user as guest to the gitlab-org/gitlab-test project.
  2. Login as the user added as guest to the project
  3. Go to /gitlab-org/gitlab-test/-/jobs
  4. Verify that the jobs statuses are displayed as a link

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.

Edited by Eugie Limpin

Merge request reports

Loading