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
F14058049: D11532.diff
Sun, Nov 17, 9:10 AM
F13998392: D11532.diff
Thu, Oct 24, 9:18 AM
F13998094: D11532.diff
Thu, Oct 24, 7:33 AM
F13981043: D11532.id27745.diff
Oct 19 2024, 1:56 PM
F13962374: D11532.id.diff
Oct 15 2024, 9:42 AM
F13957390: D11532.id27798.diff
Oct 14 2024, 9:25 AM
Unknown Object (File)
Oct 12 2024, 1:04 AM
Unknown Object (File)
Sep 17 2024, 7:35 AM
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.