Skip to content

Refactor artifact api endpoint

Steve Xuereb requested to merge refactor-artifact-api-endpoint into master

What does this MR do?

For :id/jobs/artifacts/:ref_name/download active record was being used directly, which is not ideal. Use an already existing method that does the same thing that we are doing.

Please merge https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/23538 before this gets merged since this has the changes from https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/23538

Does this MR meet the acceptance criteria?

closes https://gitlab.com/gitlab-org/gitlab-ce/issues/55445

Edited by Grzegorz Bizon

Merge request reports