Home GnuPG

Assume utf8mb4 support
0135e57181a9Unpublished

Unpublished Commit ยท Learn More

Repository Importing: This repository is still importing.

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

Provenance
epriestley <git@epriestley.com>Authored on Nov 7 2014, 9:29 PM
Parents
rPHUTIL43a0b6d17bb0: Remove history support for console prompts on Windows
Branches
Unknown
Tags
Unknown

Event Timeline

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