Lin Jen-Shin
b07da07c82
Just enforce the output encoding for Ansi2html
...
Fixes https://sentry.gitlap.com/gitlab/gitlabcom/issues/27545/
2017-04-18 17:12:06 +08:00
Lin Jen-Shin
9350b9064c
Set the encoding in c'tor and explain why it's fine
2017-04-17 17:52:15 +08:00
Lin Jen-Shin
f05ab97e1e
Restore nil for stream
2017-04-17 17:12:43 +08:00
Lin Jen-Shin
e7d3fe44f6
Only set the encoding before passing to Ansi2html
2017-04-17 17:10:41 +08:00
Lin Jen-Shin
c26b126502
Make sure @stream.each_line would tag Encoding.default_external
2017-04-17 15:53:09 +08:00
Lin Jen-Shin
ee3b0c3a9a
Make sure we're giving Encoding.default_external
2017-04-17 14:30:42 +08:00
Lin Jen-Shin
0a3c98a7e3
Update tests for the fact that we would skip one line
2017-04-13 22:23:12 +08:00
Lin Jen-Shin
faa5dffbcc
After Trace#limit, we seek to the next line in case
...
of breaking ANSI sequence or Unicode
2017-04-13 21:44:26 +08:00
Z.J. van de Weg
23fbbe0c92
Return nil as coverage instead of a File object
...
Given a valid pipeline job, and a regex which wouldn't match to a jobs
trace, the stream of the trace would return the File object. This was
not the case when it matched a value, as that would have been return
from the block.
Now the `extract_coverage` method returns `nil` if no match was found.
2017-04-12 12:03:39 +02:00
Kamil Trzciński
828d81ee1f
Optimise trace handling code to use streaming instead of full read
2017-04-06 16:20:27 +00:00