Page MenuHomePhabricator
Differential D7978

Remove session limits and sequencing
ClosedPublic
Actions

Authored by epriestley on Jan 15 2014, 10:48 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Apr 20, 6:18 PM
Unknown Object (File)
Tue, Apr 16, 3:42 PM
Unknown Object (File)
Thu, Apr 11, 10:18 AM
Unknown Object (File)
Tue, Apr 9, 2:44 AM
Unknown Object (File)
Sat, Apr 6, 3:53 PM
Unknown Object (File)
Sat, Apr 6, 3:53 PM
Unknown Object (File)
Sat, Apr 6, 3:03 PM
Unknown Object (File)
Sat, Apr 6, 2:43 PM
View All Files
Subscribers
aran

Details

Reviewers
btrahan
Maniphest Tasks
T4310: Allow external users to establish real sessions
Restricted Maniphest Task
Commits
Restricted Diffusion Commit
rP2ec45d42a689: Remove session limits and sequencing
Summary

Ref T4310. Fixes T3720. This change:

  • Removes concurrent session limits. Instead, unused sessions are GC'd after a while.
  • Collapses all existing "web-1", "web-2", etc., sessions into "web" sessions.
  • Dramatically simplifies the code for establishing a session (like omg).
Test Plan

Ran migration, checked Sessions panel and database for sanity. Used existing session. Logged out, logged in. Ran Conduit commands.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

btrahan accepted this revision.Jan 16 2014, 1:05 AM
btrahan added inline comments.
src/applications/auth/engine/PhabricatorAuthSessionEngine.php
103

nice catch

epriestley closed this revision.Jan 16 2014, 1:29 AM

Closed by commit rP2ec45d42a689 (authored by @epriestley).

Revision Contents
Changeset List

PathSize
conf/
9 lines
resources/
sql/
autopatches/
26 lines
src/
applications/
auth/
engine/
155 lines
query/
16 lines
config/
check/
6 lines
option/
16 lines

Diff 18056

View Options

conf/default.conf.php

Loading...
View Options

resources/sql/autopatches/20140115.auth.3.unlimit.php

Loading...
View Options

src/applications/auth/engine/PhabricatorAuthSessionEngine.php

Loading...
View Options

src/applications/auth/query/PhabricatorAuthSessionQuery.php

Loading...
View Options

src/applications/config/check/PhabricatorSetupCheckExtraConfig.php

Loading...
View Options

src/applications/config/option/PhabricatorAuthenticationConfigOptions.php

Loading...
Phabricator · Built by Phacility