Page MenuHomePhabricator

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

Authored by epriestley on Nov 15 2018, 1:28 PM.



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

rPHU libphutil
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

epriestley created this revision.Nov 15 2018, 1:28 PM
epriestley requested review of this revision.Nov 15 2018, 1:29 PM
amckinley accepted this revision.Nov 16 2018, 7:22 PM
This revision is now accepted and ready to land.Nov 16 2018, 7:22 PM
This revision was automatically updated to reflect the committed changes.