HomePhorge

Fix two minor issues

Description

Fix two minor issues

Summary:
Ref T7811. Fixes two minor issues I observed in the cluster:

  • Sometimes APC doesn't give us key names. Not sure exactly what's up here, but we can do a better job with this.
  • The % in 25% actually needs more escaping, since it's interpreted by both pht() (immediately) and console_format() (later).

Test Plan:

  • First one is just from an error log, not sure how to repro offhand.
  • Ran bin/phd help start for the second one.

Reviewers: btrahan

Reviewed By: btrahan

Subscribers: epriestley

Maniphest Tasks: T7814, T7811

Differential Revision: https://secure.phabricator.com/D12395

Details

Provenance
epriestleyAuthored on Apr 13 2015, 11:11 AM
themackabuPushed on Mar 25 2025, 8:07 PM
Parents
rPd56012ce262b: Move Conpherence when warning is present
Branches
Unknown
Tags
Unknown

Event Timeline