Page MenuHomePhabricator

Give sessions real PHIDs and slightly modernize session queries
ClosedPublic

Authored by epriestley on Dec 13 2018, 6:34 PM.
Tags
None
Referenced Files
F18786330: D19881.diff
Oct 14 2025, 6:33 PM
F18611010: D19881.diff
Sep 14 2025, 5:56 AM
F18604078: D19881.diff
Sep 13 2025, 5:56 PM
F18510009: D19881.id.diff
Sep 5 2025, 3:43 AM
F18503630: D19881.diff
Sep 4 2025, 11:15 PM
F18187152: D19881.id47484.diff
Aug 16 2025, 7:24 PM
F18124426: D19881.id.diff
Aug 13 2025, 12:38 PM
F18112842: D19881.diff
Aug 12 2025, 9:24 PM
Subscribers
Restricted Owners Package

Details

Summary

Ref T13222. See PHI873. I'm preparing to introduce a new MFA "Challenge" table which stores state about challenges we've issued (to bind challenges to sessions and prevent most challenge reuse).

This table will reference sessions (since each challenge will be bound to a particular session) but sessions currently don't have PHIDs. Give them PHIDs and slightly modernize some related code.

Test Plan
  • Ran migrations.
  • Verified table got PHIDs.
  • Used var_dump() to dump an organic user session.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Owners added a subscriber: Restricted Owners Package.Dec 13 2018, 6:34 PM
This revision is now accepted and ready to land.Dec 13 2018, 7:48 PM
This revision was automatically updated to reflect the committed changes.