Homec4science

Convert Phabricator to handle "%s" / "%B" properly

Authored by epriestley <git@epriestley.com> on Feb 24 2014, 01:20.

Description

Convert Phabricator to handle "%s" / "%B" properly

Summary:
Ref T1191. I believe we only have three meaningful binary fields across all applications:

  • The general cache may contain gzipped content.
  • The file storage blob may contain arbitrary binary content.
  • The Passphrase secret can store arbitrary binary data (although it currently never does).

This adds Lisk config for binary fields, and uses %B where necessary.

Test Plan:

  • Added and executed unit tests.
  • Forced file uploads to use MySQL, uploaded binaries.
  • Disabled the CONFIG_BINARY on the file storage blob and tried again, got an appropraite failure.
  • Tried to register with an account containing a G-Clef, and was stopped before the insert.

Reviewers: btrahan, arice

Reviewed By: arice

CC: arice, chad, aran

Maniphest Tasks: T1191

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

Details

Committed
epriestley <git@epriestley.com>Feb 24 2014, 01:20
Pushed
aubortJan 31 2017, 17:16
Parents
rPH70b008d18da5: Add test coverage that our definition of BMP agrees with MySQL
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPHa298a79bdac6: Convert Phabricator to handle "%s" / "%B" properly (authored by epriestley <git@epriestley.com>).Feb 24 2014, 01:20