Homec4science

Assume utf8mb4 support

Authored by epriestley <git@epriestley.com> on Nov 7 2014, 21:29.

Description

Assume utf8mb4 support

Summary:
Ref T1191. This stops throwing an exception for non-BMP data when constructing queries.

  • For installs which have already adjusted, this allows emoji and other 4-byte characters to work.
  • For installs which have not adjusted yet but do have STRICT_ALL_TABLES set, MySQL will raise an error intead ("Incorrect string value...").
  • For installs which have not adjusted and also do not use STRICT_ALL_TABLES, this will silently truncate data. This is potentially a security risk. However, these installs are warned about the risk of leaving STRICT_ALL_TABLES disabled, there's no known way to exploit this issue, and they'll be auto-adjusted soon.

Test Plan: Ran locally and on secure.phabricator.com with this patch for a long time.

Reviewers: btrahan

Reviewed By: btrahan

Subscribers: epriestley

Maniphest Tasks: T1191

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

Details

Committed
epriestley <git@epriestley.com>Nov 7 2014, 21:29
Pushed
aubortMar 17 2017, 12:03
Parents
rPHU43a0b6d17bb0: Remove history support for console prompts on Windows
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPHU0135e57181a9: Assume utf8mb4 support (authored by epriestley <git@epriestley.com>).Nov 7 2014, 21:29