Page MenuHomePhabricator

Modularize generation of supplemental login messages
ClosedPublic

Authored by epriestley on Sep 4 2015, 4:30 PM.
Tags
None
Referenced Files
F14066810: D14057.diff
Tue, Nov 19, 11:58 AM
F14054132: D14057.diff
Fri, Nov 15, 10:45 PM
F14054130: D14057.diff
Fri, Nov 15, 10:45 PM
F14012839: D14057.id.diff
Fri, Nov 1, 7:45 PM
F13982269: D14057.id33992.diff
Oct 19 2024, 10:01 PM
F13982268: D14057.id.diff
Oct 19 2024, 10:01 PM
F13973485: D14057.id33992.diff
Oct 18 2024, 1:06 AM
F13972291: D14057.id33986.diff
Oct 17 2024, 6:11 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.