Skip to content

Explicitly give Encoding.default_external for trace

username-removed-423915 requested to merge fix-trace-encoding into master

Update:

Since Ci::Ansi2html::Converter would read from @stream directly via each_line, we'll need to setup the encoding for read data. It's somehow using an private API from Gitlab::Ci::Trace::Stream

This is an attempt to fix #30796 (closed), again, but without this I still cannot reproduce the issue. However I think we still need this patch anyway, and we should keep finding why it's failing in the first place.

See #30796 (closed)

Merge request reports