From 5dcfd7d06754fecb245475cc005c112677693801 Mon Sep 17 00:00:00 2001
From: Tomasz Maczukin <tomasz@maczukin.pl>
Date: Mon, 11 Jan 2016 15:49:25 +0100
Subject: [PATCH] Add TODO notice to build trace feature in CI API

---
 lib/api/builds.rb | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/lib/api/builds.rb b/lib/api/builds.rb
index 8c21754596b..d3f4e33ebbf 100644
--- a/lib/api/builds.rb
+++ b/lib/api/builds.rb
@@ -61,6 +61,10 @@ module API
       #   build_id (required) - The ID of a build
       # Example Request:
       #   GET /projects/:id/build/:build_id/trace
+      #
+      # TODO: We should use `present_file!` and leave this implementation for backward compatibility (when build trace
+      #       is saved in the DB instead of file). But before that, we need to consider how to replace the value of
+      #       `runners_token` with some mask (like `xxxxxx`) when sending trace file directly by workhorse.
       get ':id/builds/:build_id/trace' do
         build = get_build(params[:build_id])
         return not_found!(build) unless build
-- 
GitLab