Refactor artifact api endpoint
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.
Does this MR meet the acceptance criteria?
- [-] Changelog entry added, if necessary
- [-] Documentation created/updated
- [-] Tests added for this feature/bug
-
Conforms to the code review guidelines -
Conforms to the merge request performance guidelines -
Conforms to the style guides - [-] Conforms to the database guides
- [-] Link to e2e tests MR added if this MR has Requires e2e tests label. See the Test Planning Process.
-
Security reports checked/validated by reviewer