From ac7c20cca81c3d23ee66f224030b316bdff2274a Mon Sep 17 00:00:00 2001 From: Patrick Uiterwijk Date: Wed, 8 Jul 2015 12:37:04 +0200 Subject: [PATCH] Only initialize the OpenID IDP when actually enabled This is needed because otherwise it will try to set the database schema version before it read the configuration for providers, which means it will do this in the default (openid.sqlite) database file. If you are running as a non-privileged user (as your should) with the working directory pointing somewhere this user is unable to write, this means it will fail to write this. Note: the working directory is not in the default wsgi file, which means that people using that will not likely hit this bug. Signed-off-by: Patrick Uiterwijk Reviewed-by: Simo Sorce --- ipsilon/providers/openidp.py | 1 - 1 file changed, 1 deletion(-) diff --git a/ipsilon/providers/openidp.py b/ipsilon/providers/openidp.py index 8f74578..dc47faf 100644 --- a/ipsilon/providers/openidp.py +++ b/ipsilon/providers/openidp.py @@ -104,7 +104,6 @@ Provides OpenID 2.0 authentication infrastructure. """ return self.get_config_value('default allowed attributes') def get_tree(self, site): - self.init_idp() self.page = OpenID(site, self) # self.admin = AdminPage(site, self) -- 2.20.1