Page MenuHomePhabricator

Modularize generation of supplemental login messages
ClosedPublic

Authored by epriestley on Sep 4 2015, 4:30 PM.
Tags
None
Referenced Files
F14360749: D14057.diff
Fri, Dec 20, 10:38 AM
Unknown Object (File)
Tue, Dec 17, 11:21 AM
Unknown Object (File)
Tue, Dec 17, 10:45 AM
Unknown Object (File)
Fri, Dec 13, 7:44 PM
Unknown Object (File)
Thu, Dec 5, 1:42 PM
Unknown Object (File)
Fri, Nov 29, 5:34 PM
Unknown Object (File)
Thu, Nov 28, 1:50 PM
Unknown Object (File)
Wed, Nov 27, 9:40 PM
Subscribers
None

Details

Summary

Ref T9346. This mostly allows us to give users additional advice based on which instance they are trying to log in to in the Phacility cluster.

It's also slightly more flexible than auth.login-message was, and maybe we'll add some more hooks here eventually.

This feels like it's a sidegrade in complexity rather than really an improvement, but not too terrible.

Test Plan
  • Wrote the custom handler in T9346 to replicate old config functionality.
  • Wrote a smart handler for Phacility that can provide context-sensitive messages based on which OAuth client you're trying to use.

See new message box at top (implementation in next diff):

Screen Shot 2015-09-04 at 9.25.34 AM.png (1×1 px, 163 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Modularize generation of supplemental login messages.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
epriestley added a child revision: Restricted Differential Revision.Sep 4 2015, 4:36 PM
chad edited edge metadata.
This revision is now accepted and ready to land.Sep 4 2015, 5:25 PM
This revision was automatically updated to reflect the committed changes.