Skip to content
  • epriestley's avatar
    Remove an old digest in Celerity code and some obsolete configuration options · afa69eed
    epriestley authored
    Summary:
    Ref T12509. This upgrades a `weakDigest()` callsite to SHA256-HMAC and removes three config options:
    
      - `celerity.resource-hash`: Now hard-coded, since the use case for ever adjusting it was very weak.
      - `celerity.enable-deflate`: Intended to make cache inspection easier, but we haven't needed to inspect caches in ~forever.
      - `celerity.minify`: Intended to make debugging minification easier, but we haven't needed to debug this in ~forever.
    
    In the latter two cases, the options were purely developer-focused, and it's easy to go add an `&& false` somewhere in the code if we need to disable these features to debug something, but the relevant parts of the code basically work properly and never need debugging. These options were excessively paranoid, based on the static resource enviroment at Facebook being far more perilous.
    
    The first case theoretically had end-user utility for fixing stuck content caches. In modern ...
    afa69eed