Page MenuHomePhabricator

Use %B when writing to Harbormaster build logs
ClosedPublic

Authored by hach-que on Jan 28 2015, 1:16 AM.
Tags
None
Referenced Files
F13998392: D11532.diff
Thu, Oct 24, 9:18 AM
F13998094: D11532.diff
Thu, Oct 24, 7:33 AM
F13981043: D11532.id27745.diff
Sat, Oct 19, 1:56 PM
F13962374: D11532.id.diff
Tue, Oct 15, 9:42 AM
F13957390: D11532.id27798.diff
Mon, Oct 14, 9:25 AM
Unknown Object (File)
Sat, Oct 12, 1:04 AM
Unknown Object (File)
Sep 17 2024, 7:35 AM
Unknown Object (File)
Sep 7 2024, 11:58 PM
Subscribers

Details

Summary

Fixes T7007. Using %B permits non-UTF8 data to be appended to Harbormaster build logs. Since we're not really in control of the processes Harbormaster is running remotely, and since they may output invalid UTF8 data, we should store the invalid data instead of failing the build (due to UTF8 exception).

Test Plan

@epriestley said this was the right fix, though I haven't tested it on our production system which actually exhibits the issue yet.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

hach-que retitled this revision from to Using %B when writing to Harbormaster build logs.
hach-que updated this object.
hach-que edited the test plan for this revision. (Show Details)
hach-que added a reviewer: epriestley.
hach-que added a subscriber: epriestley.
hach-que retitled this revision from Using %B when writing to Harbormaster build logs to Use %B when writing to Harbormaster build logs.Jan 28 2015, 1:18 AM
hach-que edited edge metadata.
epriestley edited edge metadata.
This revision is now accepted and ready to land.Jan 28 2015, 7:11 PM
This revision was automatically updated to reflect the committed changes.