Commit 0e548473 authored by Kamil Trzcinski's avatar Kamil Trzcinski

Fix: CI token removal regression from build trace

parent 9d825723
...@@ -143,12 +143,6 @@ module Ci ...@@ -143,12 +143,6 @@ module Ci
html ||= '' html ||= ''
end end
def trace
if project && read_attribute(:trace).present?
read_attribute(:trace).gsub(project.token, 'xxxxxx')
end
end
def started? def started?
!pending? && !canceled? && started_at !pending? && !canceled? && started_at
end end
...@@ -223,7 +217,7 @@ module Ci ...@@ -223,7 +217,7 @@ module Ci
end end
end end
def trace def raw_trace
if File.exist?(path_to_trace) if File.exist?(path_to_trace)
File.read(path_to_trace) File.read(path_to_trace)
else else
...@@ -232,6 +226,15 @@ module Ci ...@@ -232,6 +226,15 @@ module Ci
end end
end end
def trace
trace = raw_trace
if project && trace.present?
trace.gsub(project.token, 'xxxxxx')
else
trace
end
end
def trace=(trace) def trace=(trace)
unless Dir.exists? dir_to_trace unless Dir.exists? dir_to_trace
FileUtils.mkdir_p dir_to_trace FileUtils.mkdir_p dir_to_trace
......
...@@ -178,6 +178,17 @@ describe Ci::Build do ...@@ -178,6 +178,17 @@ describe Ci::Build do
it { is_expected.to include(text) } it { is_expected.to include(text) }
it { expect(subject.length).to be >= text.length } it { expect(subject.length).to be >= text.length }
end end
context 'if build.trace hides token' do
let(:token) { 'my_secret_token' }
before do
build.project.update_attributes(token: token)
build.update_attributes(trace: token)
end
it { is_expected.to_not include(token) }
end
end end
describe :timeout do describe :timeout do
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment