Opened 8 years ago
Closed 5 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 , 7 years ago
| Status: | new → review |
|---|
comment:2 by , 5 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 , 5 years ago
| Resolution: | → fixed |
|---|---|
| Status: | review → closed |
We've now removed the persona plugin and added a note in the pluginwriters docs.

This should be made into a patch.