Opened 7 years ago

Closed 3 years ago

#5512 closed defect (fixed)

Disable Mozilla Persona plugin

Reported by: ayleph Owned by:
Priority: trivial Milestone:
Component: programming Keywords: persona, plugin
Cc: Parent Tickets:

Description

Mozilla Persona has been decommissioned. We should disable or remove the built-in Persona plugin. I like the idea of disabling it but leaving it there, so we have more examples of plugins. But I'd also be okay with just removing it altogether.

Here's something I had in mind for disabling the plugin.

diff --git a/mediagoblin/plugins/persona/__init__.py b/mediagoblin/plugins/persona/__init__.py
index 8fab726ae..2b8928c6b 100644
--- a/mediagoblin/plugins/persona/__init__.py
+++ b/mediagoblin/plugins/persona/__init__.py
@@ -25,10 +25,21 @@ from mediagoblin.tools import pluginapi
 from mediagoblin.tools.staticdirect import PluginStatic
 from mediagoblin.tools.translate import pass_to_ugettext as _
 
+from mediagoblin.init import ImproperlyConfigured
+
 PLUGIN_DIR = os.path.dirname(__file__)
 
 
 def setup_plugin():
+    """
+    Mozilla Persona has been decommissioned. This plugin exists for 
+    historical reasons and serves as an example of an authentication 
+    plugin.
+    """
+    raise ImproperlyConfigured(
+        _('Mozilla Persona has been decommissioned. MediaGoblin cannot '
+          'be started with the Persona plugin enabled.'))
+
     config = pluginapi.get_config('mediagoblin.plugins.persona')
 
     routes = [

Change History (3)

comment:1 by ShawnRisk, 5 years ago

Status: newreview

This should be made into a patch.

comment:2 by Ben Sturmfels, 4 years ago

We do have the magic of version control though, so I suggest we remove the plugin entirely and add a note to the plugin dev documentation that encourages authentication plugin authors to refer to the Persona plugin we had in MediaGoblin v0.10.0.

comment:3 by Ben Sturmfels, 3 years ago

Resolution: fixed
Status: reviewclosed

We've now removed the persona plugin and added a note in the pluginwriters docs.

Note: See TracTickets for help on using tickets.