HomePhorge

Validate settings before writing them to the user cache

Description

Validate settings before writing them to the user cache

Summary:
Fixes T11960. In D16998 I removed some code which validated settings on read to improve performance, but lost this replacement validation in shuffling the patch stack.

This restores similar validation before we write the cache. This has the same effect, it's just faster.

Also, bump the cache key to wipe out anything that got bitten (like my account on secure rendering dates wrong).

Test Plan:

  • Edited settings, verified the edits held.
  • Faked invalid settings, saw the check throw exceptions.

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T11960

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

Details

Provenance
epriestleyAuthored on Dec 7 2016, 12:48 PM
themackabuPushed on Mar 25 2025, 8:07 PM
Parents
rPe8d9c676b1a7: Fix a bad link in the database partitioning docs
Branches
Unknown
Tags
Unknown

Event Timeline