Page MenuHomePhabricator

Keep the new "%P" query conversion out of the service call profiler by unmasking later
ClosedPublic

Authored by epriestley on Nov 15 2018, 1:28 PM.
Tags
None
Referenced Files
F14147942: D19811.diff
Wed, Dec 4, 8:36 AM
Unknown Object (File)
Sat, Nov 23, 5:12 AM
Unknown Object (File)
Thu, Nov 14, 7:45 AM
Unknown Object (File)
Fri, Nov 8, 6:23 PM
Unknown Object (File)
Fri, Nov 8, 6:09 PM
Unknown Object (File)
Fri, Nov 8, 2:28 PM
Unknown Object (File)
Fri, Nov 8, 2:28 PM
Unknown Object (File)
Fri, Nov 8, 12:35 PM
Subscribers
None

Details

Summary

Ref T13216. Ref T13217. Ref T6960. Although the new "%P" (password/secret) conversion can mask values, we currently convert it too early in the stack, and pass the unmasked query to the service call profiler. This means the unmasked query appears in the service call profiler, etc.

Instead, unmask slightly later in the stack so the call profiler gets the masked string.

Test Plan

See next change.

Diff Detail

Repository
rPHU libphutil
Branch
query6
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 21162
Build 28774: Run Core Tests
Build 28773: arc lint + arc unit