Page MenuHomePhabricator

Remove the Persona login method
ClosedPublic

Authored by eadler on Aug 5 2016, 9:32 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Jun 26, 3:23 AM
Unknown Object (File)
Sat, Jun 25, 12:44 PM
Unknown Object (File)
Wed, Jun 22, 4:37 AM
Unknown Object (File)
Sat, Jun 11, 11:36 AM
Unknown Object (File)
Fri, Jun 10, 3:59 PM
Unknown Object (File)
Thu, Jun 9, 2:35 PM
Unknown Object (File)
Tue, Jun 7, 6:57 AM
Unknown Object (File)
Jun 3 2022, 10:52 AM
Subscribers

Details

Summary

Persona is going to be decommed November 30th, 2016.
It is highly unlikely that anyone is currently using persona as a real
login method at this point.

Test Plan

tried locally to add auth adapter.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

eadler retitled this revision from to Remove the Persona login method.
eadler updated this object.
eadler edited the test plan for this revision. (Show Details)

See T10125 for some additional discussion.

In theory, this should also regenerate the actual login sprites with scripts/celerity/generate_sprites.php, although that's kind of a pain (you need optipng installed) and doesn't really matter.

(There's also probably little real value in spriting these anyway, and replacing them with a dynamic CSS file after T11217 provides that capability is probably a good step in the long run.)

Since at least one install does actually use this (Fedora QA) and it costs us very little to keep it around for 3-4 more months until it actually breaks, I'm inclined to just wait until Mozilla terminates it on Nov 30th. I'll accept this upstream after that.

epriestley added a reviewer: epriestley.

The Persona login dialog still sort of works I guess but now has a prominent notice that it's no longer supported, so I think we're good to get rid of it. I've disabled the provider on this install.

This revision is now accepted and ready to land.Dec 5 2016, 11:55 PM
This revision was automatically updated to reflect the committed changes.