Page MenuHomePhabricator

Upgrade object reply addresses to SHA256 and remove "phabricator.mail-key"
ClosedPublic

Authored by epriestley on Jan 3 2019, 1:18 PM.
Tags
None
Referenced Files
F18803519: D19945.id47615.diff
Sat, Oct 18, 12:41 AM
F18790218: D19945.id47637.diff
Wed, Oct 15, 1:04 PM
F18771351: D19945.id.diff
Wed, Oct 8, 7:18 PM
F18672311: D19945.diff
Thu, Sep 25, 7:15 AM
F18607608: D19945.diff
Sep 14 2025, 12:05 AM
F18569053: D19945.id47615.diff
Sep 10 2025, 12:57 AM
F18566725: D19945.id.diff
Sep 9 2025, 2:51 PM
F18529422: D19945.diff
Sep 6 2025, 11:29 PM
Subscribers
None

Details

Summary

Ref T12509.

  • Upgrade an old SHA1 to SHA256.
  • Replace an old manually configurable HMAC key with an automatically generated one.

This is generally both simpler (less configuration) and more secure (you now get a unique value automatically).

This causes a one-time compatibility break that invalidates old "Reply-To" addresses. I'll note this in the changelog.

If you leaked a bunch of addresses, you could force a change here by mucking around with phabricator_auth.auth_hmackey, but AFAIK no one has ever used this value to react to any sort of security issue.

(I'll note the possibility that we might want to provide/document this "manually force HMAC keys to regenerate" stuff some day in T6994.)

Test Plan

Grepped for removed config. I'll vet this pathway more heavily in upcoming changes.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable